Changeset 224 in josm for src/org/openstreetmap/josm/data/osm/visitor/MergeVisitor.java

Timestamp:

2007-05-05T23:33:16+02:00 (17 years ago)

Author:

framm

Message:

Fixed MergeVisitor so that it will never merge between objects that are
part of the group of objects being added - only between pre-existing
objects and those being added.

File:

• src/org/openstreetmap/josm/data/osm/visitor/MergeVisitor.java (modified) (6 diffs)

src/org/openstreetmap/josm/data/osm/visitor/MergeVisitor.java

@@ -29 +29 @@
 
         private final DataSet ds;
+        private final DataSet mergeds;
 
         /**
@@ -43 +44 @@
         private final Map<Segment, Segment> mergedSegments = new HashMap<Segment, Segment>();
 
-        public MergeVisitor(DataSet ds) {
+        public MergeVisitor(DataSet ds, DataSet mergeds) {
                 this.ds = ds;
+                this.mergeds = mergeds;
         }
 
@@ -57 +59 @@
                 Node my = null;
                 for (Node n : ds.nodes) {
-                        if (match(n, other)) {
+                        if (match(n, other) && ((mergeds == null) || (!mergeds.nodes.contains(n)))) {
                                 my = n;
                                 break;
@@ -87 +89 @@
                 Segment my = null;
                 for (Segment ls : ds.segments) {
-                        if (match(other, ls)) {
+                        if (match(other, ls) && ((mergeds == null) || (!mergeds.segments.contains(ls)))) {
                                 my = ls;
                                 break;
                         }
                 }
+                
                 if (my == null)
                         ds.segments.add(other);
@@ -145 +148 @@
                 Way my = null;
                 for (Way w : ds.ways) {
-                        if (match(other, w)) {
+                        if (match(other, w) && ((mergeds == null) || (!mergeds.ways.contains(w)))) {
                                 my = w;
                                 break;
@@ -291 +294 @@
                 else
                         my.keys.putAll(other.keys);
+                
                 my.modified = true;
         }