<AWT Dev> Review request for JDK-8026021: more fix of javadoc errors and warnings reported by doclint

Anthony Petrov anthony.petrov at oracle.com
Tue Oct 8 01:51:46 PDT 2013


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, christine.lu at oracle.com 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
>
>


More information about the awt-dev mailing list