Changeset 16187 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/ValidatorDialog.java

Timestamp:

2020-03-21T21:27:36+01:00 (6 years ago)

Author:

Don-vip

Message:

fix #18962 - introduce DataSet.update to avoid repetitive begin/endUpdate statements

File:

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

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

@@ -694 +694 @@
          */
         private void tryUndo() {
-            final DataSet ds = MainApplication.getLayerManager().getActiveDataSet();
-            int i = fixCommands.size() - 1;
-            ds.beginUpdate();
-            for (; i >= 0; i--) {
-                fixCommands.get(i).undoCommand();
-            }
-            ds.endUpdate();
-        }
-
+            MainApplication.getLayerManager().getActiveDataSet().update(() -> {
+                for (int i = fixCommands.size() - 1; i >= 0; i--) {
+                    fixCommands.get(i).undoCommand();
+                }
+            });
+        }
     }
 
@@ -723 +720 @@
         @Override
         public void undoCommand() {
-            getAffectedDataSet().beginUpdate();
-            super.undoCommand();
-            getAffectedDataSet().endUpdate();
+            getAffectedDataSet().update(super::undoCommand);
         }
 
         @Override
         public boolean executeCommand() {
-            getAffectedDataSet().beginUpdate();
-            boolean rc = super.executeCommand();
-            getAffectedDataSet().endUpdate();
-            return rc;
+            return getAffectedDataSet().update(super::executeCommand);
         }
     }