Changeset 35976 in osm for applications/editors/josm/plugins/CommandLine

Timestamp:

2022-06-14T20:11:21+02:00 (2 years ago)

Author:

taylor.smock

Message:

see #22104: Remove usages of Node#getCoor where possible

This also accounts for cases where Node has the methods used later,
so a new LatLon is unnecessary.

Location:

applications/editors/josm/plugins/CommandLine/src/org/openstreetmap/josm/plugins/commandline

Files:

• LengthAction.java (modified) (2 diffs)
• PointAction.java (modified) (2 diffs)

applications/editors/josm/plugins/CommandLine/src/org/openstreetmap/josm/plugins/commandline/LengthAction.java

@@ -137 +137 @@
         mousePos = e.getPoint();
         if (nearestNode != null) {
-            drawStartPos = MainApplication.getMap().mapView.getPoint(nearestNode.getCoor());
+            drawStartPos = MainApplication.getMap().mapView.getPoint(nearestNode);
         } else {
             drawStartPos = mousePos;
@@ -186 +186 @@
         MapFrame map = MainApplication.getMap();
         if (nearestNode != null)
-            drawEndPos = map.mapView.getPoint(nearestNode.getCoor());
+            drawEndPos = map.mapView.getPoint(nearestNode);
         else
             drawEndPos = mousePos;

applications/editors/josm/plugins/CommandLine/src/org/openstreetmap/josm/plugins/commandline/PointAction.java

@@ -76 +76 @@
         if (e.getButton() == MouseEvent.BUTTON1) {
             if (isCtrlDown) {
-                if (pointList.size() > 0) {
+                if (!pointList.isEmpty()) {
                     pointList.remove(pointList.size() - 1);
                     updateTextEdit();
@@ -88 +88 @@
                     return;
                 }
-                LatLon coor = node.getCoor();
-                String point = String.valueOf(coor.getX()) + "," + String.valueOf(coor.getY());
+                String point = node.lon() + "," + node.lat();
                 int maxInstances = parentPlugin.currentCommand.parameters.get(parentPlugin.currentCommand.currentParameterNum).maxInstances;
                 if (maxInstances == 1) {