On Thu, 15 Aug 2024 21:06:13 GMT, Kevin Driver <kdri...@openjdk.org> wrote:

>> This method can be made "private" since there is no need for sub-class to 
>> override it?
>
>> Use the name `inputKeyMaterial` instead of `inputKey` for consistency? Seems 
>> easier to use `byte[]` for its type since all you need is just the bytes.
> 
> We can consider a rename of this parameter. See related comments for why we 
> would use SecretKey instead of byte[].

Well, I don't share your comments/reason on why SecretKey is needed.

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

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

Reply via email to