Re: RFR: 8381862: Refactor remaining TestNG tests under jdk/net/ExtendedSocketOption to use JUnit [v3]

[Daisuke Yamazaki]

On Wed, 29 Apr 2026 20:47:54 GMT, Marcono1234 <d...@openjdk.org> wrote:

>> Daisuke Yamazaki has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Refactor NAPI tests to use ${test.main.class} for test execution
>
> test/jdk/jdk/net/ExtendedSocketOption/SocketNAPITest.java line 133:
> 
>> 131:                         sID = s.getOption(SO_INCOMING_NAPI_ID);
>> 132:                         cID = c.getOption(SO_INCOMING_NAPI_ID);
>> 133:                         if(initialRun) {
> 
> Missing space? (was already the case before)
> 
> Suggestion:
> 
>                         if (initialRun) {

Thanks @Marcono1234
I've fixed it

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30964#discussion_r3165733551