Changeset 11932 in josm for trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/NodeElement.java

Timestamp:

2017-04-16T20:54:54+02:00 (7 years ago)

Author:

Don-vip

Message:

sonar - squid:S1941 - Variables should not be declared before they are relevant

File:

• trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/NodeElement.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/NodeElement.java

@@ -79 +79 @@
 
     private static NodeElement create(Environment env, float defaultMajorZindex, boolean allowDefault) {
-        Cascade c = env.mc.getCascade(env.layer);
-
         MapImage mapImage = createIcon(env);
         Symbol symbol = null;
@@ -87 +85 @@
         }
 
-        RotationAngle rotationAngle = createRotationAngle(env);
-
         // optimization: if we neither have a symbol, nor a mapImage
         // we don't have to check for the remaining style properties and we don't
@@ -94 +90 @@
         if (!allowDefault && symbol == null && mapImage == null) return null;
 
+        Cascade c = env.mc.getCascade(env.layer);
+        RotationAngle rotationAngle = createRotationAngle(env);
         return new NodeElement(c, mapImage, symbol, defaultMajorZindex, rotationAngle);
     }