Changeset 12290 in josm


Ignore:
Timestamp:
2017-06-01T11:20:26+02:00 (2 years ago)
Author:
Don-vip
Message:

sonar - squid:S1199 - Nested code blocks should not be used

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java

    r11932 r12290  
    675675        switch (feature) {
    676676            case "user-agent":
    677             {
    678677                String s = Cascade.convertTo(val, String.class);
    679678                return "josm".equals(s);
    680             }
    681679            case "min-josm-version":
    682             {
    683                 Float v = Cascade.convertTo(val, Float.class);
    684                 return v != null && Math.round(v) <= Version.getInstance().getVersion();
    685             }
     680                Float min = Cascade.convertTo(val, Float.class);
     681                return min != null && Math.round(min) <= Version.getInstance().getVersion();
    686682            case "max-josm-version":
    687             {
    688                 Float v = Cascade.convertTo(val, Float.class);
    689                 return v != null && Math.round(v) >= Version.getInstance().getVersion();
    690             }
     683                Float max = Cascade.convertTo(val, Float.class);
     684                return max != null && Math.round(max) >= Version.getInstance().getVersion();
    691685            default:
    692686                return false;
Note: See TracChangeset for help on using the changeset viewer.