RFR: 8343477: Remove unnecessary @SuppressWarnings annotations (compiler) [v6]

Joe Darcy darcy at openjdk.org
Mon Dec 9 19:38:42 UTC 2024


On Mon, 9 Dec 2024 19:20:35 GMT, Archie Cobbs <acobbs at openjdk.org> wrote:

>> Please review this patch which removes unnecessary `@SuppressWarnings` annotations.
>
> Archie Cobbs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 10 additional commits since the last revision:
> 
>  - Merge branch 'master' into SuppressWarningsCleanup-compiler
>  - Merge branch 'master' into SuppressWarningsCleanup-compiler
>  - Merge branch 'master' into SuppressWarningsCleanup-compiler
>  - Merge branch 'master' into SuppressWarningsCleanup-compiler
>  - Update copyright years.
>  - Merge branch 'master' into SuppressWarningsCleanup-compiler
>  - Merge branch 'master' into SuppressWarningsCleanup-compiler
>  - Apply change that was missed somehow.
>  - Undo change that will be moved to the core-libs branch.
>  - Remove unnecessary @SuppressWarnings annotations.

Looks fine; thanks for the cleanup @archiecobbs .

Bumped the reviewer count so someone who more directly works on javac will take a look too.

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

Marked as reviewed by darcy (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21851#pullrequestreview-2489826295


More information about the compiler-dev mailing list