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