On Thu, 28 Apr 2022 23:08:17 GMT, Valerie Peng <valer...@openjdk.org> wrote:

>> So, "the underlying signature implementation supports returning the 
>> parameters as {@code AlgorithmParameters}" is quite necessary. Xuelei's 
>> suggestion is quite good, just change the last "and" to "or".
>
> I assume you were suggesting this? `"The returned parameters may be the same 
> that were used to initialize this signature, or may contain additional 
> default or random parameter values used by the underlying signature 
> implementation. {https://github.com/code null} is returned if the required 
> parameters were not supplied and the underlying signature implementation 
> cannot generate the parameter values."`
> But the "the underlying signature implementation supports returning the 
> parameters as {https://github.com/code AlgorithmParameters}" is necessary. 
> Strictly speaking, this is somewhat different than the "cannot generate 
> parameter values" though. Perhaps we should go a bit broader for the last 
> sentence regarding null return value?

I suggest the last sentence to be "null is returned if the required parameters 
were not supplied **or** the underlying signature implementation cannot 
generate the parameter values." I used "or" because for EdDSA parameters are 
supplied but the impl cannot generate parameter values.

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

PR: https://git.openjdk.java.net/jdk/pull/8396

Reply via email to