Looks good to me. But the testcase has no bearing to this bug and should be removed and "noreg-doc" label needs to be added to bugreport.
Also, a ccc needs to be raised for this.

Regards
Prasanta

*From:*Muneer Kolarkunnu
*Sent:* Thursday, May 12, 2016 5:19 PM
*To:* [email protected]
*Subject:* <AWT Dev> Review request for 8153184: BorderLayout javadoc says current version of JDK is 1.2

Hi All,

                Please review the fix for bug 8153184,

Webrev: http://cr.openjdk.java.net/~jcm/8153184/webrev.00/ <http://cr.openjdk.java.net/%7Ejcm/8153184/webrev.00/>

                Bug: https://bugs.openjdk.java.net/browse/JDK-8153184

Issue:

It is a documentation bug, it is mentioning version information in the javadoc, ie;”[Currently (in the Java 2 platform v1.2)]” . So it is difficult to maintain this kind of version information in the javadoc, just removing that version information.

Links to faulty javadoc:http://download.java.net/java/jdk9/docs/api/java/awt/BorderLayout.html

https://docs.oracle.com/javase/8/docs/api/java/awt/BorderLayout.html

Fix:

Removed the version information from javadoc as it is difficult to maintain in different versions

Regards,

Muneer


Reply via email to