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