On Thu, 22 Dec 2022 19:29:18 GMT, Matthew Donovan <d...@openjdk.org> wrote:

>> Updated to use specific ciphersuite and TLS protocol.
>> Updated tests based on changes to SSLEngine implementation
>
> Matthew Donovan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   removed unused variables, made fields final, changed to RuntimeException 
> everywhere

test/jdk/javax/net/ssl/SSLEngine/Basics.java line 51:

> 49:     private static final String keyStoreFile = "keystore";
> 50:     private static final String trustStoreFile = "truststore";
> 51:     private static final String passwd = "passphrase";

passwd variable defined here is unused. Also can you please capitalize all the 
constants.

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

PR: https://git.openjdk.org/jdk/pull/11760

Reply via email to