On Tue, 4 Mar 2025 18:23:42 GMT, Daniel Fuchs <[email protected]> wrote:
>> serhiysachkov has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8281511: review comments updates
>
> test/jdk/java/net/ipv6tests/Tests.java line 161:
>
>> 159: if (got < expected) {
>> 160: throw new RuntimeException("checkIfTimeOut failed: got " +
>> got
>> 161: + ", expected at least " + expected );
>
> `checkTime` is also called in `TcpTests`. You will need to modify that test
> and verify that the new semantics applies there too.
I see that `TcpTests` also uses `System.currentTimeMillis` so you could take
the opportunity to fix that too.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23840#discussion_r1979991904
