RFR: 8012665: CharSequence.chars, CharSequence.codePoints

Alan Bateman Alan.Bateman at oracle.com
Fri Apr 26 13:21:33 UTC 2013

On 25/04/2013 21:25, Henry Jen wrote:
> Hi,
> Please review two default methods add to CharSequence returns IntStream
> of char value or code point value.
> http://cr.openjdk.java.net/~henryjen/tl/8012665.0/webrev/
> The synchronization test is relieved so lambda and other synthetic
> method is not tested. If synchronization is needed for those two default
> methods, subclass should override the methods.
> With charAt and codePointAt properly synchronized, the default
> implementation is sufficient. However as noted, if the sequence is
> mutated while the stream is being read, the result is undefined.
> Cheers,
> Henry
I looked through the webrev and it looks okay as a first version. I 
agree with Martin's suggestion on the performance. I also wonder if 
CharBuffer will need to override at least chars(). This could of course 
be done as a follow-on piece of work.

In TestSynchronization then I assume the commented System.out should be 


More information about the core-libs-dev mailing list