RFR (S/M): 8149127: Rename g1/concurrentMarkThread.* to g1/g1ConcurrentMarkThread.*

Thomas Schatzl thomas.schatzl at oracle.com
Mon Nov 6 07:47:01 UTC 2017


Hi Sangheon, Stefan,

On Fri, 2017-11-03 at 16:20 -0700, sangheon.kim wrote:
> Hi Thomas,
> 
> On 11/03/2017 06:47 AM, Thomas Schatzl wrote:
> > 
[...]
> > > Looks really good. Just one small thing, could you rename the
> > > getter in G1CollectedHeap:
> > > 1392   G1ConcurrentRefine* concurrent_g1_refine() const { return
> > > _cg1r; }
> > > 
> > > I think it would make sense to rename it to
> > > g1_concurrent_refine().
> > > 
> > 
> >    renamed to concurrent_refine() - also renamed the few "cg1r"
> > names
> > floating around for consistency.
> > 
> > http://cr.openjdk.java.net/~tschatzl/8149127/webrev.1/ (full)
> > http://cr.openjdk.java.net/~tschatzl/8149127/webrev.b.1 (which also
> > represents the 0_to_1 patch)
> 
> Looks good.
> Nice cleanup!

  thanks for your reviews.

Thomas




More information about the hotspot-gc-dev mailing list