RFR: 8365983: Tests should throw SkippedException when SCTP not supported [v2]
SendaoYan
syan at openjdk.org
Tue Sep 2 07:01:56 UTC 2025
On Tue, 2 Sep 2025 06:03:34 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> SendaoYan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Replace "throw new jtreg.SkippedException" ad "throw new SkippedException"
>
> test/jdk/com/sun/nio/sctp/SctpMultiChannel/SocketOptionTests.java line 42:
>
>> 40: import com.sun.nio.sctp.AbstractNotificationHandler;
>> 41: import com.sun.nio.sctp.Association;
>> 42: import com.sun.nio.sctp.SctpStandardSocketOptions.InitMaxStreams;
>
> Was this intentional? Few lines below, there's already a static wildcard import for this:
>
>
> import static com.sun.nio.sctp.SctpStandardSocketOptions.*;
Maybe the redundant import added by IDE automatically. I have deleted the unnecessary import and verified it locally. Thanks your reviews.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26903#discussion_r2315107144
More information about the net-dev
mailing list