On Tue, 2 Sep 2025 07:04:03 GMT, SendaoYan <s...@openjdk.org> wrote:

>> Currently, take test/jdk/com/sun/nio/sctp/SctpMultiChannel/Branch.java as 
>> example. When Util.isSCTPSupported() return false, then test print a warning 
>> and the exit.
>> 
>> I think it's better to throw jtreg.SkippedException rather than only print a 
>> warning and then report test passed.
>> 
>> Change has been verified locally, test-fix only, almost no risk.
>
> SendaoYan has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Remove extra blank line

These test-only changes look OK to me.

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

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26903#pullrequestreview-3175155396

Reply via email to