Initial webrev with changes for JDK 9
Bhavesh Patel
bhavesh.x.patel at oracle.com
Wed Mar 16 05:56:10 UTC 2016
Hi,
I have review the javadoc (including doclet and tests) changes. It
mostly looks good. Following are my comments on the changes
1) jdk/javadoc/internal/doclets/formats/html/ConfigurationImpl.java
- (Line 522) PackageSymbol "pd" is cast to ModuleSymbol which is incorrect.
2) jdk/javadoc/internal/doclets/toolkit/AbstractDoclet.java
- (Lines 109 and 111) e.printStackTrace() needs to be deleted.
3) jdk/javadoc/internal/doclets/formats/html/PackageWriterImpl.java
- (Lines 130 - 135) The Module label and name gets enclosed in <p>. We
need to update this to enclose them in a <div> and <span> similar to
what we do in
jdk/javadoc/internal/doclets/formats/html/ClassWriterImpl.java (Lines
200 - 205).
Other minor changes that are needed are as follows
4) com/sun/tools/doclets/formats/html/PackageWriterImpl.java
- The class lists no changes other than the copyright year change. If no
changes have happened, the copyright year change needs to be reverted.
5) com/sun/tools/doclets/internal/toolkit/taglets/TagletManager.java
- (Line 275) Missing @param.
- (Line 292) Missing documentation.
6) com/sun/tools/doclets/internal/toolkit/util/DocFileFactory.java
- Copyright year shows 2012 instead of 2016.
7) com/sun/tools/javadoc/DocletInvoker.java
- (Line 367) Missing @param.
- (Line 384) Missing documentation.
8) jdk/javadoc/internal/doclets/formats/html/WriterFactoryImpl.java
- (Lines 94 - 102) needs to be deleted.
9) jdk/javadoc/internal/doclets/toolkit/AbstractDoclet.java
- Line 121 needs to be deleted
10) jdk/javadoc/internal/doclets/toolkit/Configuration.java
- Extra line 398 needs to be deleted.
- (Line 654) Not updated as a part of this fix but @return needs to be
updated.
11) jdk/javadoc/internal/doclets/toolkit/WriterFactory.java
- (Line 74) The param name should be "mdle" instead of "module"
12) jdk/javadoc/internal/doclets/toolkit/builders/BuilderFactory.java
- (Line 108) The param name should be "mdle" instead of "module"
13) jdk/javadoc/internal/doclets/toolkit/taglets/TagletManager.java
- (Line 266) Missing @param.
- (Line 285) Missing documentation.
14) jdk/javadoc/internal/doclets/toolkit/util/DocPaths.java
- (Line 158) should be "The name of the file for the module overview frame."
15) jdk/javadoc/internal/tool/Start.java
- (Line 313) Missing @param.
16) jdk/javadoc/internal/doclets/toolkit/builders/ModuleSummaryBuilder.java
- (Line 80) "module" should be "mdle".
- (Line 95) "module" should be "mdle".
Regards,
Bhavesh.
More information about the jigsaw-dev
mailing list