Changeset 2869 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/FilterDialog.java

Timestamp:

17.01.2010 11:56:07 (2 years ago)

Author:

jttt

Message:

Removed ToggleDialog.tearDown(). All listeners should be unregistered in hideNotify()

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/FilterDialog.java (modified) (4 diffs)

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

@@ -7 +7 @@
 import java.awt.BorderLayout;
 import java.awt.Component;
-import java.awt.GridLayout;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
@@ -15 +14 @@
 import javax.swing.JCheckBox;
 import javax.swing.JPanel;
-import javax.swing.JPopupMenu;
 import javax.swing.JScrollPane;
 import javax.swing.JTable;
@@ -47 +45 @@
     private SideButton upButton;
     private SideButton downButton;
-    private JPopupMenu popupMenu;
 
     public FilterDialog(){
         super(tr("Filter"), "filter", tr("Filter objects and hide/disable them."),
                 Shortcut.registerShortcut("subwindow:filter", tr("Toggle: {0}", tr("Filter")), KeyEvent.VK_F, Shortcut.GROUP_LAYER, Shortcut.SHIFT_DEFAULT), 162);
-
+        build();
+    }
+
+    @Override
+    public void showNotify() {
         MapView.addLayerChangeListener(this);
-        build();
     }
 
     @Override
-    public void tearDown() {
+    public void hideNotify() {
         MapView.removeLayerChangeListener(this);
     }
@@ -144 +144 @@
                     @Override
                     public String getToolTipText(MouseEvent e) {
-                        String tip = null;
                         java.awt.Point p = e.getPoint();
                         int index = columnModel.getColumnIndexAtX(p.x);