On Thu, 15 May 2025 00:27:31 GMT, Bradford Wetmore <wetm...@openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/ssl/SSLSessionImpl.java line 1727:
>> 
>>> 1725: 
>>> 1726:             // context length must fit in 2 unsigned bytes.
>>> 1727:             if ((context != null) && context.length >= 65536) {
>> 
>> Nit: `context.length > 0xFFFF` would be more readable IMO
>
> Sure.

Also, the parentheses are not needed around `context != null`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24976#discussion_r2091102162

Reply via email to