On Thu, 24 Apr 2025 20:12:50 GMT, Mark Reinhold <m...@openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with two 
>> additional commits since the last revision:
>> 
>>  - javadoc updates
>>  - code review comments
>
> src/java.base/share/classes/java/security/PEMEncoder.java line 74:
> 
>> 72:  *
>> 73:  * <p>{@code String} values returned by this class use character set
>> 74:  * {@link java.nio.charset.StandardCharsets#ISO_8859_1 ISO-8859-1}.
> 
> As above, did you mean to write something like, "Byte arrays returned by 
> methods in this class represent characters encoded using the ISO-8859-1 
> charset" ?

Yes, only `byte[]` has an encoding. BTW, please be specific this is about the 
`encode` method.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17543#discussion_r2076276493

Reply via email to