On Fri, 31 Jan 2025 01:45:47 GMT, Bradford Wetmore <[email protected]> wrote:
> Update and clarify the sample code.
>
> Docs only, no additional testing other than verifying javadoc is correctly
> output.
src/java.base/share/classes/javax/net/ssl/SSLParameters.java line 668:
> 666: * String encodedHukUnI =
> 667: * new String(bytes, StandardCharsets.ISO_8859_1);
> 668: *
Maybe a better comment here would be:
* // Preserve bytes by using ISO_8859_1 encoding
because we're not converting anything into anything.
As far as I understand what you are trying to do is to shoe-horn a byte array
into a string, so that you can retrieve the original array using
`String.getBytes(StandardCharsets.ISO_8859_1)`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23379#discussion_r1941585321