On Thu, 8 Jun 2023 19:05:54 GMT, Doug Simon <dnsi...@openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> update copyright comments > > src/hotspot/share/runtime/sharedRuntime.cpp line 641: > >> 639: JRT_ENTRY(void, SharedRuntime::notify_jvmti_vthread_start(oopDesc* vt, >> jboolean hide, JavaThread* current)) >> 640: assert(hide == JNI_FALSE, "must be VTMS transition finish"); >> 641: jobject vthread = JNIHandles::make_local(const_cast<oopDesc*>(vt)); > > Since the current thread is in the `current` arg, it could be used here when > creating the local handle. That's right. Thanks. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13484#discussion_r1223851214
