Changeset 18899 in josm for trunk/src

Timestamp:

2023-11-15T19:00:25+01:00 (13 months ago)

Author:

stoecker

Message:

combine two loops (remove sonar warning)

File:

• trunk/src/org/openstreetmap/josm/gui/layer/MainLayerManager.java (modified) (1 diff)

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

@@ -383 +383 @@
         remainingLayers.remove(except);
 
-        // First look for visible data layer
-        for (Layer layer : remainingLayers) {
-            if (layer instanceof OsmDataLayer && layer.isVisible()) {
-                return layer;
-            }
-        }
-
-        // Then any data layer
+        // First look for visible data layer (and store first data layer for later)
+        Layer osmlayer = null;
         for (Layer layer : remainingLayers) {
             if (layer instanceof OsmDataLayer) {
-                return layer;
-            }
-        }
+                if (layer.isVisible()) {
+                    return layer;
+                } else if (osmlayer == null) {
+                    osmlayer = layer;
+                }
+            }
+        }
+
+        // Then any data layer
+        if (osmlayer != null)
+            return osmlayer;
 
         // Then any layer