RFR: 8343483: Remove unnecessary @SuppressWarnings annotations (serviceability) [v3]

Archie Cobbs acobbs at openjdk.org
Tue Nov 12 19:07:30 UTC 2024


On Tue, 12 Nov 2024 18:54:30 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> 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 five additional commits since the last revision:
>> 
>>  - Merge branch 'master' into SuppressWarningsCleanup-serviceability
>>  - Update copyright years.
>>  - Merge branch 'master' into SuppressWarningsCleanup-serviceability
>>  - Merge branch 'master' into SuppressWarningsCleanup-serviceability
>>  - Remove unnecessary @SuppressWarnings annotations.
>
> How were these discovered and what testing did you do?

Hi @plummercj,

> How were these discovered

See [this thread](https://mail.openjdk.org/pipermail/compiler-dev/2024-November/028573.html). There is a proposal to add a new compiler lint warning for unnecessary `@SuppressWarnings` annotations and these were flagged by the prototype. 

> and what testing did you do?

The only testing done was verifying that the JDK still builds and passes the Github checks without them.

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

PR Comment: https://git.openjdk.org/jdk/pull/21857#issuecomment-2471340369


More information about the serviceability-dev mailing list