Changeset 2419 in josm for trunk/src/org/openstreetmap

Timestamp:

2009-11-09T08:21:13+01:00 (15 years ago)

Author:

jttt

Message:

Reindex node or way in QuadBuckets after coordinates change

Location:

trunk/src/org/openstreetmap/josm/data/osm

Files:

• DataSet.java (modified) (2 diffs)
• Node.java (modified) (2 diffs)
• Way.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/data/osm/DataSet.java

@@ -756 +756 @@
      * Reindex all nodes and ways after their coordinates were changed. This is a temporary solution, reindexing should
      * be automatic in the future
-     */
+     * @deprecated Reindexing should be automatic
+     */
+    @Deprecated
     public void reindexAll() {
         List<Node> ntmp = new ArrayList<Node>(nodes);
@@ -764 +766 @@
         ways.clear();
         ways.addAll(wtmp);
+    }
+
+    void reindexNode(Node node) {
+        nodes.remove(node);
+        nodes.add(node);
+        for (Way way:OsmPrimitive.getFilteredList(node.getReferrers(), Way.class)) {
+            ways.remove(way);
+            ways.add(way);
+        }
+    }
+
+    void reindexWay(Way way) {
+        ways.remove(way);
+        ways.add(way);
     }
 

trunk/src/org/openstreetmap/josm/data/osm/Node.java

@@ -23 +23 @@
                 this.coor.setCoor(coor);
             }
+            if (getDataSet() != null) {
+                getDataSet().reindexNode(this);
+            }
         }
     }
@@ -37 +40 @@
             } else {
                 coor = new CachedLatLon(eastNorth);
+            }
+            if (getDataSet() != null) {
+                getDataSet().reindexNode(this);
             }
         }

trunk/src/org/openstreetmap/josm/data/osm/Way.java

@@ -59 +59 @@
 
         clearCached();
+        reindex();
     }
 
@@ -287 +288 @@
         newNodes[nodes.length] = n;
         nodes = newNodes;
+        reindex();
     }
 
@@ -309 +311 @@
         newNodes[offs] = n;
         nodes = newNodes;
+        reindex();
     }
 
@@ -320 +323 @@
             }
         }
+        reindex();
         super.setDeleted(deleted);
     }
@@ -352 +356 @@
         return OsmPrimitiveType.WAY;
     }
+
+    private void reindex() {
+        if (getDataSet() != null) {
+            getDataSet().reindexWay(this);
+        }
+    }
 }