5071718: (bf) Add ByteBuffer.slice(int offset, int length)

Brian Burkhalter brian.burkhalter at oracle.com
Tue Feb 26 15:55:32 UTC 2019


> On Feb 26, 2019, at 12:43 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> 
> On 26/02/2019 01:48, Brian Burkhalter wrote:
>> Hi Brent,
>> 
>> That does read better.
> It's consistent with the existing slice method so I think it's good too.

Webrev [1] and CSR [2] updated accordingly.

> Otherwise looks good to me, kudos to Roger for spotting the error with the 0 length case, that would have been embarrassing to get wrong.

Indeed: good catch, Roger!

Thanks,

Brian

[1] http://cr.openjdk.java.net/~bpb/5071718/webrev.02/
[2] https://bugs.openjdk.java.net/browse/JDK-8219608
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/nio-dev/attachments/20190226/66580251/attachment.html>


More information about the nio-dev mailing list