Re: RFR 8072394: Performance improvement for X.509 certificate parsing

2015-02-13 Thread Florian Weimer
On 02/13/2015 08:43 PM, Sean Mullan wrote: > This fix looks fine, but I am trying to remember why a Set/LinkedHashSet > was used in the first place, it seem like an List/ArrayList would have > been more suitable. Even though PolicyInformation is an internal class, > it's probably better to not chan

Re: RFR 8072394: Performance improvement for X.509 certificate parsing

2015-02-13 Thread Sean Mullan
This fix looks fine, but I am trying to remember why a Set/LinkedHashSet was used in the first place, it seem like an List/ArrayList would have been more suitable. Even though PolicyInformation is an internal class, it's probably better to not change that detail at this point unless we have a b