Changeset 35868 in osm

Timestamp:

2021-11-28T08:59:42+01:00 (3 years ago)

Author:

GerdP

Message:

fix java warning StaticAssignmentInConstructor

Location:

applications/editors/josm/plugins/reverter/src/reverter

Files:

• RevertChangesetCommand.java (modified) (1 diff)
• ReverterPlugin.java (modified) (2 diffs)
• ReverterUploadHook.java (modified) (1 diff)

applications/editors/josm/plugins/reverter/src/reverter/RevertChangesetCommand.java

@@ -23 +23 @@
     public RevertChangesetCommand(String name, Collection<Command> sequenz) {
         super(name, sequenz);
-        ReverterPlugin.reverterUsed = true;
+        ReverterPlugin.reverterUsed.set(true);
         setSequenceComplete(true);
     }

applications/editors/josm/plugins/reverter/src/reverter/ReverterPlugin.java

@@ -1 +1 @@
 // License: GPL. For details, see LICENSE file.
 package reverter;
+
+import java.util.concurrent.atomic.AtomicBoolean;
 
 import javax.swing.JMenu;
@@ -15 +17 @@
  */
 public class ReverterPlugin extends Plugin {
-    static boolean reverterUsed;
+    static AtomicBoolean reverterUsed = new AtomicBoolean();
 
     /**

applications/editors/josm/plugins/reverter/src/reverter/ReverterUploadHook.java

@@ -27 +27 @@
     @Override
     public boolean checkUpload(APIDataSet apiDataSet) {
-        if (ReverterPlugin.reverterUsed) {
+        if (ReverterPlugin.reverterUsed.get()) {
             for (Command cmd : UndoRedoHandler.getInstance().getUndoCommands()) {
                 if (isReverterCmd(cmd)) {