Changeset 37688 for main

Timestamp:

2023-04-20T16:21:53+12:00 (12 months ago)

Author:

anupama

Message:

Some improvements to comments that don't belong with upcoming commits

File:

• main/trunk/gli/src/org/greenstone/gatherer/file/FileAssociationManager.java (modified) (2 diffs)

main/trunk/gli/src/org/greenstone/gatherer/file/FileAssociationManager.java

@@ -89 +89 @@
             } else if(StaticStrings.LINUX_OPEN_COMMAND == null) {
                 // StaticStrings.LINUX_OPEN_COMMAND not set yet: if we never tried to
-                // work out the default linux file launch command yet, then we'll test
-                // whether any known linux open command is available on this linux.
+                // work out the default linux file launch command yet, then we'll now
+                // test whether any known linux open command is available on this linux.
 
                 // xdg-open comes pre-installed on Ubuntu linux, but not on all linux
@@ -110 +110 @@
 
                 // either we have a default linux open command or not
-                // If we don't, set it to empty, so we don't work out
-                // linux value for each file extension (for each time
-                // FileAssociationManager is called) hereafter.
+                // If we don't, set it to empty, so we don't try working out
+                // the linux open command value for each file extension (for
+                // each iteration of the FileAssociationManager loop) hereafter.
                 if(StaticStrings.LINUX_OPEN_COMMAND == null) {
                 StaticStrings.LINUX_OPEN_COMMAND = "";
                 }
             }
-            // else StaticStrings.LINUX_OPEN_COMMAND = ""; which means
-            // we could never work out a feasible file open command on this linux
+            // else StaticStrings.LINUX_OPEN_COMMAND = ""; which means at some point
+            // we'd tried but failed to work out a feasible file open command for this linux
             }
         }