RFR: 8224228: No way to locally suppress lint warnings in parser/tokenizer or preview features [v13]

[Archie Cobbs]

On Wed, 26 Feb 2025 12:57:11 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

> rather than coming up with a more general way to map positions back to tree (which, if done, well at that point you could well be using positions everywhere)

Yes that was the original idea... i.e., eventually use positions everywhere, but "using positions everywhere" would require a larger refactoring of how warnings are handled that is a little too far in the future to assume right now.

> I think I would prefer not to augment the trees with end positions until/unless we need them.

Agreed - I'll plan to remove that part and close #23669 as part of the next update here.

> Regarding the new lint warnings - we probably should be careful with introducing new lints until we have some time to think about both conceptual and implementation directions for lints inside javac.

That's fine, right now I'm interested in simply making suppressible lexical warnings feasible. Actually adding (more of) them is a separate issue we can consider later.

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

PR Comment: https://git.openjdk.org/jdk/pull/23237#issuecomment-2685933603