Here's the final set of changes including those 2 diffs.

http://cr.openjdk.java.net/~prr/8049893.1 <http://cr.openjdk.java.net/%7Eprr/8049893/>

-phil.

On 07/11/2014 04:08 AM, Otávio Gonçalves de Santana 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] <mailto:[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] <mailto:[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]
        <mailto:[email protected]>> wrote:

        https://bugs.openjdk.java.net/browse/JDK-8049893
        http://cr.openjdk.java.net/~prr/8049893/
        <http://cr.openjdk.java.net/%7Eprr/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 <tel:55%20%2811%29%2098255-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


Reply via email to