diff --git a/commons/src/main/java/fr/free/nrw/commons/caching/CacheController.java b/commons/src/main/java/fr/free/nrw/commons/caching/CacheController.java
index acbc28106..2d007b405 100644
--- a/commons/src/main/java/fr/free/nrw/commons/caching/CacheController.java
+++ b/commons/src/main/java/fr/free/nrw/commons/caching/CacheController.java
@@ -46,7 +46,7 @@ public class CacheController {
         List displayCatList = new ArrayList();
         Log.d("Cache", "Points found in quadtree: " + pointsFound);
 
-        //TODO: Potentially flatten catList and iterate thru it to convert into unique Set? But currently just displays one point...
+        //TODO: Make this return a proper flat array
         if (pointsFound.length != 0) {
 
             Log.d("Cache", "Entering for loop");
diff --git a/commons/src/main/java/fr/free/nrw/commons/upload/ShareActivity.java b/commons/src/main/java/fr/free/nrw/commons/upload/ShareActivity.java
index cdb780ab8..be0e360b8 100644
--- a/commons/src/main/java/fr/free/nrw/commons/upload/ShareActivity.java
+++ b/commons/src/main/java/fr/free/nrw/commons/upload/ShareActivity.java
@@ -215,7 +215,7 @@ public  class       ShareActivity
                     apiCall.request(decimalCoords);
                     Log.d("Cache", "displayCatList size 0, calling MWAPI");
                 } else {
-                    //TODO: Set categoryList in MwVolleyApi
+                    //TODO: Set categoryList in MwVolleyApi. Not filling up right. Maybe do global singleton for MwVolleyApi
                     MwVolleyApi.setGpsCat(displayCatList);
                     Log.d("Cache", "Cache found, setting categoryList in MwVolleyApi to " + displayCatList.toString());
                 }