RFR: JDK-8218998: Add metadata to generated API documentation files

Hannes Wallnöfer hannes.wallnoefer at oracle.com
Tue Feb 19 15:27:40 UTC 2019


Hi Jon,

Looks good, a few minor issues listed below.

 - In PackageUseWriter.java there is a duplicate invocation of getDescription() assigned to an unused variable.
 - Not directly related to your patch, but parameter includeScript in HtmlDocletWriter::printHtmlDocument is never used.
 - Line 74 in TestMetadata.java is misaligned.

Hannes

PS: TestMetadata.java is quite dense, took me some time to review but very instructional.


> Am 15.02.2019 um 21:16 schrieb Jonathan Gibbons <jonathan.gibbons at oracle.com>:
> 
> Please review a medium-small patch to add some new metadata into each HTML document that is generated by javadoc.
> The metadata provides the following:
> 
> 	• a short stylized description of the contents of the file, for potential use by other tools to filter the set of generated files
> 	• an indication of the part of javadoc that generated the file, to help group the different kinds of pages generated by javadoc
> A new test program is added. It's a bit of overkill, but it is designed to be future-proof, meaning if we add new types of pages, this test will (intentionally) fail until it is updated to know about those new pages.  I also note that the test was instrumental in helping to discover two unrelated bugs in javadoc. These are linked in the "related links" in the JBS issue.
> 
> -- Jon
> JBS: https://bugs.openjdk.java.net/browse/JDK-8218998
> Webrev: http://cr.openjdk.java.net/~jjg/8218998/webrev.00/
> 
> 
> 
> 
> 



More information about the javadoc-dev mailing list