RFR (S) 8136500: Integer/Long getChars and stringSize should be more idiomatic
Ivan Gerasimov
ivan.gerasimov at oracle.com
Mon Nov 23 16:23:48 UTC 2015
Great! Now I like it the best.
Though, it may be better to get yet another pair of eyes.
One minor nit: In the tests, in the summary, it is written, "Test
Integer.toString method*s*", but only one of the overloads is tested.
Sincerely yours,
Ivan
On 23.11.2015 18:08, Aleksey Shipilev wrote:
> On 11/23/2015 04:34 PM, Ivan Gerasimov wrote:
>> With this fixed patch:
>> http://cr.openjdk.java.net/~igerasim/8136500/8136500-addition-1.patch
>> all tests from test/lang pass.
>>
>> Would you give it another chance?
> Okay, but this is better be the last non-cosmetic change to board this
> departing train:
> http://cr.openjdk.java.net/~shade/8136500/webrev.05/
>
> This passes java/lang, java/util tests, and performance improvement is
> still very good.
>
> Thanks,
> -Aleksey
>
More information about the core-libs-dev
mailing list