8058779: Faster implementation of String.replace(CharSequence, CharSequence)

[Ivan Gerasimov]

Thanks!

On 02.06.2015 11:54, Remi Forax wrote:
> Looks Ok to me,
> thumb up :)
>
> Rémi
>
> On 06/01/2015 08:53 PM, Ivan Gerasimov wrote:
>>
>>
>> On 01.06.2015 11:33, Paul Sandoz wrote:
>>> On May 31, 2015, at 6:03 PM, Ivan Gerasimov 
>>> <ivan.gerasimov at oracle.com> wrote:
>>>> Which is right here:
>>>> http://cr.openjdk.java.net/~igerasim/8058779/05/webrev/
>>>>
>>> Much better.
>>>
>>> For the test can you use RandomFactory recently added to the test 
>>> library?
>>
>> Sure.
>> Here the updated webrev with this change and a few other minor changes.
>> http://cr.openjdk.java.net/~igerasim/8058779/06/webrev/
>>
>> The changes are:
>> - move declaration of i below,
>> - indent .append(),
>> - use RandomFactory in the test,
>> - extend number of test cases with null input.
>>
>> Do you think it's ready to go?
>>
>> Sincerely yours,
>> Ivan
>>
>>> Paul.
>>>
>>>
>>>
>>
>
>
>