<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<font face="Times New Roman, Times, serif">Change looks good.<br>
<br>
Reviewed.<br>
<br>
Would you consider in this patch changing<br>
</font><br>
<pre><span class="new"> 185 size_t cards_looked_up() { return _cards;}
_cards -> _cards_looked_up
or changing
cards_looked_up() -> cards()
</span></pre>
<font face="Times New Roman, Times, serif">Your call. I don't need
a new webrev.<br>
<br>
Jon<br>
<br>
</font><br>
<div class="moz-cite-prefix">On 11/10/2015 02:55 AM, Erik Helin
wrote:<br>
</div>
<blockquote cite="mid:20151110105521.GE12500@ehelin.jrpg.bea.com"
type="cite">
<pre wrap="">Hi all,
this small patches move the declaration of ScanRSClosure from
g1RemSet.cpp to g1RemSet.hpp.
Enhancement:
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8142390">https://bugs.openjdk.java.net/browse/JDK-8142390</a>
Webrev:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~ehelin/8142390/webrev.00/">http://cr.openjdk.java.net/~ehelin/8142390/webrev.00/</a>
Testing:
JPRT
Thanks,
Erik
</pre>
</blockquote>
<br>
</body>
</html>