RFR: 8331008: Implement JEP 478: Key Derivation Function API (Preview) [v36]

Kevin Driver kdriver at openjdk.org
Mon Sep 16 16:45:31 UTC 2024


On Sat, 14 Sep 2024 22:27:36 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> Kevin Driver has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   refine wording on DPS getInstance with params exception
>
> src/java.base/share/classes/javax/crypto/KDF.java line 207:
> 
>> 205:      * Returns a {@code KDF} object that implements the specified algorithm.
>> 206:      *
>> 207:      * @see <a href="#DelayedProviderSelection">Delayed Provider Selection</a>
> 
> Put `@see` at the end. Put `@implNote` before `@param`. This is the order they are rendered in HTML javadoc.

Nit. I don't personally see the merit in this comment, since the renderer knows what to do, but I've done as you suggest.

> src/java.base/share/classes/javax/crypto/KDF.java line 266:
> 
>> 264:      *     list
>> 265:      * @throws NullPointerException
>> 266:      *     if the {@code algorithm} or {@code provider} is {@code null}
> 
> Remove the `the` word. Same below.

This was here to make it read more like proper English, but I'll remove as you suggest.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1761488389
PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1761487319


More information about the security-dev mailing list