<Swing Dev> RFR : 8049893 Replace uses of 'new Integer()' with appropriate alternative across client classes

Petr Pchelko petr.pchelko at oracle.com
Fri Jul 11 11:20:07 UTC 2014


Thank you, Otávio.

The fix looks good to me. 

With best regards. Petr.

> On Jul 11, 2014, at 3:08 PM, Otávio Gonçalves de Santana <otaviopolianasantana at gmail.com> wrote:
> 
> 
> JInternalFrame
> Because it has a method, line 1285, with int, setLayer(int), so it will capture just to do boxing, so I forced the boxing to avoid this method and go directly to setLayer(Integer), line 1254.
> 
> JlayeredPane
> Done.
> 
> BasicLookAndFeel
> Integer four //fixed
> How was talked 500 is not in cached range.
> 
> 
> On Fri, Jul 11, 2014 at 6:03 AM, Petr Pchelko <petr.pchelko at oracle.com> wrote:
>> 500 and 1000L are outside of the range [-127, 128]. Therefore they won't be cached.
> Agreed, that’s what I didn’t think about.
> 
> With best regards. Petr.
> 
>> On Jul 11, 2014, at 1:00 PM, Andrej Golovnin <andrej.golovnin at gmail.com> wrote:
>> 
>> Hi Petr,
>> 
>> BasicLookAndFeel - making “shared Integers” and “shared longs” is pointless after your fix. Please inline.
>> 
>> It's not pointless.
>> 
>>  458         Integer fiveHundred = 500;
>>  459 
>>  460         // *** Shared Longs
>>  461         Long oneThousand = 1000L;
>> 
>> 500 and 1000L are outside of the range [-127, 128]. Therefore they won't be cached.
>> 
>> But this one:
>> 
>> 676         Integer four = 4;
>> 
>> can be indeed inlined.
>> 
>> Best regards,
>> Andrej Golovnin
>> 
>>  
>> 
>> With best regards. Petr.
>> 
>>> On Jul 11, 2014, at 5:36 AM, Otávio Gonçalves de Santana <otaviopolianasantana at gmail.com> wrote:
>>> 
>>> https://bugs.openjdk.java.net/browse/JDK-8049893
>>> http://cr.openjdk.java.net/~prr/8049893/
>>> 
>>> -- 
>>> Cheers!.
>>> 
>>> Otávio Gonçalves de Santana
>>> 
>>> blog:     http://otaviosantana.blogspot.com.br/
>>> twitter: http://twitter.com/otaviojava
>>> site:     http://about.me/otaviojava
>>> 55 (11) 98255-3513
>>> 
>> 
>> 
> 
> 
> 
> 
> -- 
> Cheers!.
> 
> Otávio Gonçalves de Santana
> 
> blog:     http://otaviosantana.blogspot.com.br/
> twitter: http://twitter.com/otaviojava
> site:     http://about.me/otaviojava
> 55 (11) 98255-3513
> 
> <JLayeredPane.diff><BasicLookAndFeel.diff>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20140711/9ad3c084/attachment.html>


More information about the swing-dev mailing list