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 src/hotspot/share/prims/jvmtiEnvBase.cpp line 1976: > 1974: oop thread_obj = nullptr; > 1975: > 1976: jvmtiError err = > JvmtiEnvBase::get_threadOop_and_JavaThread(tlh.list(), target, current, > &java_thread, &thread_obj); I think a good cleanup would be to also replace `current` with `current_thread`, although I'm not sure how common each are. I see 3 `current` references in this webrev. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/18986#discussion_r1581610079
