<AWT Dev> [8] Review Request: JDK-8025236 [javadoc] fix some errors in AWT
Anthony Petrov
anthony.petrov at oracle.com
Tue Oct 1 11:14:53 PDT 2013
You should only change the <code> issues in lines that are already
modified in your fix. No massive reformatting please.
And yes, the rest of the procedure is to publish the second version of
the webrev, and post a link to this mailing list.
--
best regards,
Anthony
On 09/30/2013 05:52 PM, Dmitry Ginzburg wrote:
> So, should I fix all the issues with this "<code>"'s, do webrev again
> and send it to yan, and send it here again?
>
> Thanks,
>
> -Dmitry
>
> 27.09.2013 17:52, Anthony Petrov wrote:
>> The fix looks good to me. Again, if possible, I'd suggest to replace
>> <code> with {@code ..} if you're changing a line anyway.
>>
>> --
>> best regards,
>> Anthony
>>
>> On 09/26/2013 06:21 PM, Dmitry Ginzburg wrote:
>>> Hello, AWT Team.
>>>
>>> Please review the fix for the following issue:
>>> https://bugs.openjdk.java.net/browse/JDK-8025236
>>> The fix is available at:
>>> http://cr.openjdk.java.net/~yan/jdk-8025236/webrev.00/
>>>
>>> This is the fix for javadoc errors, on which doclint was showing some
>>> issues.
>>>
>>> The patch contains only simple markup fixes; no changes/fixes in
>>> documentation text; the specification itself wasn't changed.
>>>
>>> Thanks,
>>> -Dmitry
More information about the awt-dev
mailing list