8016251: Balanced spliterator for SpinedBuffer

Mike Duigou mike.duigou at oracle.com
Wed Jun 12 21:37:53 UTC 2013


Looks good.

Mike

On Jun 10 2013, at 07:40 , Paul Sandoz wrote:

> Hi,
> 
> This patch contains the following changes:
> 
> 1) implements balanced splits for the spliterator of SpinedBuffer, which is contributed by Peter Levart; and

For context, http://mail.openjdk.java.net/pipermail/lambda-dev/2013-April/009306.html is where Peter proposed this.

> 
> 2) syncs lambda's version SpinedBuffer with tl, hence why there are some other changes they may not be entirely obvious until other code follows later on but will make that process easier e.g. the renaming of asXxxArray to asPrimitiveArray or the method SpinedBuffer.OfPrimitive.newArray changing from protected to public.

Seems OK.

> 
>  http://cr.openjdk.java.net/~psandoz/tl/JDK-8016251-spinedBuffer-splitr/webrev/
> 
> Paul.
> 




More information about the core-libs-dev mailing list