Changeset 11029 in josm for trunk/src

Timestamp:

2016-09-19T19:21:49+02:00 (8 years ago)

Author:

michael2402

Message:

MapView: Use AtomicBoolean instead of custom synchronization.

File:

• trunk/src/org/openstreetmap/josm/gui/MapView.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/gui/MapView.java

@@ -29 +29 @@
 import java.util.TreeSet;
 import java.util.concurrent.CopyOnWriteArrayList;
+import java.util.concurrent.atomic.AtomicBoolean;
 
 import javax.swing.AbstractButton;
@@ -215 +216 @@
     private final transient List<Layer> nonChangedLayers = new ArrayList<>();
     private int lastViewID;
-    private boolean paintPreferencesChanged = true;
+    private AtomicBoolean paintPreferencesChanged = new AtomicBoolean(true);
     private Rectangle lastClipBounds = new Rectangle();
     private transient MapMover mapMover;
@@ -466 +467 @@
         }
 
-        boolean canUseBuffer;
-
-        synchronized (this) {
-            canUseBuffer = !paintPreferencesChanged;
-            paintPreferencesChanged = false;
-        }
-        canUseBuffer = canUseBuffer
+        boolean canUseBuffer = !paintPreferencesChanged.getAndSet(false)
                 && nonChangedLayers.size() <= nonChangedLayersCount
                 && lastViewID == getViewID()
@@ -711 +706 @@
     @Override
     public void preferenceChanged(PreferenceChangeEvent e) {
-        synchronized (this) {
-            paintPreferencesChanged = true;
-        }
+        paintPreferencesChanged.set(true);
     }