On Sun, 3 Nov 2024 03:17:14 GMT, Archie Cobbs <aco...@openjdk.org> wrote:
>> Please review this patch which removes unnecessary `@SuppressWarnings` >> annotations and `-Xlint:-foo` options. > > 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 four additional > commits since the last revision: > > - Update copyright years. > - Merge branch 'master' into SuppressWarningsCleanup-build > - Merge branch 'master' into SuppressWarningsCleanup-build > - Remove unnecessary @SuppressWarnings annotations and -Xlint:-key flags. Changes under `make` looks good, thanks for cleaning this up! I'm not sure who should review the changes under `src`. The `i18n` label does not seem correct for changes in dynalink and jsobject. ------------- Marked as reviewed by ihse (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/21859#pullrequestreview-2413243273
