cURL / Mailing Lists / curl-library / Single Mail


RE: retrieving negotiated ssl/tls version

From: Steve Holme <>
Date: Sun, 8 Mar 2015 20:00:57 +0000

On Fri, 6 Mar 2015, bch wrote:

> Is the inline patch I posted above acceptable, or should it be
> re-submitted as a collection of patches or something different ?

I'm personally a Windows man so I would like attachments if possible - trying to apply an inline git patch is a nightmare (just doesn't work due to line endings and Outlook / Hotmail messing about with the formatting!)

I know others here prefer inline so I would always recommend sending in both ;-)

From my brief review I would recommend the following naming as they are more in keepingwith the existing SSL informational items:



...and for consistency:

> + info->negotiated_ssl = -1L;


A couple of small nits:

* I would recommend removing such lines as "/* bch ref -- NEGOTIATED_SSL info here (?) */"
* Open braces after a function declaration should be on the following line not as per conditional code like you appear to have done with "set_ssl_version_long(SSL *ssl, struct connectdata *conn) {"
* I would also recommend that the return type be on the beginning rather than on a separate line - I appreciate we have a mix of this at present :(

Kind Regards


List admin:
Received on 2015-03-08