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

Timestamp:

2020-12-08T11:03:48+01:00 (4 years ago)

Author:

GerdP

Message:

see #19885: memory leak with "temporary" objects in validator and actions

• always call way.setNodes(null)

File:

• trunk/src/org/openstreetmap/josm/actions/mapmode/ExtrudeAction.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/actions/mapmode/ExtrudeAction.java

@@ -713 +713 @@
             // we only need to change the way if its node list was really modified
             cmds.add(new ChangeNodesCommand(selectedSegment.way, wnew.getNodes()));
-            wnew.setNodes(null); // see #19885
-        }
+        }
+        wnew.setNodes(null); // see #19885
         Command c = new SequenceCommand(tr("Extrude Way"), cmds);
         UndoRedoHandler.getInstance().add(c);