RFR: 8340327: A common framework to support public key algorithms with standard parameter sets [v6]
Kevin Driver
kdriver at openjdk.org
Fri Oct 4 19:52:38 UTC 2024
On Fri, 4 Oct 2024 17:40:39 GMT, Weijun Wang <weijun at openjdk.org> wrote:
>> src/java.base/share/classes/sun/security/pkcs/NamedPKCS8Key.java line 55:
>>
>>> 53: /// Ctor from family name, parameter set name, raw key bytes.
>>> 54: /// Key bytes won't be cloned, caller must relinquish ownership
>>> 55: public NamedPKCS8Key(String fname, String pname, byte[] h) {
>>
>> This may be a "style" nit, but it might be better to have a name longer than the single-character `h`. I had to refer to the comments to know what this is rather than relying on the variable name to be sufficient.
>
> How about `rawKey`?
Sure. Anything you like that fits what it holds.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21167#discussion_r1788240541
More information about the security-dev
mailing list