Hi Peter,

On Dec 22, 2017, at 1:21 AM, Peter Levart <peter.lev...@gmail.com> wrote:

> I think this looks good. No more suggestions from my side for this patch. As 
> Paul notes, it would be interesting to see if using 
> Unsafe.allocateUninitializedArray() to allocate the final array (and 
> intermediate buffers too) has a measurable impact.

Thanks for the final look. I am going to run regression tests once again today 
and if there is no problem I’ll check this in. I’ll also file a follow-up issue 
about the uninitialized array case.

Brian

Reply via email to