RFR: jsr166 jdk10 integration wave 3

Paul Sandoz paul.sandoz at oracle.com
Thu Sep 21 17:29:21 UTC 2017


Looks good, i just need to look a little more closely at the ConcurrentSkipListMap changes.


> On 20 Sep 2017, at 20:49, David Holmes <david.holmes at oracle.com> wrote:
> 
> Hi Martin,
> 
> On 21/09/2017 1:20 PM, Martin Buchholz wrote:
>> http://cr.openjdk.java.net/~martin/webrevs/openjdk10/jsr166-integration/ <http://cr.openjdk.java.net/~martin/webrevs/openjdk10/jsr166-integration/>
>> Our usual webrev, rejiggered for the consolidated repo.
>> David, this includes your nestmates changes.
> 
> Thanks - much appreciated. The refactoring looks fine - though I wouldn't have immortalized the "nestmates" part in the naming. :)
> 

Perhaps use the prefix TopLevelAtomicTests or NonEnclosingAtomicTests ?

Paul.

> For others see:
> 
> https://bugs.openjdk.java.net/browse/JDK-8187607
> 
> Cheers,
> David



More information about the core-libs-dev mailing list