On Fri, 23 Jul 2021 17:05:58 GMT, Xue-Lei Andrew Fan <[email protected]> wrote:

>> Clive Verghese has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Remove for loop from validValuesOf
>
> src/java.base/share/classes/sun/security/ssl/CipherSuite.java line 972:
> 
>> 970:             boolean found = false;
>> 971:             CipherSuite cs;
>> 972:             if ((cs = cipherSuiteNames.get(name)) != null && 
>> !cs.supportedProtocols.isEmpty()) {
> 
> Nice update!  I appreciate if you could avoid lines longer than 80 
> characters, since they’re not handled well by many terminals and tools (See 
> [Java Code 
> Conventions](https://www.oracle.com/java/technologies/javase/codeconventions-indentation.html)).

Thank you @XueleiFan, Updated the length of the line.

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

PR: https://git.openjdk.java.net/jdk/pull/4783

Reply via email to