On 07/01/13 08:44, Paul Sandoz wrote:
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/
Thanks for doing this!
ConcurrentMap.replaceAll in tl was more up to date than that in 166.
Yes, we knew this would happen a few weeks ago when Mike committed, but I was still tracking lambda versions. Hopefully these mismatches will soon be things of the past.
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.
Yes, sorry for the impact on diffs, but the old file organization (based on the original version of CHM) was getting too chaotic, so I rearranged in a way that better reflects structure. -Doug