Changeset 12063 in josm

Timestamp:

2017-05-05T16:15:49+02:00 (7 years ago)

Author:

michael2402

Message:

MoveAction: Fix ds.beginUpdate()/endUpdate() try/finally

File:

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

trunk/src/org/openstreetmap/josm/actions/MoveAction.java

@@ -137 +137 @@
 
         ds.beginUpdate();
-        if (c instanceof MoveCommand && ds.equals(c.getAffectedDataSet())
-                && affectedNodes.equals(((MoveCommand) c).getParticipatingPrimitives())) {
-            ((MoveCommand) c).moveAgain(distx, disty);
-        } else {
-            c = new MoveCommand(selection, distx, disty);
-            Main.main.undoRedo.add(c);
+        try {
+            if (c instanceof MoveCommand && ds.equals(c.getAffectedDataSet())
+                    && affectedNodes.equals(((MoveCommand) c).getParticipatingPrimitives())) {
+                ((MoveCommand) c).moveAgain(distx, disty);
+            } else {
+                c = new MoveCommand(selection, distx, disty);
+                Main.main.undoRedo.add(c);
+            }
+        } finally {
+            ds.endUpdate();
         }
-        ds.endUpdate();
 
         for (Node n : affectedNodes) {