On Mon, 3 Aug 2026 20:05:34 GMT, Leonid Mesnik <[email protected]> wrote:
>> David CARLIER has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Apply feedback: use NATIVE_METHOD_PREFIX naming for the test constant
>
> test/hotspot/jtreg/serviceability/jvmti/SetNativeMethodPrefix/libPrefixedNativeStackWalk.cpp
> line 30:
>
>> 28: extern "C" {
>> 29:
>> 30: static const char* const NativeMethodPrefix = "wrapped_";
>
> Please use
> `NATIVE_METHOD_PREFIX`
> name which the standard choice for constants.
I tried temporarily the log_info route locally and showed which frames were
skipped rather than the read went out of bounds though.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32180#discussion_r3709457354
