Re: RFR: 8275775: Add jcmd VM.classes to print details of all classes [v6]

[Yi Yang]

On Fri, 11 Feb 2022 06:49:25 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> src/hotspot/share/oops/instanceKlass.cpp line 2081:
>> 
>>> 2079:   _st->print(INTPTR_FORMAT "  ", p2i(k));
>>> 2080:   // klass size
>>> 2081:   _st->print("%-4d  ", k->size());
>> 
>> Should be `%4d` so that the numbers are aligned correctly.
>
> This issue seem still outstanding.

Current:

$./jcmd 83908 VM.classes|head -10
83908:
KlassAddr           Size  State                 Flags    ClassName  
0x0000000800df8400  62    fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800df8400  
0x0000000800df8000  62    fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800df8000  
0x0000000800de4400  62    fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800de4400  
0x0000000800de4000  62    fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800de4000  
0x0000000800dc8800  62    fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800dc8800  
0x0000000800dc8400  62    fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800dc8400  
0x0000000800dc8000  62    fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800dc8000  
0x0000000800db9800  62    fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800db9800 

After using "%4d":

$./jcmd 75481 VM.classes|head
75481:
KlassAddr           Size  State                 Flags    ClassName  
0x0000000800df8400    62  fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800df8400  
0x0000000800df8000    62  fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800df8000  
0x0000000800de4400    62  fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800de4400  
0x0000000800de4000    62  fully_initialized     W        
java.lang.invoke.LambdaForm$DMH/0x0000000800de4000 

So we do not need to change this.

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

PR: https://git.openjdk.java.net/jdk/pull/7105