RFR: JDK-8210555: create --source --target synonyms for -source -target
Hannes Wallnöfer
hannes.wallnoefer at oracle.com
Wed Sep 26 14:01:32 UTC 2018
Looks good to me.
Hannes
> Am 25.09.2018 um 22:22 schrieb Jonathan Gibbons <jonathan.gibbons at oracle.com>:
>
> Please review a mostly-trivial change to add --source and --target as
> synonyms for -source and -target in javac and javadoc.
>
> There's also some related test-cleanup.
>
> Here's a brief explanation of the changes, listed in the order given in the
> webrev/index.html file.
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/main/Option.java
> Add synonyms in javac
>
> src/jdk.javadoc/share/classes/com/sun/tools/javadoc/main/Start.java
> Replace a hard-coded use of the option name with a symbolic reference
>
> src/jdk.javadoc/share/classes/com/sun/tools/javadoc/main/ToolOption.java
> Add synonym in the old/deprecated version of the javadoc tool
> In this version, option aliases are only available using distinct ToolOption members
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/tool/ToolOption.java
> Add synonym in the main/current version of the javadoc tool
> In this version, option aliases are supported directly, as in javac
>
> test/langtools/jdk/javadoc/tool/api/basic/IsSupportedOptionTest.java
> We don't have good regular testing infrastructure for testing javadoc options;
> this test is as good as any for a basic test that the alias is recognized.
>
> test/langtools/jdk/javadoc/tool/sourceOption/SourceOption.java
> There is no code change in this test, but the comment was
> broken, and way too version-specific. The comment is rewritten
> to be less version-specific.
>
> test/langtools/tools/javac/options/IsSupportedOptionTest.java
> As with the javadoc version of this test given above,
> we don't have good regular testing infrastructure for testing javac options;
> this test is as good as any for a basic test that the alias is recognized.
>
> test/langtools/tools/javadoc/sourceOption/SourceOption.java
> This is the version of the SourceOption test for the old/deprecated version
> of javadoc. The new comment is a cut-n-paste copy of the new text
> for the version of the test for the standard version of javadoc, given above.
>
> test/langtools/tools/javadoc/api/basic/IsSupportedOptionTest.java
> This test was in the set of tests for the old/deprecated version of javadoc,
> but was not specific to the old doclet, and the text was almost identical
> to the other version, given above. The only differences in the text was
> a 1-character difference in the copyright date, and a 1-character typo
> that has already been corrected in the new version.
>
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8210555
> Webrev: http://cr.openjdk.java.net/~jjg/8210555/webrev.00/
> CSR: https://bugs.openjdk.java.net/browse/JDK-8210556
>
> -- Jon
More information about the compiler-dev
mailing list