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
