Changeset 19107 in josm

Timestamp:

2024-06-17T15:25:41+02:00 (18 months ago)

Author:

taylor.smock

Message:

Replace most calls for getNodes in VectorWay with direct calls to underlying list

This is primarily to avoid new unmodifiable collection instantiations where we
don't need a copy of the list. This significantly reduces the cost of the
post-processing merge ways step. Specifically, there is a 55% drop in the memory
cost, and 90% fewer CPU cycles.

File:

• trunk/src/org/openstreetmap/josm/data/vector/VectorWay.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/vector/VectorWay.java

@@ -57 +57 @@
     @Override
     public int getNodesCount() {
-        return this.getNodes().size();
+        return this.nodes.size();
     }
 
     @Override
     public VectorNode getNode(int index) {
-        return this.getNodes().get(index);
+        return this.nodes.get(index);
     }
 
@@ -83 +83 @@
     @Override
     public List<Long> getNodeIds() {
-        return this.getNodes().stream().map(VectorNode::getId).collect(Collectors.toList());
+        return this.nodes.stream().map(VectorNode::getId).collect(Collectors.toList());
     }
 
     @Override
     public long getNodeId(int idx) {
-        return this.getNodes().get(idx).getId();
+        return this.nodes.get(idx).getId();
     }