Thanks guys, pushed. -Aleksey
On 10.02.2016 16:19, Vladimir Ivanov wrote: > Looks good. > > Best regards, > Vladimir Ivanov > > On 2/10/16 3:51 PM, Aleksey Shipilev wrote: >> See new webrev that adds a few more tests: >> http://cr.openjdk.java.net/~shade/8149459/webrev.02/ >> >> On 10.02.2016 14:25, Remi Forax wrote: >>> Hi Aleksey, if you have a dumper, you should append a value at the >>> end of the generated class otherwise if you have more than one string >>> concatenation, you will see only one class on the disk. >> >> No, we are actually okay, because our class names are unique since >> Unsafe.defineAnonymousClass handles this: >> >> $ ls -lR dump/java.lang.String\$Concat/ >> dump/java.lang.String$Concat/2008017533.class >> dump/java.lang.String$Concat/665576141.class >> >> Cheers, >> -Aleksey >>
