RFR: 8348611: Eliminate DeferredLintHandler and emit warnings after attribution [v10]
Archie Cobbs
acobbs at openjdk.org
Thu Aug 14 04:00:13 UTC 2025
On Wed, 13 Aug 2025 18:10:28 GMT, Archie Cobbs <acobbs at openjdk.org> wrote:
>> src/jdk.compiler/share/classes/com/sun/tools/javac/code/LintMapper.java line 250:
>>
>>> 248: Span span, // declaration's lexical range
>>> 249: Lint lint, // the Lint configuration that applies at this declaration
>>> 250: Symbol symbol, // declaration symbol (for debug purposes only; null for root)
>>
>> nit: should we keep this field in the final version? we can add it while debugging if needed
>
>> nit: should we keep this field in the final version? we can add it while debugging if needed
>
> I can remove it. Besides, this symbol is always the same object as the one that the `Lint` instance was created with, so if debugging were a concern, a better place to store it would be with the `Lint` instance anyway.
Unused field removed in 1f1247bc5cf.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24584#discussion_r2275275487
More information about the compiler-dev
mailing list