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