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

sunita koppar sunita.koppar at oracle.com
Fri Jul 13 05:35:02 PDT 2012


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