On Tue, 4 Mar 2025 18:25:58 GMT, Daniel Fuchs <[email protected]> wrote:
>> 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.
updated
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23840#discussion_r1981140394