RFR: 8142390: Move ScanRSClosure to header file

Jon Masamitsu jon.masamitsu at oracle.com
Tue Nov 10 18:50:36 UTC 2015


Change looks good.

Reviewed.

Would you consider in this patch changing

185 size_t cards_looked_up() { return _cards;} _cards -> 
_cards_looked_up or changing cards_looked_up() -> cards()

Your call.  I don't need a new webrev.

Jon


On 11/10/2015 02:55 AM, Erik Helin wrote:
> Hi all,
>
> this small patches move the declaration of ScanRSClosure from
> g1RemSet.cpp to g1RemSet.hpp.
>
> Enhancement:
> https://bugs.openjdk.java.net/browse/JDK-8142390
>
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8142390/webrev.00/
>
> Testing:
> JPRT
>
> Thanks,
> Erik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/hotspot-gc-dev/attachments/20151110/d9f6a88f/attachment.htm>


More information about the hotspot-gc-dev mailing list