On Mon, 9 Dec 2024 19:20:09 GMT, Archie Cobbs <aco...@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 pull request now contains 12 commits: > > - Merge branch 'master' into SuppressWarningsCleanup-security > - Remove another unnecessary @SuppressWarnings annotation. > - Remove another unnecessary @SuppressWarnings annotation. > - Merge branch 'master' into SuppressWarningsCleanup-security > - Remove another unnecessary warning suppression. > - Merge branch 'master' into SuppressWarningsCleanup-security > - Merge branch 'master' into SuppressWarningsCleanup-security > - Revert changes to imported sources. > - Update copyright years. > - Merge branch 'master' into SuppressWarningsCleanup-security > - ... and 2 more: https://git.openjdk.org/jdk/compare/cc628a13...649d2f33 The bug needs an appropriate `noreg` label (I suggest `noreg-cleanup`). Otherwise, looks good to me. ------------- Marked as reviewed by mullan (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/21844#pullrequestreview-2502865486
