On Fri, 5 Mar 2021 21:44:20 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:

> Please review a moderately simple fix to improve DocLint's checking for empty 
> tags, and in particular, bad use of `<p>` as a paragraph terminator. This 
> will help detect this bad usage at doclint-time, instead of later, with 
> doccheck.
> 
> There was already code to check if tags were empty. (`warnIfEmpty`), and code 
> to check for open tags at the end of the comment. The change is primarily to 
> invoke the "check for open tags" at the end of main body of a comment and at 
> the end of each block tag (i.e including all that allow HTML content in the 
> body of the block tag.) 
> 
> There is a secondary minor edit to check if a tag was empty when auto-closing 
> it.
> 
> The changes discovered 5 instances of bad use of `<p>` in the JDK docs. These 
> are being fixed separately in https://bugs.openjdk.java.net/browse/JDK-8263104

looks good

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

Marked as reviewed by vromero (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2853

Reply via email to