RFR 8019484 Sync j.u.c.ConcurrentHashMap from 166 to tl

Paul Sandoz paul.sandoz at oracle.com
Fri Jul 5 08:42:52 UTC 2013


A few bugs where found in CHM and fixed. I updated the webrev.

The diff can be viewed here:

  http://hg.openjdk.java.net/lambda/lambda/jdk/rev/291a872e1763

Paul.

On Jul 1, 2013, at 2:44 PM, Paul Sandoz <Paul.Sandoz at oracle.com> wrote:

> Hi,
> 
> The following is the final j.u.c patch that syncs CMH classes from 166 j.u.c to tl.
> 
>  http://cr.openjdk.java.net/~psandoz/tl/JDK-8019484-chm/webrev/
> 
> (i am sure there will be more patches to follow as tweaks are made to various areas, but hopefully they will not need so bulky.)
> 
> ConcurrentMap.replaceAll in tl was more up to date than that in 166.
> 
> Quite tricky to review CHM since there is a fair bit of code shuffling due to:
> 
> 480      * This file is organized to make things a little easier to follow
> 481      * while reading than they might otherwise: First the main static
> 482      * declarations and utilities, then fields, then main public
> 483      * methods (with a few factorings of multiple public methods into
> 484      * internal ones), then sizing methods, trees, traversers, and
> 485      * bulk operations.
> 
> Paul.




More information about the core-libs-dev mailing list