RFR(S): 7111795: G1: Various cleanups identified during walk through of changes for 6484965
Tony Printezis
tony.printezis at oracle.com
Fri Nov 18 17:28:34 UTC 2011
John,
Thanks for doing the cleanup. It's fine, one minor issue:
1989 G1CMParKeepAliveAndDrainClosure(ConcurrentMark* cm, CMTask* task) :
1990 _cm(cm), _task(task),
1991 _ref_counter_limit(G1RefProcDrainInterval)
1992 {
1993 assert(_ref_counter_limit> 0, "sanity");
1994 _ref_counter = _ref_counter_limit;
1995 }
maybe:
1991 _ref_counter_limit(G1RefProcDrainInterval) {
1992 assert(_ref_counter_limit> 0, "sanity");
1993 _ref_counter = _ref_counter_limit;
1994 }
And one stylistic question: do you prefer this style for an empty
constructor:
class A {
int _a, _b, _c;
A() : _a(0), _b(0), _c(0)
{ }
};
over this?
class A {
int _a, _b, _c;
A() : _a(0), _b(0), _c(0) { }
};
I would recommend the latter myself, given that it saves an extra line.
Tony
On 11/15/2011 12:45 PM, John Cuthbertson wrote:
> Hi Everyone,
>
> Can I have a couple of volunteers look over these changes? The webrev
> can be found at: http://cr.openjdk.java.net/~johnc/7111795/webrev.0/
>
> Summary:
> During a code walkthrough of the original set of changes for 6484965
> ("piggy-back liveness accounting on marking") a few loosely related
> cleanups were identified. I have decided to send these out as a
> separate set of changes.
>
> Testing: the GC test suite and KitchenSink on Solaris and Linux with a
> low marking threshold (10%).
>
> Thanks,
>
> JohnC
More information about the hotspot-gc-dev
mailing list