Changeset 32673 for main

Timestamp:

2018-12-12T19:38:38+13:00 (5 years ago)

Author:

ak19

Message:

Experimental. Attempt to fix the Event Dispatch Thread GUI exceptions we often see when using GLI. To reverse this, a few lines in GathererProg.java creating and showing the Gatherer object inside a run() call in invokeLater() need to be returned to happening outside those 2 methods.

File:

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

main/trunk/gli/src/org/greenstone/gatherer/GathererProg.java

@@ -31 +31 @@
 import javax.swing.*;
 import org.greenstone.gatherer.util.Utility;
-
 
 /**
@@ -74 +73 @@
     // We have a local GLI
     Gatherer.setGLIDirectoryPath(System.getProperty("user.dir") + File.separator);
+    
 
-    // Create an instance of the Gatherer class, which will parse the args and prepare the rest of the GLI
-    Gatherer gatherer = new Gatherer(args);
-
-    // Display the GUI immediately
-    gatherer.openGUI();
+    // TESTING circumventing the old GUI event dispatch thread exceptions in GLI. Follows
+    // https://docs.oracle.com/javase/6/docs/api/javax/swing/package-summary.html#threading
+    // referred from Event Dispatch Thread page at https://joel-costigliola.github.io/assertj/assertj-swing-edt.html
+    
+    // Schedule a job for the event-dispatching thread:
+        // creating and showing this application's GUI.
+        SwingUtilities.invokeLater(new Runnable() {
+            public void run() {
+        // These lines used to be done immediate after setGLIDirectoryPath() call above
+        // but are now inside an invokeLater().
+        
+        // Create an instance of the Gatherer class, which will parse the args and prepare the rest of the GLI
+        Gatherer gatherer = new Gatherer(args);
+        
+        // Display the GUI immediately
+        gatherer.openGUI();
+            }
+        });
     }
 }