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