Looks correct. mike
On Aug 14 2013, at 15:08 , Chris Hegarty wrote: > 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.
