RFR (XS): 8005116: NPG: Rename -permstat option for jmap in jdk8 to -classstats

[Jon Masamitsu]

Erik,

Early on the flag was -classstats and I understand the
change to -clstats ("sss" hurts the eyes) but it is described
as "class loader" statistics now.  How did we get to that
description?  Meaning "class loader" statistics instead of
"class" statistics?

Jon

On 02/21/13 05:06, Erik Helin wrote:
> All,
>
> I've updated the change based on internal feedback and feedback from
> Staffan Larsen.
>
> The bug has been renamed to use -clstats and the help text now uses
> "class loader" instead of "classloader".
>
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8005116/webrev.01/
>
> The first and third change have also been updated.
>
> Thanks,
> Erik
>
> On 02/20/2013 05:02 PM, Erik Helin wrote:
>> Hi all,
>>
>> this change is the second of three changes related to renaming the jmap
>> option permstat to clstats. The change has to be done as three separate
>> changes in order to not break any repositories:
>> 1. Add support for the -clstats option to the SA agent
>> 2. Change the tool jmap to use -clstats instead of -permstat
>> 3. Remove the option -permstat from the SA agent
>>
>> This CR represents the second step, which updates the jmap tool.
>>
>> Step 1 and 3 are also out for review.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~ehelin/8005116/webrev.00/
>>
>> Bug:
>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8005116
>>
>> Thanks,
>> Erik
>