Changeset 2197 in josm for trunk

Timestamp:

2009-09-27T10:51:57+02:00 (15 years ago)

Author:

stoecker

Message:

fix 3587 - patch by Dave Hansen - NPE

File:

• trunk/src/org/openstreetmap/josm/data/osm/QuadBuckets.java (modified) (3 diffs)

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

@@ -668 +668 @@
     {
         out("unsupported operation");
-        Object o = null;
-        o.hashCode();
         throw new UnsupportedOperationException();
     }
     public boolean retainAll(Collection nodes)
     {
-        unsupported();
-        return false;
+        for (Node n : this) {
+            if (nodes.contains(n))
+                continue;
+            if (!this.remove(n))
+                return false;
+        }
+        return true;
     }
     public boolean removeAll(Collection nodes)
     {
-        unsupported();
-        return false;
+        for (Object o : nodes) {
+            if (!(o instanceof Node))
+                return false;
+            Node n = (Node)o;
+            if (!this.remove(n))
+                return false;
+        }
+        return true;
     }
     public boolean addAll(Collection nodes)
     {
-        unsupported();
-        return false;
+        for (Object o : nodes) {
+            if (!(o instanceof Node))
+                return false;
+            Node n = (Node)o;
+            if (!this.add(n))
+                return false;
+        }
+        return true;
     }
     public boolean containsAll(Collection nodes)
     {
-        unsupported();
-        return false;
+        boolean ret = true;
+        for (Object o : nodes) {
+            if (!(o instanceof Node))
+                return false;
+            Node n = (Node)o;
+            if (!this.contains(n))
+                return false;
+        }
+        return true;
     }
     private void check_type(Object o)
@@ -712 +734 @@
     public boolean contains(Object o)
     {
-        check_type(o);
+        if (!(o instanceof Node))
+            return false;
         QBLevel bucket = root.find_exact((Node)o);
         if (bucket == null)
@@ -808 +831 @@
         public void remove()
         {
-            Node object = peek();
-            current_leaf.content.remove(object);
+            // two uses
+            // 1. Back up to the thing we just returned
+            // 2. move the index back since we removed
+            //    an element
+            index_in_leaf--;
+            current_leaf.content.remove(index_in_leaf);
         }
     }