RFR: 8343467: Remove unnecessary @SuppressWarnings annotations (security) [v2]

Sean Mullan mullan at openjdk.org
Mon Nov 4 19:40:29 UTC 2024


On Sun, 3 Nov 2024 03:08:12 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 four additional commits since the last revision:
> 
>  - Update copyright years.
>  - Merge branch 'master' into SuppressWarningsCleanup-security
>  - Merge branch 'master' into SuppressWarningsCleanup-security
>  - Remove unnecessary @SuppressWarnings annotations.

The files under `src/java.xml.crypto/share/classes/com/sun/org/apache/xml/internal/security/` and `src/java.xml.crypto/share/classes/org/jcp/xml/dsig/internal/dom` come from 3rd party source code where these warnings are still emitted. Please do not incorporate these changes since it would cause the JDK code to diverge.

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

PR Comment: https://git.openjdk.org/jdk/pull/21844#issuecomment-2455547055


More information about the security-dev mailing list