RFR: 8049892: Replace uses of 'new Integer()' with appropriate alternative across core classes (original) (raw)

Andrej Golovnin andrej.golovnin at gmail.com
Fri Jul 11 06:41:14 UTC 2014

Hi Otávio,

changes to the classes in the package "jdk.internal.org.objectweb.asm" should be done directly in the ASM library. Please exclude following classes from the patch:

src/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java src/share/classes/jdk/internal/org/objectweb/asm/Opcodes.java src/share/classes/jdk/internal/org/objectweb/asm/commons/GeneratorAdapter.java src/share/classes/jdk/internal/org/objectweb/asm/commons/InstructionAdapter.java src/share/classes/jdk/internal/org/objectweb/asm/tree/LookupSwitchInsnNode.java src/share/classes/jdk/internal/org/objectweb/asm/util/CheckMethodAdapter.java src/share/classes/jdk/internal/org/objectweb/asm/util/Textifier.java

And please exclude following classes from the patch too:

src/share/classes/sun/reflect/UnsafeIntegerFieldAccessorImpl.java src/share/classes/sun/reflect/UnsafeQualifiedIntegerFieldAccessorImpl.java src/share/classes/sun/reflect/UnsafeQualifiedStaticIntegerFieldAccessorImpl.java src/share/classes/sun/reflect/UnsafeStaticIntegerFieldAccessorImpl.java

This classes will be fixed by the patch for the issue 5043030. Otherwise it looks good.

Best regards, Andrej Golovnin

On Fri, Jul 11, 2014 at 3:36 AM, Otávio Gonçalves de Santana < otaviopolianasantana at gmail.com> wrote:

https://bugs.openjdk.java.net/browse/JDK-8049892 http://cr.openjdk.java.net/~prr/8049892/

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

More information about the core-libs-dev mailing list