RFR: 8224589: Improve startup behavior of SecurityProperties

Sean Mullan sean.mullan at oracle.com
Wed May 22 12:12:02 UTC 2019


Looks ok, although with the changeset you pushed, there is no desugaring 
so you should probably update the bug description to reflect that.

(I probably would have kept the desugaring in the SM case, but probably 
not a big deal either way)

--Sean

On 5/22/19 7:06 AM, Claes Redestad wrote:
> True, I'll skip the desugaring of the else branch, keeps it cleaner.
> 
> On 2019-05-22 12:50, Alan Bateman wrote:
>> On 22/05/2019 11:34, Claes Redestad wrote:
>>> :
>>>
>>> You mean the SM case? Not as much as the no-SM case, no, which can
>>> be helped a lot in the smallest affected tests.
>> Yes, I don't think the SM case should be performance/startup concern 
>> here. Up to you, I don't think it matters but I'm sure someone will be 
>> refactor it again.
>>
>> -Alan



More information about the security-dev mailing list