Changeset 2655 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/ChangesetDialog.java

Timestamp:

2009-12-19T10:47:49+01:00 (14 years ago)

Author:

jttt

Message:

Fixed #4161 Major slowdown in recent versions, used correct pattern for listeners realized using CopyOnWriteArrayList

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/ChangesetDialog.java (modified) (5 diffs)

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

@@ -14 +14 @@
 import java.util.List;
 import java.util.Set;
-import java.util.logging.Logger;
 
 import javax.swing.AbstractAction;
@@ -35 +34 @@
 import org.openstreetmap.josm.data.osm.DataSet;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
+import org.openstreetmap.josm.data.osm.event.DatasetEventManager;
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.gui.MapView;
@@ -61 +61 @@
  */
 public class ChangesetDialog extends ToggleDialog{
-    static private final Logger logger = Logger.getLogger(ChangesetDialog.class.getName());
+    //static private final Logger logger = Logger.getLogger(ChangesetDialog.class.getName());
 
     private ChangesetInSelectionListModel inSelectionModel;
@@ -98 +98 @@
 
         ChangesetCache.getInstance().addChangesetCacheListener(inActiveDataLayerModel);
-        MapView.addLayerChangeListener(inActiveDataLayerModel);
 
         DblClickHandler dblClickHandler = new DblClickHandler();
@@ -114 +113 @@
         MapView.removeLayerChangeListener(inSelectionModel);
         DataSet.selListeners.remove(inSelectionModel);
-        MapView.removeLayerChangeListener(inActiveDataLayerModel);
+    }
+
+    @Override
+    public void showNotify() {
+        DatasetEventManager.getInstance().addDatasetListener(inActiveDataLayerModel, true);
+    }
+
+    @Override
+    public void hideNotify() {
+        DatasetEventManager.getInstance().removeDatasetListener(inActiveDataLayerModel);
     }