RFR(S): 7111795: G1: Various cleanups identified during walk through of changes for 6484965
John Cuthbertson
john.cuthbertson at oracle.com
Fri Nov 18 17:57:28 UTC 2011
Hi Tony,
Thanks for the review.
On 11/18/11 09:28, Tony Printezis wrote:
> 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 }
>
Sure - no problem.
>
> 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.
I do tend to favor the former. I don't know why. But I'm not absolutely
tied to it. I'll revert back any I've changed.
Thanks,
JohnC
More information about the hotspot-gc-dev
mailing list