[jdk17] RFR: JDK-8259499: Handling type arguments from outer classes for inner class in javadoc [v2]
Hannes Wallnöfer
hannesw at openjdk.java.net
Mon Jul 12 16:43:33 UTC 2021
> This change adds support for generating HTML links to the type arguments of enclosing classes when creating a link to an inner class. Previously, only a link to the inner class was created and the type arguments were even omitted from the link label.
>
> The new feature to create separate links to the enclosing class and its type arguments is only activated if the enclosing class has type arguments. If the enclosing class is non-generic, the old behavior is preserved to create a single link to the inner class. The reason for this is that a dedicated link to the enclosing class itself provides little benefit, since it can be easily reached via the "Enclosing class" link of the inner class. Also, linking the enclosing type in absence of type arguments makes it hard to see that there are two links and easy to click on the wrong link by mistake.
>
> On the other hand, for type arguments a separate link should be useful since it is often not a "nearby" type. It is also easier to detect the different links than for non-generic nested classes. I came to like this "mixed" solution best after trying several other approaches.
Hannes Wallnöfer 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 four additional commits since the last revision:
- JDK-8259499: Add newline at end of new test
- JDK-8259499: Refactor code to use new Utils.isGenericType method, add workaround and test for JDK-8269706
- Merge branch 'master' into JDK-8259499
- JDK-8259499: Handling type arguments from outer classes for inner class in javadoc
-------------
Changes:
- all: https://git.openjdk.java.net/jdk17/pull/195/files
- new: https://git.openjdk.java.net/jdk17/pull/195/files/b4702d97..cdc9f29c
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk17&pr=195&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk17&pr=195&range=00-01
Stats: 26732 lines in 478 files changed: 16760 ins; 8050 del; 1922 mod
Patch: https://git.openjdk.java.net/jdk17/pull/195.diff
Fetch: git fetch https://git.openjdk.java.net/jdk17 pull/195/head:pull/195
PR: https://git.openjdk.java.net/jdk17/pull/195
More information about the javadoc-dev
mailing list