Changeset 34214 in osm for applications/editors

Timestamp:

2018-05-22T23:11:11+02:00 (7 years ago)

Author:

donvip

Message:

fix #josm8256, fix #josm14963 - call correct Layer API and implement own ThreadPoolExecutor like Mapillary does

File:

• applications/editors/josm/plugins/mapdust/src/org/openstreetmap/josm/plugins/mapdust/MapdustPlugin.java (modified) (4 diffs)

applications/editors/josm/plugins/mapdust/src/org/openstreetmap/josm/plugins/mapdust/MapdustPlugin.java

@@ -36 +36 @@
 import java.util.ArrayList;
 import java.util.List;
+import java.util.concurrent.ArrayBlockingQueue;
+import java.util.concurrent.ThreadPoolExecutor;
+import java.util.concurrent.TimeUnit;
 
 import javax.swing.JOptionPane;
@@ -79 +82 @@
         MapdustUpdateObserver, MapdustBugObserver {
 
+    /** Executor that will run the updates. */
+    private static ThreadPoolExecutor executor = new ThreadPoolExecutor(
+      3, 5, 100, TimeUnit.SECONDS, new ArrayBlockingQueue<>(100), new ThreadPoolExecutor.DiscardPolicy());
+
     /** The graphical user interface of the plug-in */
     private MapdustGUI mapdustGUI;
@@ -532 +539 @@
      */
     private void updatePluginData() {
-        MainApplication.worker.execute(new Runnable() {
-            @Override
-            public void run() {
-                updateMapdustData();
-            }
-        });
+        executor.execute(() -> updateMapdustData());
     }
 
@@ -596 +598 @@
                     /* MapDust data was changed */
                     mapdustGUI.update(mapdustBugList, this);
-                    mapdustLayer.destroy();
+                    mapdustLayer.invalidate();
                     mapdustLayer.update(mapdustGUI, mapdustBugList);
                     needRepaint = true;