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:30:31 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Archie Cobbs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Correct regression test expected output.
>
> 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

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

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


More information about the compiler-dev mailing list