Add getChars to CharSequence

Alan Bateman Alan.Bateman at oracle.com
Tue Apr 23 11:17:46 UTC 2013


On 22/04/2013 20:45, Martin Buchholz wrote:
> Another preliminary webrev is out at
> http://cr.openjdk.java.net/~martin/webrevs/openjdk8/getChars/ 
> <http://cr.openjdk.java.net/%7Emartin/webrevs/openjdk8/getChars/>
>
> Alan et al: Before continuing, can we:
>
> Have thumbs up on the changes to out of bounds exceptions?
I looked over the bound checking. The only one that isn't clear is 
AbstractStringBuilder.insert where dstOffset is specified to be greater 
than this.length rather than count.


>
> The handling of the rw conditional in the preprocessed sources is very 
> confusing and error-prone.  I cleaned up some of that code and added 
> tests to catch the most obvious mistakes.  We can do this all in a 
> single changeset, but if you like, we can separate the general nio 
> buffer improvements into a separate changeset; if so, file a bug.
>
The templates are complicated to work with. Can you provide more details 
on the issues that you found? I grabbed the changes to the buffer tests 
in your patch and ran them with a build on jdk8/tl and the new tests you 
added pass there. Maybe it is the comedy of errors that some of these 
cases actually work but it would be good to get a summary.

-Alan.



More information about the core-libs-dev mailing list