On Wed, 3 Jun 2026 11:18:24 GMT, Alan Bateman <[email protected]> wrote:

>> Anthony Scarpino has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   review comments
>
> src/java.base/share/classes/java/security/PEM.java line 212:
> 
>> 210:      *
>> 211:      * @return a newly-allocated byte array containing the decoded 
>> content
>> 212:      * @throws IllegalArgumentException if decoding fails
> 
> The decode method has no arguments so it's a bit strange to throw 
> IllegalArgumentException. Has this been discussed already?

`EncryptedPrivateKeyInfo.getEncoded()` throws an IOException, so I needed a way 
to communicate that.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29640#discussion_r3350457066

Reply via email to