| commit | 2c3c3d88fcda1558d8d57301c21bd548af71a04e | |
| author | Jeff King <peff@peff.net> | |
| Tue, 29 Aug 2023 23:45:33 +0000 (29 19:45 -0400) | ||
| committer | Junio C Hamano <gitster@pobox.com> | |
| Wed, 30 Aug 2023 00:56:26 +0000 (29 17:56 -0700) | ||
| tree | f46c2f5adaaecdbd61e82cb12abf9bc6f98cc23f | treesnapshot (tar.gz zip) |
| parent | 2b0e46f563c21429e45f7c57a22b1337b039a3f9 | commitdiff |
imap-send: mark unused parameters with NO_OPENSSL
Earlier patches annotating unused parameters in imap-send missed a few
cases in code that is compiled only with NO_OPENSSL. These need to
retain the extra parameters to match the interfaces used when we compile
with openssl support.
Note in the case of socket_perror() that the function declaration and
parts of its code are shared between the two cases, and only the openssl
code looks at "sock". So we can't simply mark the parameter as always
unused. Instead, we can add a noop statement that references it. This is
ugly, but should be portable.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Earlier patches annotating unused parameters in imap-send missed a few
cases in code that is compiled only with NO_OPENSSL. These need to
retain the extra parameters to match the interfaces used when we compile
with openssl support.
Note in the case of socket_perror() that the function declaration and
parts of its code are shared between the two cases, and only the openssl
code looks at "sock". So we can't simply mark the parameter as always
unused. Instead, we can add a noop statement that references it. This is
ugly, but should be portable.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| imap-send.c | diffblobblamehistory |