RFR: JDK-8200363 Add javadoc command line setting to fail on warnings
Hannes Wallnöfer
hannes.wallnoefer at oracle.com
Wed Jan 29 15:18:46 UTC 2020
Hi Jon,
This looks good. I just found a few minor things to clean up:
- in ToolOptions.java the definitions of the rejectWarnings field and method both have comments mentioning the old -Xwerror option. These should be updated to refer to the new -Werror option
- main.opt.Xold.desc property should probably also be removed from javadoc_ja.properties and javadoc-zh_CN.properties. It is also documented in share/man/javadoc.1
Hannes
> Am 28.01.2020 um 23:46 schrieb Jonathan Gibbons <jonathan.gibbons at oracle.com>:
>
> Please review a simple change to (finally) implement the long-overdue -Werror option.
>
> The existing undocumented -Xwerror option is repurposed and renamed, and its behavior modified to match the javac -Werror option. The old name is retained as a alias "for a while" but since it is an undocumented option it may be removed at any time, and perhaps in the next release after this change (i.e. 16.)
>
> In the resource file, an unused property is removed, and an error (mismatched braces) is fixed in another.
>
> -- Jon
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8200460
> Webrev: http://cr.openjdk.java.net/~jjg/8200363/webrev.00/
>
More information about the javadoc-dev
mailing list