Erik,

Looks good!

Bengt


On 2/21/13 2:06 PM, Erik Helin wrote:
All,

based on some internal feedback, I've updated the change. The removal of
PermStat.java could be done in the first change, reducing the size of
this change.

Webrev:
http://cr.openjdk.java.net/~ehelin/8008541/webrev.01/

The first and second change have also been updated.

Thanks,
Erik

On 02/20/2013 05:02 PM, Erik Helin wrote:
Hi all,

this change is the third 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 third step, which removes the support for the
option -permstats in the SA agent.

Step 1 and 2 are also out for review.

Webrev:
http://cr.openjdk.java.net/~ehelin/8008541/webrev.00/

Bug:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8008541

Thanks,
Erik


Reply via email to