On Tue, 8 Nov 2022 14:55:17 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

>> The JVM code took a ThreadGroup lock before poking into ThreadGroup fields.  
>> Call a method in the ThreadGroup to call the synchronized method instead.
>> Tested with tier 1-4.
>
> 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 564:

> 562: 
> 563:   for (int i=0; i<length; i++) {
> 564:     objArray[i] = JNIHandles::make_local(groups->obj_at(i));

Nit: Spaces are missed around '=' and '<' signs.

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

PR: https://git.openjdk.org/jdk/pull/11033

Reply via email to