RFR: 8330852: All callers of JvmtiEnvBase::get_threadOop_and_JavaThread should pass current thread explicitly
Chris Plummer
cjplummer at openjdk.org
Fri Apr 26 23:41:06 UTC 2024
On Fri, 26 Apr 2024 22:59:43 GMT, Alex Menkov <amenkov at 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
More information about the hotspot-dev
mailing list