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