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