On Thu, 24 Dec 2020 21:16:35 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:

> Using the `@serial` tag, a class can explicitly opt-in to being documented on 
> the serialized form page.  However, a class that does not want to opt-in 
> should not be required to have a doc comment, and should not trigger the 
> "missing comment" warning if there is no comment.
> 
> The fix is to check for the presence of the comment on a class before 
> analyzing it to see if it contains `@serial` tags.
> 
> The code in `Utils.getDocCommentTree0` is slightly refactored to allow a new 
> predicate `hasDocCommentTree` to be added.

Looks good, apart from two typos in a comment.

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/Utils.java
 line 2689:

> 2687: 
> 2688:     /**
> 2689:      * Checks whether an element has an associated doc copmment.

typo: copmment

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/Utils.java
 line 2691:

> 2689:      * Checks whether an element has an associated doc copmment.
> 2690:      * @param element the element
> 2691:      * @return {@code true} if the eleme nt has a comment, and false 
> otherwise

typo: eleme nt

-------------

Marked as reviewed by hannesw (Reviewer).

PR: https://git.openjdk.java.net/jdk16/pull/70

Reply via email to