RFR 8010293 <was> Re: Potential issue with CHM.toArray

Paul Sandoz paul.sandoz at oracle.com
Sun Sep 15 14:21:43 UTC 2013


On Sep 10, 2013, at 4:34 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:

> On 09/09/2013 14:35, Paul Sandoz wrote:
>> On Sep 6, 2013, at 4:56 PM, Alan Bateman<Alan.Bateman at oracle.com>  wrote:
>> 
>>> :
>>> The comments are very educational as the resizing is difficult to completely grok without going through examples on a whiteboard. Anyway, I don't see anything obviously wrong after going through it. The test case is useful although creating the list of threads is quite a mouth full to take in.
>>> 
>> Yeah, i left that in a convoluted intermediate state and wanted to use CountedCompleter instead, see below for a revised and preferred version.
>> 
>> Paul.
> Thanks, the "preferred version" looks good to me. In passing I wonder if it should be renamed to ToArray.java while you're there.
> 

Thanks.

Renamed and pushed.

However, although the rename (made from a refactor in the IDE) was reflected in my local workspace (with no diffs) it did not appear to be reflected remotely:

  http://hg.openjdk.java.net/jdk8/tl/jdk/diff/ff6c76f7733e/test/java/util/concurrent/ConcurrentHashMap/toArray.java

:-( something very strange is going on.

I logged another bug and pushed a fix (with you as reviewer) to resolve this:

  http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5025ed287a4a

Apologies for the noise.

Paul.


More information about the core-libs-dev mailing list