RFR: 8352612: No way to add back lint categories after "none" [v3]

Maurizio Cimadamore mcimadamore at openjdk.org
Thu Apr 10 15:36:35 UTC 2025


On Thu, 10 Apr 2025 15:32:39 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> src/jdk.compiler/share/classes/com/sun/tools/javac/resources/javac.properties line 171:
>> 
>>> 169:     Append to the bootstrap class path
>>> 170: javac.opt.Xlint=\
>>> 171:     Enable recommended lint warning categories
>> 
>> I think the use of "mandatory warning" above is good. However, I find the use of `lint warning` a bit too "implementation-related". I believe I liked the old text better, even if it was more vague.
>
> Related, as part of this PR it might be needed to do a pass on the man pages, to make sure they are in sync -- I see for instance this:
> 
>        -nowarn
>               Disables warning messages. This option operates the same as the -Xlint:none option.
> 
> Which would no longer be valid after this change. E.g.
> 
> 
> javac -Xlint:none -Xlint:serial
> 
> is not the same as:
> 
> javac -nowarn -Xlint:serial

I also wonder if the combo `nowarn` + (non-empty lint set) should result in an option warning (as the lint directive will be skipped) ?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/24227#discussion_r2037696269


More information about the compiler-dev mailing list