On Wed, 29 Apr 2026 01:49:06 GMT, Artur Barashev <[email protected]> wrote:
>> Volkan Yazici has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Big facelift
>
> src/java.base/share/classes/sun/security/ssl/X509KeyManagerCertChecking.java
> line 394:
>
>> 392: try {
>> 393: @SuppressWarnings("deprecation")
>> 394: var sni = new
>> SNIHostName(serverName.getEncoded());
>
> Why do we need `sni` temporary variable here? Same goes for
> `X509TrustManagerImpl.java`
Annotations are not allowed on assignment, but declaration.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30747#discussion_r3166530913
