RFR: 8327246: Add a jcmd diagnostic command to list the jar files loaded by a process [v2]

David Holmes dholmes at openjdk.org
Tue Jan 13 05:36:40 UTC 2026


On Tue, 6 Jan 2026 23:41:52 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Larry Cable has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   JDK-8327246: updated copyright year. fixed Capitialization nit and restructured use of InstanceKlass local as per comments
>
> src/hotspot/share/oops/instanceKlass.cpp line 2359:
> 
>> 2357:   // initialization state
>> 2358: 
>> 2359:   InstanceKlass *ik = k->is_instance_klass() ? InstanceKlass::cast(k) : nullptr;
> 
> Suggestion:
> 
>   InstanceKlass* ik = k->is_instance_klass() ? InstanceKlass::cast(k) : nullptr;

Hotspot-style places the asterisk adjacent to the type not the variable name.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29048#discussion_r2684891342


More information about the hotspot-dev mailing list