Changeset 5712 in josm for trunk/src/org/openstreetmap

Timestamp:

2013-02-12T23:10:23+01:00 (12 years ago)

Author:

Don-vip

Message:

fix EDT violations

Location:

trunk/src/org/openstreetmap/josm/gui/dialogs

Files:

• RelationListDialog.java (modified) (2 diffs)
• UserListDialog.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java

@@ -74 +74 @@
 import org.openstreetmap.josm.gui.layer.Layer;
 import org.openstreetmap.josm.gui.layer.OsmDataLayer;
+import org.openstreetmap.josm.gui.util.GuiHelper;
 import org.openstreetmap.josm.gui.widgets.DisableShortcutsOnFocusGainedTextField;
 import org.openstreetmap.josm.gui.widgets.ListPopupMenu;
@@ -678 +679 @@
 
         @Override
-        public void selectionChanged(Collection<? extends OsmPrimitive> newSelection) {
-            setEnabled(newSelection != null && !newSelection.isEmpty());
+        public void selectionChanged(final Collection<? extends OsmPrimitive> newSelection) {
+            GuiHelper.runInEDT(new Runnable() {
+                @Override
+                public void run() {
+                    setEnabled(newSelection != null && !newSelection.isEmpty());
+                }
+            });
         }
     }

trunk/src/org/openstreetmap/josm/gui/dialogs/UserListDialog.java

@@ -42 +42 @@
 import org.openstreetmap.josm.gui.layer.Layer;
 import org.openstreetmap.josm.gui.layer.OsmDataLayer;
+import org.openstreetmap.josm.gui.util.GuiHelper;
 import org.openstreetmap.josm.tools.ImageProvider;
 import org.openstreetmap.josm.tools.Shortcut;
@@ -127 +128 @@
     public void refresh(Collection<? extends OsmPrimitive> fromPrimitives) {
         model.populate(fromPrimitives);
-        if(model.getRowCount() != 0) {
-            setTitle(trn("{0} Author", "{0} Authors", model.getRowCount() , model.getRowCount()));
-        } else {
-            setTitle(tr("Authors"));
-        }
+        GuiHelper.runInEDT(new Runnable() {
+            @Override
+            public void run() {
+                if (model.getRowCount() != 0) {
+                    setTitle(trn("{0} Author", "{0} Authors", model.getRowCount() , model.getRowCount()));
+                } else {
+                    setTitle(tr("Authors"));
+                }
+            }
+        });
     }
 
@@ -305 +311 @@
             }
             Collections.sort(data);
-            fireTableDataChanged();
+            GuiHelper.runInEDTAndWait(new Runnable() {
+                @Override
+                public void run() {
+                    fireTableDataChanged();
+                }
+            });
         }