Review request: Add new jstat -classload option

[Alan Bateman]

Mandy Chung wrote:
> Fixed 6959965: jstat: Add new -classload option to print class loading 
> statistics
>
> I added several new perf counters for collecting class loading related 
> metrics some time ago (6857194).  This fix updates jstat to print 
> these new statistics (I finally got time to do this).
>
> Webrev:
>    http://cr.openjdk.java.net/~mchung/6959965/webrev.00/
>
> I added this -classload as an internal option for now.
>
> Mandy
I went through the webrev.

Do you plan to make this a "standard" option at some point? If so then 
maybe it would be best to do that now and save introducing a second file 
of options. Related to this is terminology and I wonder if "unsupported" 
would be better than "internal" given that the the jstat.showUnsupported 
property needs to be set to enable this option.

Minor comment is that the "findClass" and "defineClass" headers have the 
first character in lower case whereas all other headers (I think) uses 
capital. Another minor comment is that OptionFinder.optionsSources and 
OptionLister.sources can be final.

Do you plan to add a test to  test/sun/tools/jstate for this option?

-Alan