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