Changeset 14586 in josm for trunk/src

Timestamp:

2018-12-22T13:07:40+01:00 (5 years ago)

Author:

GerdP

Message:

Fix some findbugs warnings, no functional change

File:

• trunk/src/org/openstreetmap/josm/tools/ShapeClipper.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/tools/ShapeClipper.java

@@ -69 +69 @@
                 points[num++] = x;
                 points[num++] = y;
-            } else if (type != PathIterator.SEG_CLOSE) {
-                    //Logging.warn("unhandled path iterator");
+                // } else if (type != PathIterator.SEG_CLOSE) {
+                //Logging.warn("unhandled path iterator");
             }
             pit.next();
@@ -116 +116 @@
         if (num < 2)
             return null;
-        if (points[0] == points[num - 2] && points[1] == points[num - 1])
+        if (Double.compare(points[0], points[num - 2]) == 0 && Double.compare(points[1], points[num - 1]) == 0) {
             num -= 2;
+        }
         if (num < 6)
             return null;
@@ -154 +155 @@
 
         int countVals = num;
-        if (points[0] == points[num - 2] && points[1] == points[num - 1]) {
+        if (Double.compare(points[0], points[num - 2]) == 0 && Double.compare(points[1], points[num - 1]) == 0) {
             countVals -= 2;
         }
+
         double[] outputList = points;
         double[] input;