On Wed, 30 Nov 2022 13:50:13 GMT, Daniel Fuchs <dfuchs@openjdk.org> wrote:

>> Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   deprecated api doc text
>
> src/java.management/share/classes/javax/management/loading/MLetObjectInputStream.java line 43:
> 
>> 41: class MLetObjectInputStream extends ObjectInputStream {
>> 42: 
>> 43:     @SuppressWarnings("removal")
> 
> Shouldn't `MLetObjectInputStream` be deprecated for removal too? I mean - if MLet was removed - would we need to keep that class? If it were deprecated for removal too then I suspect that there would be no need to suppress the warning here (and below).

It's not public so not part of the API, but you may be right that it would reduce the number of places where the warning needs to be suppressed.

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

PR: https://git.openjdk.org/jdk/pull/11430