Changeset 26997 in osm

Timestamp:

2011-10-31T19:06:59+01:00 (13 years ago)

Author:

mzdila

Message:

Oon't throw NPE if the simplification leads to removing all nodes. Instead leave the way alone.

File:

• applications/editors/josm/plugins/simplifyarea/src/sk/zdila/josm/plugin/simplify/SimplifyAreaAction.java (modified) (2 diffs)

applications/editors/josm/plugins/simplifyarea/src/sk/zdila/josm/plugin/simplify/SimplifyAreaAction.java

@@ -230 +230 @@
                 Node node2 = null;
 
+                final int len = nodes.size();
+                if (len == 0) {
+                    break;
+                }
+
                 // find smallest distance
-                for (int i = 0, len = nodes.size(); i <= len; i++) {
+                for (int i = 0; i <= len; i++) {
                     final Node n1 = nodes.get(i % len);
                     final Node n2 = nodes.get((i + 1) % len);
@@ -354 +359 @@
             final int size2 = nodes.size();
 
+            if (size2 == 0) {
+                break;
+            }
+
             for (int i = 0, len = size2 + (closed ? 2 : 1); i < len; i++) {
                 final int index = i % size2;