<html>

  <head>

    <meta content="text/html; charset=ISO-8859-1"

      http-equiv="Content-Type">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    <div class="moz-cite-prefix"><br>

      Hi Kim,<br>

      <br>

      On 12/11/14 12:29 AM, Kim Barrett wrote:<br>

    </div>

    <blockquote

      cite="mid:13AAD5F2-90B8-4A49-A337-6DB1F5AA4742@oracle.com"

      type="cite">

      <pre wrap="">Note: I will need a sponsor for this change.

Please review this small change to remove an unused function.  This

function has had a "delete this function" comment since the dawn of

the mercurial age.

While I was in the vicinity, I made a couple of additional code

cleanups:

* removed a couple of unnecessary

  debug_only(public:)

    ...

  debug_only(private:)

member exposure blocks that were cluttering up the code.

* Removed some unnecessary upcasts of return values to match the

return type of the function being returned from.

CR:

<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8066822">https://bugs.openjdk.java.net/browse/JDK-8066822</a>

Webrev:

<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~kbarrett/8066822">http://cr.openjdk.java.net/~kbarrett/8066822</a>

Testing: JPRT

</pre>

    </blockquote>

    <br>

    Looks good to me. Thanks for cleaning this up!<br>

    <br>

    The

    <meta http-equiv="content-type" content="text/html;

      charset=ISO-8859-1">

    counters() method is only used as a private accessor. Do you want to

    move that to the private section as part of this cleanup?

    Alternatively it could be removed and the only use of it in

    PSMarkSweep::invoke_no_policy() could access the instance variable

    _counters directly.<br>

    <br>

    I can sponsor the fix.<br>

    <br>

    Thanks,<br>

    Bengt<br>

  </body>

</html>