On Fri, 22 Mar 2024 04:09:28 GMT, Chris Plummer <[email protected]> wrote:
>> Alex Menkov has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - updated comments
>> - eventsReceived -> allEventsReceived
>
> test/hotspot/jtreg/vmTestbase/nsk/jdi/ClassPrepareEvent/referenceType/refType001.java
> line 276:
>
>> 274: try {
>> 275: if (!eventsReceivedLatch.await(eventTimeout,
>> TimeUnit.MILLISECONDS)) {
>> 276: log.complain("FAILURE 20: Timeout for waiting event
>> was exceeded");
>
> Should it be "Timeout waiting for all events was exceeded" or maybe "Timeout
> while waiting for all events"? Same thing for thread001.java.
Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18442#discussion_r1536240012