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.hol...@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
