]> granicus.if.org Git - curl/commit
vtls: Fixed compilation warning and an ignored return code
authorSteve Holme <steve_holme@hotmail.com>
Sun, 28 Dec 2014 17:21:02 +0000 (17:21 +0000)
committerSteve Holme <steve_holme@hotmail.com>
Sun, 28 Dec 2014 17:33:01 +0000 (17:33 +0000)
commit037cd0d99159cd39b8e1217ad9a9362e7ab0b746
tree0dc5df221687ea09e93a0f8cef5be29ddade4a3a
parent6d79722d7898122d7a91a931e03612cb2dab85b1
vtls: Fixed compilation warning and an ignored return code

curl_schannel.h:123: warning: right-hand operand of comma expression
                     has no effect

Some instances of the curlssl_close_all() function were declared with a
void return type whilst others as int. The schannel version returned
CURLE_NOT_BUILT_IN and others simply returned zero, but in all cases the
return code was ignored by the calling function Curl_ssl_close_all().

For the time being and to keep the internal API consistent, changed all
declarations to use a void return type.

To reduce code we might want to consider removing the unimplemented
versions and use a void #define like schannel does.
lib/vtls/curl_schannel.h
lib/vtls/gskit.c
lib/vtls/gskit.h
lib/vtls/nss.c
lib/vtls/nssg.h
lib/vtls/openssl.c
lib/vtls/openssl.h