On Wed, 20 Oct 2021 13:34:44 GMT, Sean Mullan <mul...@openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/util/AlgorithmDecomposer.java line 
>> 106:
>> 
>>> 104:         // "SHA-256" and "SHA256" to make the right constraint 
>>> checking.
>>> 105: 
>>> 106:         for (Map.Entry<String, String> e : 
>>> DECOMPOSED_DIGEST_NAMES.entrySet()) {
>> 
>> If you're going to change this code, you can save me a PR if you surround 
>> this by "if (algorithm.contains("SHA") {  ...  }"
>> Its a perf change to eliminate the unnecessary map lookups when SHA isn't in 
>> the algorithm string
>
> That's a fine suggestion, although I'll note that your suggested perf 
> improvement also applies to the previous code which did not check the 
> algorithm parameter first to see if it contained `SHA`.
> Also, another small perf imp: I realized below that in the loop, if the first 
> `if` block gets executed, then the 2nd `if` block will always be false, so I 
> changed it to an if/else.

Yes, I was about to submit a PR to change the previous code, since you changed 
this code it makes more sense to ask you to do it at the same time.

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

PR: https://git.openjdk.java.net/jdk/pull/5928

Reply via email to