RFR: 8224228: No way to locally suppress lint warnings in parser/tokenizer or preview features
Archie Cobbs
acobbs at openjdk.org
Thu Jan 23 18:26:54 UTC 2025
On Thu, 23 Jan 2025 18:17:12 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> This PR updates the `DeferredLintHandler` so that deferred warnings can be registered during parsing, before any `JCTree` nodes have been created, and it uses this new capability to update the `"preview"` and `"text-blocks"` Lint warnings so that they can be suppressed via `@SuppressWarnings` annotations. More details are provided in additional comments.
>
> Question: the change looks beneficial, but there's a lot going on in the PR. Would it be possible, do you think, to break it down into simpler parts? For instance, the push/pop logic seems something that we could review and integrate separately (and it's adding quite a bit of noise to the PR). Same for missing flushes etc.
Hi @mcimadamore,
Thanks for taking a look.
> Would it be possible, do you think, to break it down into simpler parts?
Absolutely, I will work on that.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23237#issuecomment-2610638948
More information about the compiler-dev
mailing list