RFR(M): 8245226: Clean-up FlagSetting and remove misuse.

David Holmes david.holmes at oracle.com
Thu Jul 9 09:09:13 UTC 2020


Hi Patric,

On 9/07/2020 5:17 am, Patric Hedlin wrote:
> Dear all,
> 
> I would like to ask for help to review the following change/update:
> 
> Issue:  https://bugs.openjdk.java.net/browse/JDK-8245226
> Webrev: http://cr.openjdk.java.net/~phedlin/tr8245226/
> 
> 
> FlagSetting is sometimes used as a general mechanism for local 
> save/modify and restore. This is not the intention (it should work as a 
> red-flag when modifying debug options). Instead, use a general mechanism 
> in these cases (introduced in here), and preserve FlagSetting for its 
> intended purpose (including clean-up).

Sorry but I'm not seeing why FlagSetting can't be used as a general 
mechanism even though it is described as being for debug flags. 
Introducing a similar mechanism seems redundant to me.

Thanks,
David

> 
> Testing: hs-tier1-3
> 
> 
> Best regards,
> Patric


More information about the hotspot-dev mailing list