RFR: JDK-8266748: Move modifiers code to Signatures.java [v2]
Hannes Wallnöfer
hannesw at openjdk.java.net
Mon Jun 7 13:30:58 UTC 2021
On Thu, 3 Jun 2021 17:22:26 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
> Minor changes suggestion, to access items from the configuration where possible.
>
> As a followup, it would be good to look at the remaining uses of `HtmlDocletWriter` in `Signatures`. There is at least a theme of "methods to create links", and if those methods are just used in the `Signatures` class, they would be candidates to move into that class as well.
I changed the mentioned accesses to use configuration instead of writer.
Most of the link methods in `HtmlDocletWriter` are used all over the place. `addSrcLink` is only used in `TypeSignature` and `MemberSignature`, but there's various circumstances that makes a move to Signatures unattractive (it accesses fields in HtmlDocletWriter, to which static nested classes TypeSignature and MemberSignature don't share a common reference).
-------------
PR: https://git.openjdk.java.net/jdk/pull/4142
More information about the javadoc-dev
mailing list