On Wed, 13 Aug 2025 07:16:05 GMT, Serguei Spitsyn <[email protected]> wrote:
>> Leonid Mesnik has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> simplified after feedback
>
> src/hotspot/share/prims/jvmtiExport.cpp line 1837:
>
>> 1835: JvmtiThreadState *state = nullptr;
>> 1836: {
>> 1837: ThreadInVMfromJava __tiv(thread);
>
> Nit: Maybe rename: `__tiv` => `tiv`. The prefix `__` is normally used in
> macros to avoid potential naming conflicts.
Thank you for the update on this!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26713#discussion_r2274671305
