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.
> 

Reply via email to