On Thu, 21 Aug 2025 20:38:22 GMT, Weijun Wang <[email protected]> wrote:
>> Valerie Peng has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Revised the wording again to avoid referring to NoSuchPaddingException
>> in NoSuchAlgorithmException
>
> src/java.base/share/classes/javax/crypto/Cipher.java line 609:
>
>> 607: * @throws NoSuchPaddingException if a {@code CipherSpi}
>> implementation
>> 608: * from the specified {@code provider} is found but it does
>> not
>> 609: * support the padding scheme
>
> The wording is correct now. Small nit: the NSAE one uses "but does not" and
> the NSPE one uses "but it does not". You might want to make them the same.
Yes, will do.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26489#discussion_r2292091118
