Changeset 12127 in josm for trunk/src/org/openstreetmap/josm/gui/MainApplication.java

Timestamp:

2017-05-12T19:53:09+02:00 (7 years ago)

Author:

Don-vip

Message:

fix #14783 - fix NPE when loading early plugins. Restored the static Main.mainPanel, no other choice in order to maintain public listener methods for early plugins

File:

• trunk/src/org/openstreetmap/josm/gui/MainApplication.java (modified) (6 diffs)

trunk/src/org/openstreetmap/josm/gui/MainApplication.java

@@ -88 +88 @@
     private static final List<String> COMMAND_LINE_ARGS = new ArrayList<>();
 
+    private static ProgramArguments args;
+    private static boolean skipLoadingPlugins;
+
     private final MainFrame mainFrame;
 
@@ -110 +113 @@
     protected void initializeMainWindow() {
         if (mainFrame != null) {
-            mainFrame.preInitialize();
             panel = mainFrame.getPanel();
             mainFrame.initialize();
@@ -208 +210 @@
      */
     public static void main(final String[] argArray) {
+        // First initializes all stuff that do not require AWT/Swing
+        mainNoGui(argArray);
+        // Then initializes all AWT/Swing stuff
+        mainGui();
+    }
+
+    /**
+     * Initializes all stuff that do not require AWT/Swing.
+     * @param argArray Command-line arguments
+     */
+    private static void mainNoGui(final String[] argArray) {
         I18n.init();
 
         // construct argument table
-        ProgramArguments args = null;
         try {
             args = new ProgramArguments(argArray);
@@ -261 +273 @@
         COMMAND_LINE_ARGS.addAll(Arrays.asList(argArray));
 
-        boolean skipLoadingPlugins = args.hasOption(Option.SKIP_PLUGINS);
+        skipLoadingPlugins = args.hasOption(Option.SKIP_PLUGINS);
         if (skipLoadingPlugins) {
             Main.info(tr("Plugin loading skipped"));
@@ -286 +298 @@
 
         Main.platform.afterPrefStartupHook();
-
+    }
+
+    /**
+     * Initializes all AWT/Swing stuff.
+     */
+    private static void mainGui() {
         FontsManager.initialize();
 
@@ -297 +314 @@
                 !args.hasOption(Option.NO_MAXIMIZE) && Main.pref.getBoolean("gui.maximized", false));
         final MainFrame mainFrame = new MainFrame(contentPanePrivate, geometry);
+        Main.mainPanel = mainFrame.getPanel();
         Main.parent = mainFrame;