<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&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