Ignore:
Timestamp:
2016-12-13T01:57:37+01:00 (3 years ago)
Author:
Don-vip
Message:

sonar - squid:S1066 - Collapsible "if" statements should be merged

Location:
trunk/src/org/openstreetmap/josm/data/osm
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/osm/FilterMatcher.java

    r10716 r11385  
    134134            }
    135135        } else {
    136             if (filter.mode == SearchMode.replace) {
    137                 if (filter.hiding) {
    138                     hiddenFilters.clear();
    139                     disabledFilters.clear();
    140                 }
     136            if (filter.mode == SearchMode.replace && filter.hiding) {
     137                hiddenFilters.clear();
     138                disabledFilters.clear();
    141139            }
    142140
  • trunk/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

    r11383 r11385  
    10501050            int counter = 0;
    10511051            for (OsmPrimitive o : (OsmPrimitive[]) referrers) {
    1052                 if (dataSet == o.dataSet && o instanceof Way) {
    1053                     if (++counter >= n)
    1054                         return true;
    1055                 }
     1052                if (dataSet == o.dataSet && o instanceof Way && ++counter >= n)
     1053                    return true;
    10561054            }
    10571055            return false;
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/relations/Multipolygon.java

    r11360 r11385  
    645645        PolyData result = null;
    646646        for (PolyData combined : outerPolygons) {
    647             if (combined.contains(inner.poly) != Intersection.OUTSIDE) {
    648                 if (result == null || result.contains(combined.poly) == Intersection.INSIDE) {
    649                     result = combined;
    650                 }
     647            if (combined.contains(inner.poly) != Intersection.OUTSIDE
     648                    && (result == null || result.contains(combined.poly) == Intersection.INSIDE)) {
     649                result = combined;
    651650            }
    652651        }
Note: See TracChangeset for help on using the changeset viewer.