RFR 8010293 <was> Re: Potential issue with CHM.toArray
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.
> Thanks, the "preferred version" looks good to me. In passing I wonder if it should be renamed to ToArray.java while you're there.
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:
:-( something very strange is going on.
I logged another bug and pushed a fix (with you as reviewer) to resolve this:
Apologies for the noise.
More information about the core-libs-dev