RFR: 8331008: Implement JEP 478: Key Derivation Function API (Preview) [v36]
Kevin Driver
kdriver at openjdk.org
Mon Sep 16 19:11:09 UTC 2024
On Mon, 16 Sep 2024 16:42:15 GMT, Kevin Driver <kdriver at openjdk.org> wrote:
>> 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.
See: https://github.com/openjdk/jdk/pull/20301/commits/ea5ba798819568ca86d639ba6c7a9489d4e66c95.
>> 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.
See: https://github.com/openjdk/jdk/pull/20301/commits/ea5ba798819568ca86d639ba6c7a9489d4e66c95.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1761711488
PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1761711767
More information about the security-dev
mailing list