Changeset 14048 in osm for applications/editors/josm/plugins/terracer

Timestamp:

2009-03-09T15:46:56+01:00 (15 years ago)

Author:

zere

Message:

Fixed bug in segment length method - index out of bounds.

File:

• applications/editors/josm/plugins/terracer/src/terracer/TerracerAction.java (modified) (2 diffs)

applications/editors/josm/plugins/terracer/src/terracer/TerracerAction.java

@@ -321 +321 @@
 
         /**
-         * Calculate the length of a side (from node i to i+1) in a way.
+         * Calculate the length of a side (from node i to i+1) in a way. This assumes that
+         * the way is closed, but I only ever call it for buildings.
          */
         private double sideLength(Way w, int i) {
                 Node a = w.nodes.get(i);
-                Node b = w.nodes.get(i+1);
+                Node b = w.nodes.get((i+1) % (w.nodes.size() - 1));
                 return a.coor.greatCircleDistance(b.coor);
         }
@@ -369 +370 @@
 
         /**
-         * Calculate "sideness" metric for each segment in a way.
+         * Calculate "sideness" metric for each segment in a way.  
          */
         private double[] calculateSideness(Way w) {