RFR: 8275775: Add jcmd VM.classes to print details of all classes [v2]
Yi Yang
yyang at openjdk.java.net
Fri Feb 11 03:35:10 UTC 2022
On Tue, 18 Jan 2022 02:59:11 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Yi Yang has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
>>
>> 8275775 Add VM.classes to print details of all classes
>
> src/hotspot/share/oops/instanceKlass.cpp line 2069:
>
>> 2067: ResourceMark rm;
>> 2068: _st->print("%-18s", "KlassAddr");
>> 2069: _st->print(" ");
>
> Can't you just print the two spaces in the previous line:
>
> _st->print("%-18s ", "KlassAddr");
>
> and save all the additional print calls. This applies throughout where you have " ".
@dholmes-ora David, Can you please take a look at the latest version? I've addressed all problems you pointed out.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7105
More information about the serviceability-dev
mailing list