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 > <[email protected]> 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 <[email protected]> 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 <[email protected]> >> 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 >>> <[email protected]> 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>
