On Sun, 3 Nov 2024 03:08:12 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 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

Reply via email to