Changeset 19508 in josm for trunk/src/org

Timestamp:

2026-02-11T09:15:31+01:00 (41 hours ago)

Author:

stoecker

Message:

PMD fixes

Location:

trunk/src/org/openstreetmap/josm/data

Files:

• imagery/vectortile/mapbox/MapboxVectorCachedTileLoader.java (modified) (1 diff)
• osm/visitor/paint/StyledTiledMapRenderer.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/data/imagery/vectortile/mapbox/MapboxVectorCachedTileLoader.java

@@ -77 +77 @@
         return DEFAULT_DOWNLOAD_JOB_DISPATCHER;
     }
+    
+    public static void shutdown() {
+        DEFAULT_DOWNLOAD_JOB_DISPATCHER.shutdownNow();
+    }
 }

trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledTiledMapRenderer.java

@@ -52 +52 @@
     private int zoom;
     private Consumer<TileZXY> notifier;
-    private final ExecutorService worker;
 
     /**
@@ -66 +65 @@
     public StyledTiledMapRenderer(Graphics2D g, NavigatableComponent nc, boolean isInactiveMode) {
         super(g, nc, isInactiveMode);
-        this.worker = MainApplication.worker;
     }
 
@@ -76 +74 @@
             return;
         }
-        final Executor worker = this.worker;
         final BufferedImage tempImage;
         final Graphics2D tempG2d;
@@ -132 +129 @@
                 // FIXME figure out how to make this thread safe? Probably not necessary, since UI isn't blocked, but it would be a nice to have
                 TileLoader loader = new TileLoader(data, tile, tileSize, new ArrayList<>());
-                worker.execute(loader);
+                MainApplication.worker.execute(loader);
                 if (tImg == null) {
                     this.cache.put(tile, new ImageCache(null, loader, false));
@@ -162 +159 @@
         // Force another render pass if there may be more tiles to render
         if (submittedTile <= 0) {
-            worker.execute(nc::invalidate);
+            MainApplication.worker.execute(nc::invalidate);
         }
         final double percentDrawn = 100 * painted / (double) toRender.size();