RFR: JDK-6251738: Want a top-level summary page that itemizes all spec documents referenced from javadocs (OEM spec) [v4]
Jonathan Gibbons
jjg at openjdk.java.net
Tue May 24 21:16:02 UTC 2022
On Tue, 24 May 2022 16:39:25 GMT, Pavel Rappo <prappo at openjdk.org> wrote:
>> Jonathan Gibbons has updated the pull request incrementally with one additional commit since the last revision:
>>
>> address review feedback
>
> src/jdk.compiler/share/classes/com/sun/source/util/DocTreeScanner.java line 515:
>
>> 513: * {@inheritDoc}
>> 514: *
>> 515: * @param node {@inheritDoc}
>
> There's something wrong with the `@implSpec` tags in this and the immediately following two methods. Might be an artifact of the patch.
Believed fixed.
> src/jdk.compiler/share/classes/com/sun/tools/javac/parser/DocCommentParser.java line 1533:
>
>> 1531: public DCTree parse(int pos) throws ParseException {
>> 1532: skipWhitespace();
>> 1533: DCText url = inlineWord();
>
> Nit: call it `uri` for consistency.
Update, the terminology is converging on `URL` in all public places, with `java.net.URI` being used as needed within the doclet implementation.
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/standard.properties line 432:
>
>> 430: doclet.extSpec.url.title=\
>> 431: url: {0}, title: "{1}"
>> 432:
>
> Use `uri` in all 4 resources?
Converging on URL.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8439
More information about the compiler-dev
mailing list