RFR ConcurrentHashMap typo

Chris Hegarty chris.hegarty at oracle.com
Wed Aug 14 22:08:01 UTC 2013


Typo that crept in during a refactoring.

http://cs.oswego.edu/pipermail/concurrency-interest/2013-August/011695.html

This change is already in the jsr166 CVS.

diff --git 
a/src/share/classes/java/util/concurrent/ConcurrentHashMap.java 
b/src/share/classes/java/util/concurrent/ConcurrentHashMap.java
--- a/src/share/classes/java/util/concurrent/ConcurrentHashMap.java
+++ b/src/share/classes/java/util/concurrent/ConcurrentHashMap.java
@@ -2785,7 +2785,7 @@ public class ConcurrentHashMap<K,V> exte
                          return;
                      }
                  }
-                else if ((s | WAITER) == 0) {
+                else if ((s & WAITER) == 0) {
                      if (U.compareAndSwapInt(this, LOCKSTATE, s, s | 
WAITER)) {
                          waiting = true;
                          waiter = Thread.currentThread();

-Chris.



More information about the core-libs-dev mailing list