RFR: JDK-8263507: Improve structure of package summary pages [v5]

Hannes Wallnöfer hannesw at openjdk.java.net
Fri May 7 10:43:00 UTC 2021


On Thu, 6 May 2021 14:54:11 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

>> Hannes Wallnöfer has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   JDK-8263507: More suggested code cleanup
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/AllClassesIndexWriter.java line 125:
> 
>> 123:                 .addTab(contents.exceptions.toString(), e -> utils.isException((TypeElement)e))
>> 124:                 .addTab(contents.errors.toString(), e -> utils.isError((TypeElement)e))
>> 125:                 .addTab(contents.annotationTypes.toString(), utils::isAnnotationType);
> 
> Maybe the `*Tab` methods could be overloaded to accept `Content` ?  Maybe later cleanup?
> What does `.toString()` do in the malformed case of providing an `HtmlTree` ... should there be a method on `Content`? Maybe later cleanup?

The `*Tab` methods could and should be converted to accept `Content`. I didn't want to pack too many unrelated changes into this issue so I have filed a separate one for it:

https://bugs.openjdk.java.net/browse/JDK-8266651

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

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


More information about the javadoc-dev mailing list