Changeset 32039 in osm for applications/editors/josm

Timestamp:

2016-01-31T19:14:39+01:00 (9 years ago)

Author:

nokutu

Message:

Enhanced painting performance and fixed concurrency bug in MapillarySequence class.

Location:

applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary

Files:

• MapillaryLayer.java (modified) (1 diff)
• MapillarySequence.java (modified) (3 diffs)

applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/MapillaryLayer.java

@@ -326 +326 @@
     }
     for (MapillaryAbstractImage imageAbs : this.data.getImages()) {
-      if (imageAbs.isVisible()) {
+      if (imageAbs.isVisible() && Main.map.mapView.contains(Main.map.mapView.getPoint(imageAbs.getLatLon()))) {
         final Point p = mv.getPoint(imageAbs.getLatLon());
         ImageIcon icon = MapillaryPlugin.MAP_ICON;

applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/MapillarySequence.java

@@ -4 +4 @@
 import java.util.ArrayList;
 import java.util.List;
+import java.util.concurrent.CopyOnWriteArrayList;
 
 /**
@@ -30 +31 @@
    */
   public MapillarySequence() {
-    this.images = new ArrayList<>();
+    this.images = new CopyOnWriteArrayList<>();
   }
 
@@ -40 +41 @@
    */
   public MapillarySequence(String key, long createdAt) {
-    this.images = new ArrayList<>();
+    this.images = new CopyOnWriteArrayList<>();
     this.key = key;
     this.createdAt = createdAt;