On Wed, 16 Oct 2024 21:36:02 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> review: resolved comments from Alex and Chris
>
> test/hotspot/jtreg/serviceability/jvmti/events/NotifyFramePopStressTest/libNotifyFramePopStressTest.cpp
> line 99:
>
>> 97: res = jvm->GetEnv((void **) &jvmti, JVMTI_VERSION_9);
>> 98: if (res != JNI_OK || jvmti == nullptr) {
>> 99: LOG("GetEnv(JVMTI_VERSION_9) failedL error(%d)", res);
>
> Suggestion:
>
> LOG("GetEnv(JVMTI_VERSION_9) failed: error(%d)", res);
Thanks, fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21468#discussion_r1803965649
