RFR (S): 8219369: Add named constants for iterating ExtRootScan phases

Thomas Schatzl thomas.schatzl at oracle.com
Thu Feb 28 09:28:15 UTC 2019


Hi Kim, Sangheon,

On Wed, 2019-02-27 at 12:23 -0500, Kim Barrett wrote:
> > On Feb 27, 2019, at 4:37 AM, Thomas Schatzl <
> > thomas.schatzl at oracle.com> wrote:
> > 
> > Hi,
> > 
[...]
> > > I have something of a preference against cluttering enums with
> > > such pseudo-enumerators, and instead prefer class constants.  But
> > > I'll leave it up to you.
> > > 
> > 
> > Fixed in
> > http://cr.openjdk.java.net/~tschatzl/8219369/webrev.0_to_1/ (diff)
> > http://cr.openjdk.java.net/~tschatzl/8219369/webrev.1/ (full)
> > 
> > Yeah, looks better. Thanks for making me look at this option.
> > 
> Looks good.
> 

On Wed, 2019-02-27 at 11:14 -0800, sangheon.kim at oracle.com wrote:
> Hi Thomas,
> 
> On 2/27/19 1:37 AM, Thomas Schatzl wrote:
> > Hi,
[...] 
> > Yeah, looks better. Thanks for making me look at this option.
> 
> Still looks good.
> 
> Thanks,
> Sangheon
> 

  thanks for your reviews.

Thomas




More information about the hotspot-gc-dev mailing list