RFR ConcurrentHashMap typo
Mike Duigou
mike.duigou at oracle.com
Wed Aug 14 23:11:09 UTC 2013
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.
More information about the core-libs-dev
mailing list