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.
