Index: /trunk/src/org/openstreetmap/josm/data/osm/QuadBuckets.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/data/osm/QuadBuckets.java	(revision 2419)
+++ /trunk/src/org/openstreetmap/josm/data/osm/QuadBuckets.java	(revision 2420)
@@ -2,4 +2,5 @@
 import java.lang.reflect.Array;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
@@ -20,7 +21,5 @@
     static void abort(String s)
     {
-        out(s);
-        Object o = null;
-        o.hashCode();
+        throw new AssertionError(s);
     }
     static void out(String s)
@@ -82,9 +81,6 @@
                     (xmax >  180.0) ||
                     (ymin <  -90.0) ||
-                    (ymax >   90.0)) {
-                out("bad BBox: " + this);
-                Object o = null;
-                o.hashCode();
-            }
+                    (ymax >   90.0))
+                throw new IllegalArgumentException("bad BBox: " + this);
         }
         @Override
@@ -253,5 +249,4 @@
             return ret;
         }
-        @SuppressWarnings("unchecked")
         QBLevel[] newChildren()
         {
@@ -272,5 +267,4 @@
             }
             if (o instanceof Node) {
-                Node n = (Node)o;
                 LatLon coor = ((Node)o).getCoor();
                 if (coor == null)
@@ -365,5 +359,4 @@
         void add_to_leaf(T o)
         {
-            QBLevel ret = this;
             add_content(o);
             if (content.size() > MAX_OBJECTS_PER_LEVEL) {
@@ -545,5 +538,5 @@
             if (content == null) {
                 if (debug) {
-                    out("["+level+"] leaf size: null content, children? " + children);
+                    out("["+level+"] leaf size: null content, children? " + Arrays.toString(children));
                 }
                 return 0;
@@ -580,5 +573,4 @@
         QBLevel find_exact(T n)
         {
-            QBLevel ret = null;
             if (content != null && content.contains(n))
                 return this;
@@ -855,5 +847,5 @@
         throw new UnsupportedOperationException();
     }
-    public boolean retainAll(Collection objects)
+    public boolean retainAll(Collection<?> objects)
     {
         for (T o : this) {
@@ -874,5 +866,5 @@
         return false;
     }
-    public boolean removeAll(Collection objects)
+    public boolean removeAll(Collection<?> objects)
     {
         for (Object o : objects) {
@@ -884,5 +876,5 @@
         return true;
     }
-    public boolean addAll(Collection objects)
+    public boolean addAll(Collection<? extends T> objects)
     {
         for (Object o : objects) {
@@ -894,7 +886,6 @@
         return true;
     }
-    public boolean containsAll(Collection objects)
-    {
-        boolean ret = true;
+    public boolean containsAll(Collection<?> objects)
+    {
         for (Object o : objects) {
             if (!canStore(o))
@@ -987,5 +978,5 @@
         return this.toArrayList().toArray();
     }
-    public <T> T[] toArray(T[] template)
+    public <A> A[] toArray(A[] template)
     {
         return this.toArrayList().toArray(template);
@@ -1082,5 +1073,5 @@
             //    an element
             content_index--;
-            T object = peek();
+            peek(); //TODO Is the call to peek() necessary?
             current_node.content.remove(content_index);
         }
@@ -1196,6 +1187,4 @@
             tmp = tmp.parent;
         }
-        if (ret == null || ret.size() == 0)
-            return Collections.emptyList();
         if (debug) {
             out("search of QuadBuckets for " + search_bbox + " ret len: " + ret.size());