RFR: 8224589: Improve startup behavior of SecurityProperties

Claes Redestad claes.redestad at oracle.com
Wed May 22 12:46:28 UTC 2019


Thanks Sean,

updated description.

When we run with a SM installed, we'll initialize a few lambdas early on
already, so gain from desugaring this path was not really measurable.

/Claes

On 2019-05-22 14:12, Sean Mullan wrote:
> 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