On Sun, 19 Apr 2026 03:27:37 GMT, Archie Cobbs <[email protected]> wrote:

>> This PR adds a new compiler warning for `@SuppressWarnings` annotations that 
>> don't actually suppress any warnings.
>> 
>> Summary of code changes:
>> 
>> * Add new warning and associated lint category `"suppression"`
>> * Update `LintMapper` to keep track of which `@SuppressWarnings` 
>> suppressions have been validated ¹
>> * Update `Log.warning()` so it validates any current suppression of the 
>> warning's lint category in effect.
>> * Add a new `validate` parameter to `Lint.isEnabled()` and 
>> `Lint.isSuppressed()` that specifies whether to also validate any current 
>> suppression.
>> * Add `Lint.isActive()` to check whether a category is enabled _or_ 
>> suppression of the category is being tracked - in other words, whether the 
>> warning calculation needs to be performed. Used for non-trivial warning 
>> calculations.
>> * Add `-Xlint:-suppression` flags to `*.gmk` build files so the build 
>> doesn't break
>> 
>> ¹ The suppression of a lint category is "validated" as soon as it suppresses 
>> some warning in that category
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Archie Cobbs has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains 143 commits:
> 
>  - Merge branch 'master' into JDK-8344159 to fix conflicts.
>  - Merge branch 'master' into JDK-8344159.
>  - Merge branch 'master' into JDK-8344159 to fix conflicts.
>  - Merge branch 'master' into JDK-8344159 to fix conflicts.
>  - Update copyrights to 2026.
>  - Merge branch 'master' into JDK-8344159
>  - Merge branch 'master' into JDK-8344159
>  - Merge branch 'master' into JDK-8344159
>  - Suppress new unnecessary suppresion warnings created by recent commits.
>  - Merge branch 'master' into JDK-8344159
>  - ... and 133 more: https://git.openjdk.org/jdk/compare/d71f070c...86889534

Marked as reviewed by prr (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/25167#pullrequestreview-4136373151

Reply via email to