andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fix containsValue to take snapshot of table and use Arrays.fill for clearing out oldTable

Browse Source
This commit is contained in:
Arya Asemanfar 2012-12-06 10:36:13 -08:00
parent 9dd0756d7a
commit 6f6e1693cf
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/main/java/org/jboss/netty/util/internal/ConcurrentIdentityHashMap.java
View File

@ -23,6 +23,7 @@ package org.jboss.netty.util.internal;
import java.util.AbstractCollection;
import java.util.AbstractMap;
import java.util.AbstractSet;
import java.util.Arrays;
import java.util.Collection;
import java.util.ConcurrentModificationException;
import java.util.Enumeration;
@ -355,7 +356,8 @@ public final class ConcurrentIdentityHashMap<K, V> extends AbstractMap<K, V>
boolean containsValue(Object value) {
if (count != 0) { // read-volatile
for (HashEntry<K, V> e: table) {
HashEntry<K, V>[] tab = table;
for (HashEntry<K, V> e: tab) {
for (; e != null; e = e.next) {
V opaque = e.value();
V v;
@ -515,9 +517,7 @@ public final class ConcurrentIdentityHashMap<K, V> extends AbstractMap<K, V>
}
}
table = newTable;
for (int i = 0; i < oldCapacity; ++i) {
oldTable[i] = null;
}
Arrays.fill(oldTable, null);
return reduce;
}