Changeset 1498 for trunk/src/org/expeditee/io/ExpReader.java

Timestamp:

12/05/19 17:23:45 (4 years ago)

Author:

bnemhaus

Message:

Refactored variable name for clarity.

Comments explaining why an exception is being suppressed.

File:

• trunk/src/org/expeditee/io/ExpReader.java (modified) (5 diffs)

trunk/src/org/expeditee/io/ExpReader.java

@@ -20 +20 @@
 
 import java.io.BufferedReader;
+import java.io.FileNotFoundException;
 import java.io.FileReader;
 import java.io.IOException;
@@ -409 +410 @@
 
         public static int getVersion(String fullpath) {
+                if (fullpath == null) return -1;
                 try {
                         BufferedReader reader = new BufferedReader(new FileReader(fullpath));
@@ -425 +427 @@
                         reader.close();
                 } catch (Exception e) {
-                        e.printStackTrace();
+                        //e.printStackTrace();
+                        // When this exception goes off, it is because it is trying to check if a frame that has yet to be saved
+                        // to the filesystem has a version number greater than in memory one.  This only happens for new frames, 
+                        // so is not a problem.
                 }
                 return -1;
@@ -431 +436 @@
         
         public static String redirectTo(String fullPath) {
+                if (fullPath == null) return null;
                 try {
                         BufferedReader reader = new BufferedReader(new FileReader(fullPath));
@@ -442 +448 @@
                         }
                         reader.close();
-                } catch (Exception e) {
-                        e.printStackTrace();
+                } catch (IOException e) {
+                        //e.printStackTrace();
+                        // When this exception goes off, it is because it is trying to check if a frame that has yet to be saved
+                        // to the filesystem has a redirect.  This comes out in teh wash later on when a redirect is next checked.
                 }
                 return null;