On Sep 9, 2013, at 3:35 PM, Paul Sandoz <paul.san...@oracle.com> wrote:
> > On Sep 6, 2013, at 4:56 PM, Alan Bateman <alan.bate...@oracle.com> wrote: > >> On 06/09/2013 12:08, Paul Sandoz wrote: >>> On Sep 2, 2013, at 3:24 PM, Paul Sandoz<paul.san...@oracle.com> wrote: >>> >>> : >>> >>>> Here is the fix in the lambda repo which can be applied to tl: >>>> >>>> http://hg.openjdk.java.net/lambda/lambda/jdk/rev/b73937e96ae0 >>>> http://hg.openjdk.java.net/lambda/lambda/jdk/raw-rev/b73937e96ae0 >>>> >>> Anyone up for reviewing this? >>> >> 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. > s/CountedCompleter/CompletableFuture Sigh F/J on the brain... Paul.
