RFR: 8345263: Make sure that lint categories are used correctly when logging lint warnings [v2]

Archie Cobbs acobbs at openjdk.org
Thu Dec 5 22:18:39 UTC 2024


On Thu, 5 Dec 2024 22:08:48 GMT, Vicente Romero <vromero at openjdk.org> wrote:

>> Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add missing lint categories in compiler.properties
>>   Simplify some lambda expressions
>>   Simplify Lint::logIfEnabled by accepting a log parameter
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Check.java line 2972:
> 
>> 2970:                     if (belongsToRestrictedPackage(sym)) {
>> 2971:                         log.warning(tree.pos(),
>> 2972:                                     Warnings.AccessToMemberFromSerializableLambda(sym));
> 
> so this one won't be a lint warning anymore I guess

Actually I think this is another accidentally omitted conversion from `Warnings` to `LintWarnings`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22553#discussion_r1872230542


More information about the build-dev mailing list