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 file PermStat.java can be removed already in this first change. This
means that 'hg mv' can be used, which provides a better history.

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

The second and third change have also been updated.

Thanks,
Erik


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

this change is the first 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 first step. Once this change has found its way
into the tl forest, then step two can be applied. Once the result of
step two is back in the hotspot-gc forest, then step three can be done.

Step 2 and 3 are also out for review.

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

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

Testing:
JPRT

Thanks,
Erik


Reply via email to