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
