From bb7846d6d1fbf6bb5de1ec1fc8a4743d3d5d56b2 Mon Sep 17 00:00:00 2001
From: Brian Sweeney <bsweeney@ciphertechsolutions.com>
Date: Tue, 22 May 2018 08:35:30 -0600
Subject: [PATCH] comments cleanup

---
 .../AllDataSourcesEamDbCommonFilesAlgorithm.java            | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/Core/src/org/sleuthkit/autopsy/commonfilesearch/AllDataSourcesEamDbCommonFilesAlgorithm.java b/Core/src/org/sleuthkit/autopsy/commonfilesearch/AllDataSourcesEamDbCommonFilesAlgorithm.java
index be1aa66d91..f7d48614f0 100644
--- a/Core/src/org/sleuthkit/autopsy/commonfilesearch/AllDataSourcesEamDbCommonFilesAlgorithm.java
+++ b/Core/src/org/sleuthkit/autopsy/commonfilesearch/AllDataSourcesEamDbCommonFilesAlgorithm.java
@@ -111,7 +111,8 @@ public class AllDataSourcesEamDbCommonFilesAlgorithm  extends CommonFilesMetadat
             }
             //Builds a 3rd list which contains instances which are in commonFiles map, uses current case objectId
             if (commonFiles.containsKey(md5)) {
-                // TODO sloppy, but we don't *have* all the information for the rows in the CR, so what do we do?
+                // we don't *have* all the information for the rows in the CR, 
+                //  so we need to consult the present case via the SleuthkitCase object
                 Long objectId = commonFiles.get(md5).getMetadata().iterator().next().getObjectId();
                 if(interCaseCommonFiles.containsKey(md5)) {
                     //Add to intercase metaData
@@ -124,8 +125,6 @@ public class AllDataSourcesEamDbCommonFilesAlgorithm  extends CommonFilesMetadat
                     Md5Metadata md5Metadata = new Md5Metadata(md5, fileInstances);
                     interCaseCommonFiles.put(md5, md5Metadata);
                 }
-            } else {
-                // TODO This should never happen. All current case files with potential matches are in comonFiles Map.
             }
         }
     }
@@ -144,7 +143,6 @@ public class AllDataSourcesEamDbCommonFilesAlgorithm  extends CommonFilesMetadat
     }
 
     private CorrelationCase getCorrelationCaseFromId(int correlationCaseId) throws EamDbException, Exception {
-        //TODO is there a better way???
         for(CorrelationCase cCase : this.dbManager.getCases()){
             if(cCase.getID() == correlationCaseId){
                 return cCase;