Lower overhead String encoding/decoding

Alan Bateman Alan.Bateman at oracle.com
Tue Oct 28 14:52:21 UTC 2014


On 26/10/2014 21:10, Richard Warburton wrote:
> :
>
> Thanks for taking the time to look at this - most appreciated. I've 
> pushed the latest iteration to 
> http://cr.openjdk.java.net/~rwarburton/string-patch-webrev-8/ 
> <http://cr.openjdk.java.net/%7Erwarburton/string-patch-webrev-8/>.
>
I think this is looking good.

For the constructor then the words "decoding the specified byte buffer", 
it might be a bit clearer as "decoding the remaining bytes in the ...".

For getBytes(ByteBuffer, Charset) then the position is advanced by the 
bytes written, no need to mention the number of chars read here.

In the constructor then you make it clear that malformed/unmappable 
sequences use the default replacement. This is important to state in the 
getBytes methods too because the encoding can fail.

-Alan.





More information about the core-libs-dev mailing list