Changeset 28614 in osm for applications/editors/josm/plugins/opendata/src/org

Timestamp:

2012-08-24T01:27:35+02:00 (12 years ago)

Author:

donvip

Message:

[josm_opendata] See #josm7980 - Fix memory leak, commit initial work on new tools button

Location:

applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core

Files:

• actions/ToolsAction.java (added)
• gui/OdDialog.java (modified) (4 diffs)

applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/OdDialog.java

@@ -23 +23 @@
 import org.openstreetmap.josm.gui.layer.OsmDataLayer;
 import org.openstreetmap.josm.plugins.opendata.core.OdConstants;
+import org.openstreetmap.josm.plugins.opendata.core.actions.ToolsAction;
 import org.openstreetmap.josm.plugins.opendata.core.layers.OdDataLayer;
 import org.openstreetmap.josm.plugins.opendata.core.layers.OdLayer;
@@ -33 +34 @@
     private final SideButton downloadButton;
     private final SideButton diffButton;
+    private final SideButton toolsButton;
     
     private final List<SideButton> buttons;
@@ -83 +85 @@
                                 selectButton = new SideButton(new SelectAction()), 
                                 downloadButton = new SideButton(new DownloadAction()), 
-                                diffButton = new SideButton(new DiffAction())
+                                diffButton = new SideButton(new DiffAction()),
+                                toolsButton = new SideButton(new ToolsAction())
         });
                 
@@ -133 +136 @@
         public void layerRemoved(Layer oldLayer) {
         }
+
+    @Override
+    public void destroy() {
+        super.destroy();
+        MapView.removeLayerChangeListener(this);
+    }
 }