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