On Tue, 29 Apr 2025 01:44:22 GMT, Chris Plummer <[email protected]> wrote:
>> test/hotspot/jtreg/vmTestbase/nsk/share/jdi/Debugee.java line 463:
>>
>>> 461: EventSet eventSet = eventQueue.remove(timeLeft);
>>> 462: if (eventSet == null) {
>>> 463: return null;
>>
>> This does not match the comment at the line 438:
>>
>> * Wait for the requested event and skip other events.
>
> What is implied here is waiting for "timeout" ms, not indefinitely. Nothing
> has changed in that regard as the method was always intended to return after
> the timeout expires.
Okay, thanks.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24839#discussion_r2067542009
