RFR: 8296492: Remove ObjectLocker in JVMTI get_subgroups call [v5]
Coleen Phillimore
coleenp at openjdk.org
Tue Nov 15 18:52:39 UTC 2022
On Tue, 8 Nov 2022 23:35:47 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Forgot a null check.
>
> src/hotspot/share/prims/jvmtiEnvBase.cpp line 540:
>
>> 538:
>> 539: jthread *
>> 540: JvmtiEnvBase::new_jthreadArray(int length, Handle *handles) {
>
> Shouldn't this method need to cast the return value to `jthread*`? And potentially shouldn't all the jobject's now be jthread's?
jthread and jthreadGroup are typedefs to jobject in JVMTI spec.
https://docs.oracle.com/en/java/javase/11/docs/specs/jvmti.html#jthread
But I updated the code to have the more specific types.
-------------
PR: https://git.openjdk.org/jdk/pull/11033
More information about the hotspot-dev
mailing list