On Fri, 8 Nov 2024 21:45:29 GMT, Sean Mullan <mul...@openjdk.org> wrote:

>> Artur Barashev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Add new test pattern
>
> src/java.base/share/classes/sun/security/util/AbstractAlgorithmConstraints.java
>  line 48:
> 
>> 46:     protected final AlgorithmDecomposer decomposer;
>> 47:     private static final Map<String, Pattern> patternCache =
>> 48:             new ConcurrentHashMap<>();
> 
> I don't know if this would matter much, but you could probably specify a 
> smaller initial size here, like 4 or 8.

Good point, it's very unlikely we'll ever use more than 4 patterns. The default 
size is 16.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/21841#discussion_r1835165560

Reply via email to