jmx-dev RFR Doclint cleanup of javax.management
Jaroslav Bachorik
jaroslav.bachorik at oracle.com
Tue Nov 12 01:48:17 PST 2013
Hi Roger,
The following comments are based on the presumption that the contained
HTML should be well-formed.
src/share/classes/javax/management/MBeanFeatureInfo.java
L183, L231 - missing </p>
src/share/classes/javax/management/MBeanInfo.java
L640, L686 - missing </p>
src/share/classes/javax/management/MBeanServerConnection.java
L630 - missing </p>
src/share/classes/javax/management/MBeanServerNotification.java
L89 - missing </p>
src/share/classes/javax/management/MXBean.java
L788 - missing </p>
src/share/classes/javax/management/ObjectName.java
L198 - missing </p>
L204 - missing </p>
L1552 - missing </p>
src/share/classes/javax/management/PersistentMBean.java
L78 - missing </p><p>
L79 - there should be <pre> after this line
L83 - should be * }</PRE></p>
src/share/classes/javax/management/loading/MLet.java
L154 - extraneous </dd>
src/share/classes/javax/management/modelmbean/ModelMBeanNotificationBroadcaster.java
L130 - missing </p>
src/share/classes/javax/management/openmbean/CompositeDataInvocationHandler.java
L83 - missing </p>
src/share/classes/javax/management/openmbean/OpenMBeanParameterInfoSupport.java
L123 - missing </p>
src/share/classes/javax/management/remote/JMXConnector.java
L79 - missing </p>
src/share/classes/javax/management/remote/rmi/RMIConnector.java
L1697 - missing </li> after this line
L1701 - missing </li> after this line
-JB-
On 11.11.2013 23:20, roger riggs wrote:
> Please review and comment.
>
> This changeset addresses the warnings and errors of doclint and
> improves the quality of the generated html.
> No spec or implementation is changed by this changeset
>
> The changes address issues including html syntax, accessibility issues,
> and html tag improper use and list structure.
> Refer to the error log attached to the bug report [1] for details.
>
> Please review:
> http://cr.openjdk.java.net/~rriggs/webrev-lint-mgmt-8028014/
>
> Thanks, Roger
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8028014
>
More information about the jmx-dev
mailing list