5071718: (bf) Add ByteBuffer.slice(int offset, int length)
Brent Christian
brent.christian at oracle.com
Tue Feb 26 00:30:18 UTC 2019
A nit for the new javadoc:
I find it a little weird for one paragraph to start, "The new buffer
will start at position {@code index}...", and the next paragraph to
start, "The new buffer's position will be zero."
I find the no-arg slice()'s reference to "the content of the new buffer"
to be a bit clearer. So my suggestion would be, e.g.:
"The content of the new buffer will start at position {@code index} in
this buffer, and will contain {@code length} elements."
Thanks,
-Brent
On 2/25/19 1:41 PM, Brian Burkhalter wrote:
> Updated [1] to allow for the case ‘slice(limit(),0)' with no IOOBE and to add ‘slice(int,int)’ to the “additional operations” section of the Buffer class-level specification.
>
> The CSR [2] description has also been updated but not the specdiff attachment: will hold off on that until the verbiage is agreed upon.
>
> Thanks,
>
> Brian
>
> [1] http://cr.openjdk.java.net/~bpb/5071718/webrev.01/
> [2] https://bugs.openjdk.java.net/browse/JDK-8219608
>
More information about the nio-dev
mailing list