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.
Thanks,
s'marks
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