Ticket #3587: josm-null-pointer-quad-0.patch

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

@@ -667 +667 @@
     public void unsupported()
     {
         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)
     {
@@ -711 +733 @@
     }
     public boolean contains(Object o)
     {
-        check_type(o);
+        if (!(o instanceof Node))
+            return false;
         QBLevel bucket = root.find_exact((Node)o);
         if (bucket == null)
             return false;
@@ -808 +831 @@
         public void remove()
         {
             Node object = peek();
+            if (object == null)
+                return;
             current_leaf.content.remove(object);
         }
     }