On Tue, 12 May 2026 06:03:51 GMT, Leonid Mesnik <[email protected]> wrote:
>> test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/events/EM02/em02t003/em02t003.cpp >> line 480: >> >>> 478: return false; >>> 479: >>> 480: /* Give some time to complete already processing cbNew* events. */ >> >> cbNew* is confusing. I would just say "Give some time to complete events >> already in progress." > > I wanted to focus that it is only needed when newEventCount are checked and > correspnsing cbNewCompiledMethodLoad and cbNewCompiledMethodUnload events. I agree, it is a little bit confusing. Maybe to say: - `cbNewCompiledMethod*` events OR - `cbNewCompiledMethodLoad` and `cbNewCompiledMethodUnload` events ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31128#discussion_r3238585190
