[PATCH v3 03/15] crypto: _notmuch_crypto_cleanup should return void
Daniel Kahn Gillmor
dkg at fifthhorseman.net
Mon Oct 9 22:49:04 PDT 2017
There's no chance that _notmuch_crypto_cleanup() will ever return
anything other than 0, and no one ever checks its return value anyway.
So make it return void instead of int.
---
crypto.c | 7 ++-----
notmuch-client.h | 2 +-
2 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/crypto.c b/crypto.c
index fc7c5d1d..4c1b7eec 100644
--- a/crypto.c
+++ b/crypto.c
@@ -117,7 +117,7 @@ _notmuch_crypto_get_gmime_context (_notmuch_crypto_t *crypto, const char *protoc
return NULL;
}
-int
+void
_notmuch_crypto_cleanup (_notmuch_crypto_t *crypto)
{
if (crypto->gpgctx) {
@@ -129,12 +129,9 @@ _notmuch_crypto_cleanup (_notmuch_crypto_t *crypto)
g_object_unref (crypto->pkcs7ctx);
crypto->pkcs7ctx = NULL;
}
-
- return 0;
}
#else
-int _notmuch_crypto_cleanup (unused(_notmuch_crypto_t *crypto))
+void _notmuch_crypto_cleanup (unused(_notmuch_crypto_t *crypto))
{
- return 0;
}
#endif
diff --git a/notmuch-client.h b/notmuch-client.h
index 62acc70e..bdcfd893 100644
--- a/notmuch-client.h
+++ b/notmuch-client.h
@@ -185,7 +185,7 @@ GMimeCryptoContext *
_notmuch_crypto_get_gmime_context (_notmuch_crypto_t *crypto, const char *protocol);
#endif
-int
+void
_notmuch_crypto_cleanup (_notmuch_crypto_t *crypto);
int
--
2.14.2
More information about the notmuch
mailing list