On Wed, 15 May 2024 19:59:59 GMT, Kevin Driver <kdri...@openjdk.org> wrote:

>> Introduce an API for Key Derivation Functions (KDFs), which are 
>> cryptographic algorithms for deriving additional keys from a secret key and 
>> other data. See [JEP 478](https://openjdk.org/jeps/478).
>
> Kevin Driver has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   javadoc formatting

src/java.base/share/classes/javax/crypto/KDF.java line 202:

> 200:      * @param provider
> 201:      *     the provider to use for this key derivation; if null, this 
> method is
> 202:      *     equivalent to {@code getInstance(String)}

It might be better to disallow null here. If the code doesn't have a provider 
name then it would be clearer to use the 1-arg getInstance method. That would 
also help catch bugs where the provider name is null due to some bug.

src/java.base/share/classes/javax/crypto/KDF.java line 228:

> 226: 
> 227:     /**
> 228:      * Returns a {code KDF} object that implements the specified 
> algorithm from

Missing the `@` here.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18924#discussion_r1602698687
PR Review Comment: https://git.openjdk.org/jdk/pull/18924#discussion_r1602696489

Reply via email to