Changeset 11392 in josm for trunk/src/org/openstreetmap

Timestamp:

2016-12-14T00:52:36+01:00 (7 years ago)

Author:

Don-vip

Message:

sonar - squid:S1206 - "equals(Object obj)" and "hashCode()" should be overridden in pairs (required equalsverifier 2.1.8)

File:

• trunk/src/org/openstreetmap/josm/data/osm/Storage.java (modified) (5 diffs)

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

@@ -3 +3 @@
 
 import java.util.AbstractSet;
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.ConcurrentModificationException;
@@ -239 +240 @@
     public synchronized int hashCode() {
         int h = 0;
-        for (T t : this) {
-            h += hash.getHashCode(t);
+        if (hash != null) {
+            for (T t : this) {
+                h += hash.getHashCode(t);
+            }
         }
         return h;
+    }
+
+    @Override
+    public boolean equals(Object obj) {
+        if (this == obj)
+            return true;
+        if (obj == null || getClass() != obj.getClass())
+            return false;
+        Storage<?> other = (Storage<?>) obj;
+        return Arrays.equals(data, other.data)
+                && hashCode() == obj.hashCode();
     }
 
@@ -321 +335 @@
         int hcode = rehash(ha.getHashCode(key));
         int bucket = hcode & mask;
-        while ((entry = data[bucket]) != null) {
+        while (bucket < data.length && (entry = data[bucket]) != null) {
             if (ha.equals(key, entry))
                 return bucket;
@@ -500 +514 @@
         @Override
         public boolean hasNext() {
+            if (data == null) return false;
             align();
             return slot < data.length;
@@ -533 +548 @@
         @Override
         public boolean hasNext() {
+            if (data == null) return false;
             align();
             return slot < data.length;