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

Kim Barrett kim.barrett at oracle.com
Tue Feb 26 22:49:08 UTC 2019


> On Feb 20, 2019, at 5:15 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
> 
> Hi all,
> 
>  can I have reviews for this change that has been suggested as part of
> another review to improve readability of the ExtRootScan sub-phase
> printing?
> 
> In particular, it introduces a few new named constants in the
> GCParPhases enum that indicate start and end of the ext root scan sub-
> phases. These are then used in the iteration.
> 
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8219369
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8219369/webrev/
> testing:
> hs-tier1,hs-tier2 (includes log output testing for completeness)
> 
> Thanks,
>  Thomas

Looks good, but...

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.





More information about the hotspot-gc-dev mailing list