Changeset 12135 in josm

Timestamp:

2017-05-13T14:03:23+02:00 (7 years ago)

Author:

bastiK

Message:

fixed #13100 - NPE on launch (2)

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• MainApplication.java (modified) (1 diff)
• preferences/ToolbarPreferences.java (modified) (3 diffs)

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

@@ -452 +452 @@
         monitor.indeterminateSubTask(tr("Loading plugins"));
         PluginHandler.loadLatePlugins(splash, pluginsToLoad, monitor.createSubTaskMonitor(1, false));
-        toolbar.refreshToolbarControl();
+        GuiHelper.runInEDTAndWait(() -> toolbar.refreshToolbarControl());
     }
 

trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java

@@ -69 +69 @@
 import org.openstreetmap.josm.data.imagery.ImageryLayerInfo;
 import org.openstreetmap.josm.gui.tagging.presets.TaggingPreset;
+import org.openstreetmap.josm.gui.util.GuiHelper;
 import org.openstreetmap.josm.tools.GBC;
 import org.openstreetmap.josm.tools.ImageProvider;
@@ -995 +996 @@
      */
     public ToolbarPreferences() {
-        control.setFloatable(false);
-        control.setComponentPopupMenu(popupMenu);
+        GuiHelper.runInEDTAndWait(() -> {
+            control.setFloatable(false);
+            control.setComponentPopupMenu(popupMenu);
+        });
         Main.pref.addPreferenceChangeListener(e -> {
             if ("toolbar.visible".equals(e.getKey())) {
@@ -1110 +1113 @@
         if (toolbar == null) {
             Main.info(tr("Registered toolbar action without name: {0}",
-            action.getClass().getName()));
+                action.getClass().getName()));
         } else {
             Action r = regactions.get(toolbar);
             if (r != null) {
                 Main.info(tr("Registered toolbar action {0} overwritten: {1} gets {2}",
-                toolbar, r.getClass().getName(), action.getClass().getName()));
+                    toolbar, r.getClass().getName(), action.getClass().getName()));
             }
         }