Changeset 224 in josm for src/org/openstreetmap/josm

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.

Location:

src/org/openstreetmap/josm

Files:

• actions/DownloadIncompleteAction.java (modified) (1 diff)
• data/osm/visitor/MergeVisitor.java (modified) (6 diffs)
• gui/layer/OsmDataLayer.java (modified) (1 diff)
• io/IncompleteDownloader.java (modified) (1 diff)

src/org/openstreetmap/josm/actions/DownloadIncompleteAction.java

@@ -48 +48 @@
 
                 @Override protected void finish() {
-                        MergeVisitor merger = new MergeVisitor(Main.ds);
+                        MergeVisitor merger = new MergeVisitor(Main.ds, reader.data);
                         for (OsmPrimitive osm : reader.data.allPrimitives())
                                 osm.visit(merger);

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;
         }

src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

@@ -157 +157 @@
 
         @Override public void mergeFrom(final Layer from) {
-                final MergeVisitor visitor = new MergeVisitor(data);
+                final MergeVisitor visitor = new MergeVisitor(data,((OsmDataLayer)from).data);
                 for (final OsmPrimitive osm : ((OsmDataLayer)from).data.allPrimitives())
                         osm.visit(visitor);

src/org/openstreetmap/josm/io/IncompleteDownloader.java

@@ -44 +44 @@
          */
         private final Collection<Way> toDownload;
-        private MergeVisitor merger = new MergeVisitor(data);
+        private MergeVisitor merger = new MergeVisitor(data, null);
 
         public IncompleteDownloader(Collection<Way> toDownload) {