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