Changeset 17486 in josm for trunk/src/org/openstreetmap

Timestamp:

2021-02-08T10:27:47+01:00 (4 years ago)

Author:

GerdP

Message:

see #20014: Tiles constantly reloading

• change synchronization a bit. No idea if it helps.
• simplify code

File:

• trunk/src/org/openstreetmap/josm/gui/layer/AbstractCachedTileSourceLayer.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/layer/AbstractCachedTileSourceLayer.java

@@ -53 +53 @@
             String key = this.getClass().getCanonicalName();
             loaderFactory = loaderFactories.get(key);
-            if (loaderFactory == null) {
-                synchronized (AbstractCachedTileSourceLayer.class) {
+            synchronized (AbstractCachedTileSourceLayer.class) {
+                if (loaderFactory == null) {
                     // check again, maybe another thread initialized factory
                     loaderFactory = loaderFactories.get(key);
@@ -78 +78 @@
      */
     private synchronized ICacheAccess<String, BufferedImageCacheEntry> getCache() {
-        if (cache != null) {
-            return cache;
+        if (cache == null) {
+            cache = JCSCacheManager.getCache(getCacheName(),
+                    0,
+                    getDiskCacheSize(),
+                    CachedTileLoaderFactory.PROP_TILECACHE_DIR.get());
         }
-        cache = JCSCacheManager.getCache(getCacheName(),
-                0,
-                getDiskCacheSize(),
-                CachedTileLoaderFactory.PROP_TILECACHE_DIR.get());
         return cache;
     }