RFR: 8344446: Remove security manager dependency from module jdk.sctp [v2]

Brian Burkhalter bpb at openjdk.org
Tue Nov 19 17:01:12 UTC 2024


On Tue, 19 Nov 2024 16:33:49 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8344446: Address reviewer comments
>
> src/jdk.sctp/unix/classes/sun/nio/ch/sctp/SctpChannelImpl.java line 1085:
> 
>> 1083:     }
>> 1084: 
>> 1085:     @SuppressWarnings({"restricted"})
> 
> Nit, you don't need the braces.

Removed in a73a313.

> src/jdk.sctp/unix/classes/sun/nio/ch/sctp/SctpServerChannelImpl.java line 243:
> 
>> 241: 
>> 242:             IOUtil.configureBlocking(newfd, true);
>> 243:             return new SctpChannelImpl(provider(), newfd);
> 
> You can also delete line 216.

Deleted in a73a313.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22225#discussion_r1848726536
PR Review Comment: https://git.openjdk.org/jdk/pull/22225#discussion_r1848726638


More information about the net-dev mailing list