On Mon, 11 Jul 2022 22:41:02 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> remove trailing white space > > test/hotspot/jtreg/serviceability/jvmti/events/FramePop/framepop02/libframepop02.cpp > line 78: > >> 76: >> 77: bool result = strncmp(inf.name, TEST_THREAD_NAME_BASE, >> strlen(TEST_THREAD_NAME_BASE)) == 0; >> 78: jvmti->Deallocate((unsigned char *)inf.name); > > Optional, we have 'deallocate' method in > './jdk/test/lib/jvmti/jvmti_common.h' which check error status. Leonid, thank you for review. Yes, I know it. But there are other places with Deallocate. I decided to keep it consistent and avoid touching other spots to avoid unnecessary risk. ------------- PR: https://git.openjdk.org/jdk19/pull/129
