review request (S): 8057623: extension class for argument handling
John Coomes
John.Coomes at oracle.com
Fri Sep 5 19:33:09 UTC 2014
Thomas Schatzl (thomas.schatzl at oracle.com) wrote:
> Hi,
>
> On Thu, 2014-09-04 at 18:03 -0700, John Coomes wrote:
> > Hi,
> >
> > Please review this change to add an extension class for argument
> > handling:
> >
> > http://cr.openjdk.java.net/~jcoomes/8u/8u40/8057623-arguments-ext/
> >
> > It's intended to allow easier customization of consistency checks and
> > ergonomic gc selection.
> >
>
> looks good except for the odd indentation of the
> check_gc_consistency_user() method.
>
> Should we now indent all methods by all possible? And is
> check_gc_consistency_ergo() performance critical so that it needs to be
> inlined?
Hi Thomas,
Thanks for reviewing it. I just lined up the decl of
check_gc_consistency_user() method with check_gc_consistency_ergo()
immediately below it. But there isn't consistency :-) in the
indentation, so I've changed it to just a single space.
check_gc_consistency_ergo() is not performance critical, but it is so
trivial inlining made sense (at least to me).
-John
More information about the hotspot-gc-dev
mailing list