Changeset 5666

Timestamp:

2003-10-17T04:43:00+13:00 (21 years ago)

Author:

jmt12

Message:

Rewrote code to avoid AppendLineOnlyFileDocument constructor being called twice (which I suppose was only really a problem on existing logs)

File:

• trunk/gli/src/org/greenstone/gatherer/gui/OptionsPane.java (modified) (3 diffs)

trunk/gli/src/org/greenstone/gatherer/gui/OptionsPane.java

@@ -475 +475 @@
     private class FileEntry {
 
-    private Date date;
-    private long last_modified;
-    private String display;
-    private String filename;
-    private String filepath;
-
-    public FileEntry(String filename, String filepath) {
-        this.date = null;
-        this.display = null;
-        this.filename = filename;
-        this.filepath = filepath;
-        this.last_modified = 0L;
-    }
+       private AppendLineOnlyFileDocument current_document;
+       private Date date;
+       private long last_modified;
+       private String display;
+       private String filename;
+       private String filepath;
+       
+       public FileEntry(String filename, String filepath) {
+      this.date = null;
+      this.display = null;
+      this.filename = filename;
+      this.filepath = filepath;
+      this.last_modified = 0L;
+       }
 
     /** returns 0 if the dates are the same, -ve number if the current FileEntry is earlier than the fe FileEntry ...*/
@@ -513 +514 @@
     }
 
-    public AppendLineOnlyFileDocument getDocument() {
-        return new AppendLineOnlyFileDocument(filepath);
+       public AppendLineOnlyFileDocument getDocument() {
+      if(current_document == null) {
+         current_document = new AppendLineOnlyFileDocument(filepath);
+      }
+      return current_document;
     }
 
@@ -537 +541 @@
         }
         catch(Exception error) {
-            error.printStackTrace();
+           System.err.println("Log '" + filepath + "' not found!");
+            Gatherer.printStackTrace(error);
         }
         switch (success) {