RFR: 8354090: Refactor import warning suppression in Check.java
Archie Cobbs
acobbs at openjdk.org
Thu Apr 10 15:28:47 UTC 2025
On Wed, 9 Apr 2025 01:20:50 GMT, Archie Cobbs <acobbs at openjdk.org> wrote:
> This is split off as a sub-task of [JDK-8224228](https://bugs.openjdk.org/browse/JDK-8224228), which seeks to add `@SuppressWarnings` support for lexical features.
>
> In JDK 9 and later, there is a special exception to the normal rules for deprecation and preview warnings, which is that these warnings are automatically suppressed for import statements. This is because an import statement is just a lexical shortcut, not an actual use of the symbol.
>
> The current code in `Check.java` implements this by manipulating the current `Lint` instance directly, but this operation is incompatible upcoming changes to deferred lint handling. Instead, there can be an explicit flag for "import warning suppression" in `Check.java` to handle this. This will also make the code a bit clearer.
Thanks for the review!
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24532#issuecomment-2794233673
More information about the compiler-dev
mailing list