Character, Byte, Short valueOf instead of new instance

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