On Sun, 8 Sep 2024 16:52:17 GMT, Alan Bateman <[email protected]> wrote:
>> Kevin Driver has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> updated comments around locking mechanism
>
> src/java.base/share/classes/javax/crypto/KDF.java line 413:
>
>> 411: * @throws InvalidAlgorithmParameterException
>> 412: * if no {@code Provider} supports a {@code KDFSpi}
>> implementation for
>> 413: * the specified algorithm and parameters
>
> Can you double check the description of InvalidAlgorithmParameterException?
> This factory method has a provider parameter so I assume it's that _the_
> provider doesn't support the ...
Right, I think it should be reworded something like:
"if the specified provider does not support a `KDFSpi` implementation for the
specified algorithm and parameters"
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1752184001
