Changeset 14467 in josm for trunk/src

Timestamp:

2018-11-30T08:28:03+01:00 (6 years ago)

Author:

GerdP

Message:

fix #17041 Let OverlappingWays also report ways where exactly one segment overlaps

File:

• trunk/src/org/openstreetmap/josm/data/validation/tests/OverlappingWays.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/validation/tests/OverlappingWays.java

@@ -194 +194 @@
             }
         }
-        if (duplicateWaySegments.size() > 1) {
-            return duplicateWaySegments;
-        } else {
-            return null;
-        }
+        return duplicateWaySegments;
     }
 
@@ -205 +201 @@
 
         final Set<WaySegment> duplicateWaySegment = checkDuplicateWaySegment(w);
-        if (duplicateWaySegment != null) {
+        if (!duplicateWaySegment.isEmpty()) {
             errors.add(TestError.builder(this, Severity.ERROR, DUPLICATE_WAY_SEGMENT)
                     .message(tr("Way contains segment twice"))