Changeset 14837 in josm for trunk/src/org

Timestamp:

2019-03-05T16:08:20+01:00 (6 years ago)

Author:

GerdP

Message:

simplify code and avoid use of CopyList constructor

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• NavigatableComponent.java (modified) (1 diff)
• layer/validation/PaintVisitor.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/NavigatableComponent.java

@@ -1517 +1517 @@
             maxWaySegLenSq *= maxWaySegLenSq;
 
-            Point2D wp1 = getPoint2D(ws.way.getNode(ws.lowerIndex));
-            Point2D wp2 = getPoint2D(ws.way.getNode(ws.lowerIndex+1));
+            Point2D wp1 = getPoint2D(ws.getFirstNode());
+            Point2D wp2 = getPoint2D(ws.getSecondNode());
 
             // is wayseg shorter than maxWaySegLenSq and

trunk/src/org/openstreetmap/josm/gui/layer/validation/PaintVisitor.java

@@ -209 +209 @@
         if (ws.lowerIndex < 0 || ws.lowerIndex + 1 >= ws.way.getNodesCount())
             return;
-        Node a = ws.way.getNodes().get(ws.lowerIndex);
-        Node b = ws.way.getNodes().get(ws.lowerIndex + 1);
-        drawSegment(a, b, color);
+        drawSegment(ws.getFirstNode(), ws.getSecondNode(), color);
     }