> 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 incrementally with one additional 
commit since the last revision:

  JDK-8259499: Remove extraneous import

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

Changes:
  - all: https://git.openjdk.java.net/jdk17/pull/195/files
  - new: https://git.openjdk.java.net/jdk17/pull/195/files/cdc9f29c..cdaf8006

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk17&pr=195&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk17&pr=195&range=01-02

  Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 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

Reply via email to