Character, Byte, Short valueOf instead of new instance (original) (raw)

Remi Forax forax at univ-mlv.fr
Fri Jul 11 16:30:32 UTC 2014

On 07/02/2014 01:07 AM, Otávio Gonçalves de Santana wrote:

Thank you Remi. The long[1] also was included? [1]https://bugs.openjdk.java.net/browse/JDK-8048267

yes, I've included all the patches (I hope) http://websvn.ow2.org/revision.php?repname=asm&path=%2Ftrunk%2F&rev=1759

There were just one place were we have though that it was better to use new Integer() instead of Integer.valueOf for debugging (the identity is guaranteed to be different from an Integer that comes from user code).

Thanks again for the patches :)

regards, Rémi

On Tue, Jul 1, 2014 at 7:28 PM, Remi Forax <forax at univ-mlv.fr_ _<mailto:forax at univ-mlv.fr>> wrote: On 07/01/2014 11:01 PM, Andrej Golovnin wrote: Hi Pavel, Thanks a lot! I've separated out all the changes from the 'jdk.internal.org.objectweb.asm' package. So I'll update both threads with new webrevs. http://cr.openjdk.java.net/~prappo/8048874/webrev.02 <http://cr.openjdk.java.net/%7Eprappo/8048874/webrev.02> Looks good to me. @Otávio: Could you please submit the ASM related changes to the ASM project [1]? Thanks! Best regards, Andrej Golovnin [1] http://asm.ow2.org/ Paul Sandoz had already sent the changes related to ASM. I'm currently reviewing them. Rémi

-- Atenciosamente. Otávio Gonçalves de Santana blog: http://otaviosantana.blogspot.com.br/ twitter: http://twitter.com/otaviojava site: http://www.otaviojava.com.br <http://www.otaviojava.com.br/> (11) 98255-3513

More information about the core-libs-dev mailing list