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