On Fri, 14 Aug 2026 21:09:37 GMT, Chris Plummer <[email protected]> wrote:

>> Leonid Mesnik has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   updated after feedback in PR
>
> src/hotspot/share/services/threadService.cpp line 1502:
> 
>> 1500: 
>> 1501:   // Locks
>> 1502:   refArrayHandle locks;
> 
> It looks like "refArrayHandle locks" can also be moved into the "if" block 
> below.

the locks is used in 
`1531:  jdk_internal_vm_ThreadSnapshot::set_locks(snapshot(), locks());`
to initialize "locks" field.
The section init whole class, so let don't change this.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32363#discussion_r3787222178

Reply via email to