RFR: 8264644: More complete ClassLoaderData for cld->print()

[Coleen Phillimore]

On Fri, 2 Apr 2021 09:42:01 GMT, Yi Yang <yyang at openjdk.org> wrote:

> Trivial chanage to make debugging happy, now cld->print() would be:
> 
> ClassLoaderData(0x00007ff17432b670)
>  - name                'platform'
>  - holder              WeakHandle: 0x00000007fef56678
>  - class loader        0x7ff17432b828
>  - metaspace           0x7ff17468a0b0
>  - unloading           false
>  - class mirror holder false
>  - modified oops       true
>  - keep alive          0
>  - claim               strong
>  - handles             43
>  - dependency count    0
>  - klasses             {java.sql.SQLFeatureNotSupportedException,java.sql.SQLNonTransientException,java.sql.SQLException,sun.util.resources.provider.NonBaseLocaleDataMetaInfo,org.ietf.jgss.GSSException,javax.sql.DataSource,java.sql.Wrapper,javax.sql.CommonDataSource,java.sql.Time,java.sql.Date,java.sql.Timestamp,sun.util.resources.cldr.provider.CLDRLocaleDataMetaInfo, }
>  - packages            0x7ff17432bc20
>  - module              0x7ff17432c520
>  - unnamed module      0x7ff17432c3d0
>  - dictionary          0x7ff17432c470
>  - deallocate list     0x7ff0a4023bd0

This looks nice. If you use -XX:+PrintSystemDictionaryAtExit with this change, is it pages and pages long?  Or does it look ok?  Thank you for doing this!

src/hotspot/share/classfile/classLoaderData.cpp line 950:

> 948:   out->print   (" - holder              ");
> 949:   _holder.print_on(out);
> 950:   out->print_cr("");

indent 2 here.

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

Marked as reviewed by coleenp (Reviewer).

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