RFR [15] 8238167: Remove stray files from jdk.javadoc
Pavel Rappo
pavel.rappo at oracle.com
Wed Jan 29 15:39:13 UTC 2020
Hello,
Please review the change for https://bugs.openjdk.java.net/browse/JDK-8238167:
http://cr.openjdk.java.net/~prappo/8238167/webrev.00/
This is a cleanup change that removes a couple of files that I have not found
to be of any use.
The first file, jquery.js, was introduced in
changeset: 33920:bd731341c405
user: bpatel
date: Fri Nov 20 20:55:07 2015 +0000
summary: 8141492: Implement search feature in javadoc
That same changeset also brought the jquery-1.10.2.js file. While the contents
of these files were identical, only the latter file was used. Since then both
files have been updated together, with the same content. Every time that happened,
the latter file, jquery-1.10.2.js, was simultaneously renamed to reflect the
version of the JQuery library it was updated with.
As of today, jquery.js is still not used.
The second file, help.html, is part of the test scenario. The file first appeared in
changeset: 36705:890c250d8da8
parent: 36503:4a95f4b1bd8b
user: ksrini
date: Mon Mar 14 15:04:57 2016 -0700
summary: 8071982: Update tests for revamped Doclet API
I flagged this file as suspicious while testing the change for 8237909: Remove
zipped index files feature. I noticed that the TestOptions test did not seem to
care whether the JSZip library was present or not, despite the fact that its
resource, help.html, explicitly referred to that library. Further investigation
showed that this file was never used by that test, or anyone for that matter.
The javadoc tool has only a handful of "help" options:
javadoc -help | grep help
--help, -help, -?, -h
--help-extra, -X
-helpfile <file>
Include file that help link links to
-nohelp Do not generate help link
all of which are covered by the TestHelpOption and TestXOption tests. There
doesn't seem to be any need in testing anything related to "help" options in
TestOptions. Its @summary, otherwise very specific, certainly doesn't mention
anything like that.
***
Both files, jquery.js and help.html, have been diligently updated over the
years. They seem to be leftovers from the development stage. Since they are not
used, updating those files is a burden to maintainers and provides nothing in
return. I propose to delete them.
All javadoc tests pass, and the functionality of the generated pages
seems to be intact.
Thanks,
-Pavel
More information about the javadoc-dev
mailing list