On Thu, 18 Sep 2025 20:20:25 GMT, Weijun Wang <[email protected]> wrote:
>> Valerie Peng has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> updated the checkTransformation() method w/ specific provider and
>> exception per Sean's suggestion
>
> src/java.base/share/classes/javax/crypto/Cipher.java line 507:
>
>> 505: *
>> 506: * @throws NoSuchAlgorithmException if {@code transformation}
>> 507: * is {@code null}, empty, in an invalid format,
>
> Not related to this change, but do we need an "or" before "in an valid
> format"?
Sure, will use this. Thanks for the suggestions @wangweij @seanjmullan
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26489#discussion_r2369597509
