On Tue, 2 Sep 2025 06:03:34 GMT, Jaikiran Pai <[email protected]> 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

Reply via email to