<AWT Dev> [8] Review request for CR JDK-8025218 [javadoc] some errors in java/awt classes
Anthony Petrov
anthony.petrov at oracle.com
Fri Sep 27 03:22:23 PDT 2013
Hi Dmitry,
While you're at it, could you also replace <code>...</code> with {@code
...} at lines that you're changing anyway. E.g.:
src/share/classes/java/awt/DefaultKeyboardFocusManager.java
> - * <code>enqueueKeyEvents</code>, or any value < 0
> + * <code>enqueueKeyEvents</code>, or any value < 0
Here, {@code enqueueKeyEvents} would look better. Etc. Please don't
update lines that aren't modified in your fix already.
Also, I second to Yuri, you don't need a <code> (or {@code}) tag within
<pre></pre> (in e.g. SystemTray.java).
--
best regards,
Anthony
On 09/26/2013 12:25 PM, Dmitry Zinkevich wrote:
> Hello everybody,
>
> please, review my changes to javadoc in awt classes.
>
> Bug with classes listed: https://bugs.openjdk.java.net/browse/JDK-8025218
> Review request:
> http://cr.openjdk.java.net/~alexsch/dmitry-zinkevich/8025218/
>
More information about the awt-dev
mailing list