[7u6] Request for phase 2 approval for CR 7182135 (original) (raw)

[7u6] Request for phase 2 approval for CR 7182135 - Impossible to use some editors directly

Edvard Wendelin edvard.wendelin at oracle.com
Fri Jul 13 07:30:38 PDT 2012

Pre-approved pending release team approval and positive reviews.

Thanks, Edvard

On Jul 13, 2012, at 2:35 PM, sunita koppar wrote:

Hi All,

Please review the below Code change: Description of the bug: Theclasses sun.beans.editors.BoolEditor and sun.beans.editors.IntEditor are being used by some customers directly. These classes have been changed in JRE7 to be sun.beans.editors.BooleanEditor and sun.beans.editors.IntegerEditor. The current change is to make this change backward compatible. Webrev: http://cr.openjdk.java.net/~mbankal/7182135/webrev.00/ Please note, this changeset is not and will not be pushed to JDK8 for below reasons: 1. The change is only to allow such private API usage up-till 7uX. 2. The webrev for 7u6 is different and incorporates only a subset of changes which have to be pushed immediately. 3. Even if we decide to port this to JDK8, the webrev for 7u8 (a greater set of changes) will go into JDK8 and not the one for 7u6 (current one). Tests done: http://cr.openjdk.java.net/~mbankal/7182135/Tests/ 1. JPRT built and tested successfully for all platforms 2. Unit testing done after modifying an existing testcase. This tries to load the BoolEditor and IntEditor classes. For details refer - http://cr.openjdk.java.net/~mbankal/7182135/Tests/UnitTest.txt 3. Regression testing with jtreg. 4. JCK tests. Risks: The change carries minimal risk as we are just extending existing classes to support additional editor names.

regards Sunita Koppar Java Client Sustaining Engineer P.S: My mercurial id is not working well. Hence requested Mala to publish the webrev for me. Thanks Mala.

More information about the jdk7u-dev mailing list