RFR: JDK-8222930: ConcurrentSkipListMapTest.clone() broken since jdk10

Stuart Marks stuart.marks at oracle.com
Wed Apr 24 16:59:17 UTC 2019

Hi Adam,

Thanks for finding this bug!

This is a bug in ConcurrentSkipListMap itself, not some test named 
ConcurrentSkipListMapTest. I'd suggest changing the bug summary line and the 
commit message accordingly.



On 4/24/19 9:20 AM, Adam Farley8 wrote:
> ConcurrentSkipListMapTest.clone() produces a clone that shares the array
> size variable of the original, and then doubles it.
> So both arrays, original and clone, tell the user that each is twice as
> big as it actually is.
> The proposed fix is to simply set the clone's array size variable to null
> during creation.
> Fix and test code available.
> Reviews and sponsor requested.
> Webrev: http://cr.openjdk.java.net/~afarley/8222930.0/jdk13/webrev/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8222930
> Best Regards
> Adam Farley
> IBM Runtimes
> P.S. Apparently this has been broken since JDK 10, so we should look at
> backporting (at least to 11 and 12) once this is in.
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU

More information about the core-libs-dev mailing list