RFR: 8014854 (was Re: RFR: 8012665: CharSequence.chars, CharSequence.codePoints)
Mike Duigou
mike.duigou at oracle.com
Fri May 31 00:28:25 UTC 2013
The 8014854 changes look good to me after Paul and Henry's suggested improvements.
Mike
On May 24 2013, at 06:39 , Alan Bateman wrote:
> On 29/04/2013 17:51, Henry Jen wrote:
>> On 04/26/2013 06:21 AM, Alan Bateman wrote:
>>> :
>>> 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.
>>>
>> I agree, let's follow up on this after first version.
>>
> Just to follow-up on this thread from a few weeks ago.
>
> I did a few tests with CharBuffer.chars() and the default implementation performs very poorly as expected. This is easily fixed by adding a spliterator with direct access to the buffer contents and that is straight-forward to do:
>
> http://cr.openjdk.java.net/~alanb/8014854/webrev/
>
> I've changed the templates used to generated the buffer classes so that ints/longs/doubles() could be added to Int/Long/DoubleBuffer if needed (which might be interesting when the buffers are backed by memory outside of the heap).
>
> The yak shaving in CompileJavaClasses.gmk is because the changes to the generated CharBuffer exposes a bug in the build of JObjC.jar on Mac.
>
> -Alan.
More information about the core-libs-dev
mailing list