RFR: 8322708: Global HTML attributes are not allowed [v3]

Pavel Rappo prappo at openjdk.org
Fri Jun 14 14:56:21 UTC 2024


On Fri, 14 Jun 2024 12:15:45 GMT, Nizar Benalla <nbenalla at openjdk.org> wrote:

>> Can I please get a review for this change, that aims to add support for Global HTML tags.
>> Here is the [link](https://cr.openjdk.org/~nbenalla/javadocGlobalPR/pkg1/package-summary.html) to the generated docs.
>> Thanks in advance.
>
> Nizar Benalla has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains ten additional commits since the last revision:
> 
>  - Null safety
>  - Merge branch 'master' into html-attributes
>  - remove trailing whitespace
>  - -Add a boolean attribute to the enum type
>    -Simply regex in `visitAttribute`
>    -Simplified the Test
>    -Added a negative test
>  - no longer print summary
>  - no longer print summary
>  - Add small comment
>  - Remove classpath exception
>  - Allow global variables

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclint/HtmlTag.java line 672:

> 670:         AttrKind kind = null;
> 671: 
> 672:         if (attr != null) {

Can the remainder of this method be simplified using `Map.getOrDefault`?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19652#discussion_r1639952309


More information about the javadoc-dev mailing list