[concurrency-interest] ConcurrentHashMapV8 typo?

Doug Lea dl at cs.oswego.edu
Fri Aug 9 14:40:58 EDT 2013


On 08/09/13 13:17, Oleksiy Stashok wrote:
> Hi,
>
> guys can you pls. doublecheck the method ConcurrentHashMapV8.contendedLock() : 2689
> The comparison
>
> else if ((s | WAITER) == 0)

Wow. Thanks/sorry. This crept in during a refactoring.
(And from there propagated elsewhere, including JDK8 CHM.)

-Doug


>
>
> looks suspicious, shouldn't it be
>
> else if ((s & WAITER) == 0)
>
> ?
>
> Thanks.
>
> WBR,
> Alexey.
>
>
> _______________________________________________
> Concurrency-interest mailing list
> Concurrency-interest at cs.oswego.edu
> http://cs.oswego.edu/mailman/listinfo/concurrency-interest
>



More information about the Concurrency-interest mailing list