(was Re: Code Review Request: TLS 1.3 Implementation)

Xuelei Fan at
Mon Jun 11 14:54:11 UTC 2018

On 6/11/2018 7:36 AM, Weijun Wang wrote:
>> On Jun 11, 2018, at 10:32 PM, Xuelei Fan < at> wrote:
>>> Another thing in
>>>    803     static CipherSuite nameOf(String ciperSuiteName) {
>>> Why isn't this method named valueOf()?
>> There is a default builtin valueOf(String) method for enum type.
> Ah, I see. But given the return type is a value, maybe simply of()?
> nameOf() sounds like it would return a name.
I hesitated among nameOf(), ofName() and of() for a while.  Good if you 
have a preference.  As there are a few more nameOf() methods defined in 
other classes.  I will find a chance to update them all.


More information about the security-dev mailing list