Changeset 2871 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/HistoryDialog.java

Timestamp:

17.01.2010 16:37:24 (2 years ago)

Author:

jttt

Message:

Partially fix situation after last layer removal - most objects still stay in memory but at least there are less references and forgotten listeners

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/HistoryDialog.java (modified) (2 diffs)

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

@@ -64 +64 @@
     protected ShowHistoryAction showHistoryAction;
     protected ReloadAction reloadAction;
+
+    public HistoryDialog() {
+        super(tr("History"), "history", tr("Display the history of all selected items."),
+                Shortcut.registerShortcut("subwindow:history", tr("Toggle: {0}", tr("History")), KeyEvent.VK_H,
+                        Shortcut.GROUP_LAYER, Shortcut.SHIFT_DEFAULT), 150);
+        build();
+        DataSet.selListeners.add(model);
+    }
 
     /**
@@ -128 +136 @@
     }
 
-    public HistoryDialog() {
-        super(tr("History"), "history", tr("Display the history of all selected items."),
-                Shortcut.registerShortcut("subwindow:history", tr("Toggle: {0}", tr("History")), KeyEvent.VK_H,
-                        Shortcut.GROUP_LAYER, Shortcut.SHIFT_DEFAULT), 150);
-        build();
-        DataSet.selListeners.add(model);
+    @Override
+    public void showNotify() {
         HistoryDataSet.getInstance().addHistoryDataSetListener(this);
+    }
+
+    @Override
+    public void hideNotify() {
+        HistoryDataSet.getInstance().removeHistoryDataSetListener(this);
     }