<Swing Dev> Review Request for 8044281: Fix doclint warnings (missing javadoc @param and @return tags) in javax.swing.text - Part 1 of 2
Petr Pchelko
petr.pchelko at oracle.com
Thu Jul 10 10:39:54 UTC 2014
Hello, Rocky.
A couple of comments.
1. Please control the correct indentation of the second line of the @param/@return tag descriptions. You have many places where it's incorrect.
2. Please use @code instead of <code></code>
3. AbstractWriter - no need to mention variable names in the javadoc. Please remove it.
4. AbstractWriter:374 - better to say {@code true} if the lines will be wrapped, {@code false} otherwise. Also please remove the last line of the doc.
5. The indentation of @param comments is incorrect. You have:
> * @param chars character array containing content to write
> * @param startIndex offset in the array to start writing from;
> * 0 starts at the beginning
> * @param length number of characters to write
And you should have:
> * @param chars character array containing content to write
> * @param startIndex offset in the array to start writing from;
> * 0 starts at the beginning
> * @param length number of characters to write
6. AsyncBoxView:451 - it's not a boolean!
With best regards. Petr.
On 10 июля 2014 г., at 13:36, Rocky Sloan <rocky.sloan at oracle.com> wrote:
> Hello,
>
> Could you please review Part 1 of the fix for the following bug:
> https://bugs.openjdk.java.net/browse/JDK-8044261
>
> This is part 1 of 2.
>
> Webrev corresponding:
> http://cr.openjdk.java.net/~yan/8044261/webrev.01/
>
> Change:
> Add missing @return and @param javadoc tags in javax.swing.text classes to fix doclint warnings.
>
> Thanks,
>
> - Rocky
>
More information about the swing-dev
mailing list