On Fri, 4 Mar 2022 07:24:51 GMT, Yi Yang <yy...@openjdk.org> wrote:

>> You should change it to `%4d`. Otherwise, when the numbers are changed in 
>> the future (e.g., to 3 or 4 digits) they will be misaligned:
>> 
>> 
>> KlassAddr           Size  State                 Flags    ClassName  
>> 0x0000000800df8400  62    fully_initialized     W        
>> java.lang.invoke.LambdaForm$DMH/0x0000000800df8400  
>> 0x0000000800df8000  123   fully_initialized     W        
>> java.lang.invoke.LambdaForm$DMH/0x0000000800df8000  
>> 0x0000000800de4400  4567  fully_initialized     W        
>> java.lang.invoke.LambdaForm$DMH/0x0000000800de4400
>
>> You should change it to `%4d`. Otherwise, when the numbers are changed in 
>> the future (e.g., to 3 or 4 digits) they will be misaligned:
>> 
>> ```
>> KlassAddr           Size  State                 Flags    ClassName  
>> 0x0000000800df8400  62    fully_initialized     W        
>> java.lang.invoke.LambdaForm$DMH/0x0000000800df8400  
>> 0x0000000800df8000  123   fully_initialized     W        
>> java.lang.invoke.LambdaForm$DMH/0x0000000800df8000  
>> 0x0000000800de4400  4567  fully_initialized     W        
>> java.lang.invoke.LambdaForm$DMH/0x0000000800de4400  
>> ```
> 
> This format looks pretty good to me, they are all aligned to left. If you 
> still think it's more proper to have a format like this:
> 
> KlassAddr           Size  State                 Flags    ClassName  
> 0x0000000800df8400  62    fully_initialized     W        
> java.lang.invoke.LambdaForm$DMH/0x0000000800df8400  
> 0x0000000800df8000  123   fully_initialized     W        
> java.lang.invoke.LambdaForm$DMH/0x0000000800df8000  
> 0x0000000800de4400  4567  fully_initialized     W        
> java.lang.invoke.LambdaForm$DMH/0x0000000800de4400  
> 
> Then I'm glad to do so ;)

Numbers should be aligned to the right. The following is what I want:


  62
 123
4567

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

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

Reply via email to