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

Patric Hedlin patric.hedlin at oracle.com
Thu Jul 9 08:16:09 UTC 2020


Thanks for reviewing Nils.

/Patric

On 2020-07-08 22:10, Nils Eliasson wrote:
> Hi Patric,
>
> Nice clean-up!
>
> Looks good.
>
> Best regards,
> Nils
>
> On 2020-07-08 21:17, 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).
>>
>>
>> Testing: hs-tier1-3
>>
>>
>> Best regards,
>> Patric
>



More information about the hotspot-dev mailing list