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