Changeset 35976 in osm for applications/editors/josm/plugins/Create_grid_of_ways/src

Timestamp:

2022-06-14T20:11:21+02:00 (3 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.

File:

• applications/editors/josm/plugins/Create_grid_of_ways/src/CreateGridOfWaysPlugin/CreateGridOfWaysAction.java (modified) (3 diffs)

applications/editors/josm/plugins/Create_grid_of_ways/src/CreateGridOfWaysPlugin/CreateGridOfWaysAction.java

@@ -16 +16 @@
 import org.openstreetmap.josm.command.SequenceCommand;
 import org.openstreetmap.josm.data.UndoRedoHandler;
+import org.openstreetmap.josm.data.coor.ILatLon;
 import org.openstreetmap.josm.data.coor.LatLon;
 import org.openstreetmap.josm.data.osm.DataSet;
@@ -85 +86 @@
         int c1=0,c2;
         double latDif,lonDif;
-        LatLon llc = nodeCommon.getCoor();
+        ILatLon llc = nodeCommon;
         for (Node n1 : nodesWay1) {
-            LatLon ll1 = n1.getCoor();
-            if (ll1 == null || llc == null) {
+            ILatLon ll1 = n1;
+            if (!ll1.isLatLonKnown() || !llc.isLatLonKnown()) {
                 Logging.warn("Null coordinates: {0} / {1}", n1, nodeCommon);
                 continue;
@@ -106 +107 @@
                     continue;
                 }
-                LatLon ll2 = n2.getCoor();
+                ILatLon ll2 = n2;
                 Node nodeOfGrid = new Node(new LatLon(ll2.lat()+latDif, ll2.lon()+lonDif));
                 cmds.add(new AddCommand(ds, nodeOfGrid));