Request for Review : 6553074 : String{Buffer, Builder}.indexOf(Str, int) contains unnecessary allocation
Mike Duigou
mike.duigou at oracle.com
Mon Nov 19 22:09:26 UTC 2012
On Nov 14 2012, at 13:24 , Jim Gish wrote:
> Mike,
>
> In String.java, with the new methods you're adding, should we make those String target parameters a CharSequence instead?
A String param enables us to extract the internal char array for the search. We could not do so with a CharSequence and would have to create a substring somewhat like the old implementation did.
Mike
> Thanks,
> Jim
>
> On 11/14/2012 01:27 PM, Mike Duigou wrote:
>> Hello all;
>>
>> This patch causes the indexOf and lastIndexOf implementation in AbstractStringBuilder to directly compare the character arrays rather than making a copy of the substring before comparing.
>>
>> http://cr.openjdk.java.net/~mduigou/6553074/0/webrev/
>>
>>
More information about the core-libs-dev
mailing list