On Tue, 20 Dec 2022 04:37:32 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> The JvmtiVTMSTransitionDisabler has to be a no-op for a non-virtual thread. 
>> It is to avoid different branches where it is possible for virtual and 
>> non-virtual cases and use this in `jvmtiEnv.cpp`:
>> `  JvmtiVTMSTransitionDisabler disabler(thread);`
>> I've added comments that mark no-op cases.
>
> I see. I assumed the name `_vthread` implied it was actually known to be a 
> vthread. Really the field and parameters should just be called `thread`.

Okay, renamed now.

-------------

PR: https://git.openjdk.org/jdk/pull/11690

Reply via email to