Hi Christine,

src/share/classes/java/awt/event/InputEvent.java
-     * <PRE>
+     * <PRE>{@code

The {@code} is redundant here because the <pre> tag forces the text to use a fixed-size font already. Same comment for src/share/classes/java/awt/font/LineBreakMeasurer.java.


src/share/classes/java/awt/event/MouseEvent.java
- * <code>Component</code>).  Due to platform-dependent Drag&Drop 
implementations,
+ * <code>Component</code>).  Due to platform-dependent Drag&amp;Drop 
implementations,

Since you change this line anyway, I also suggest to replace <code> with {@code} here. Same comment for src/share/classes/java/awt/font/NumericShaper.java.


src/share/classes/java/awt/font/OpenType.java
- * ( <a 
href=http://www.microsoft.com/typography/otspec/";>http://www.microsoft.com/typography/otspec/l</a>
 ).
+ * ( <a 
href="http://www.microsoft.com/typography/otspec/";>http://www.microsoft.com/typography/otspec/l</a>
 ).

There's an extra 'l' character right before the closing </a>, which needs to be removed.

--
best regards,
Anthony

On 10/08/2013 05:59 AM, [email protected] wrote:
Hi,

Please review a fix for

https://bugs.openjdk.java.net/browse/JDK-8026021
more fix of javadoc errors and warnings reported by doclint, see the
description

webrev: http://cr.openjdk.java.net/~cl/8026021/webrev.00/

Thanks
Christine


Reply via email to