On Fri, 26 Apr 2024 22:59:43 GMT, Alex Menkov <amen...@openjdk.org> wrote:

> Some cleanup related to JvmtiEnvBase::get_threadOop_and_JavaThread method
> 
> Testing: tier1-6

This looks good in general. Thank you for taking care about it. The Runtime 
team suggested to use the name `current` for the current thread. The plan is to 
unify the JVMTI code as well. So, I'd suggest to rename all occurrences of the 
`current_thread` that you touched to the `current`. It will be a good step in a 
right direction. I'm already doing it for some time.

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

PR Review: https://git.openjdk.org/jdk/pull/18986#pullrequestreview-2030037342

Reply via email to