diff options
author | Daniel Kahn Gillmor <dkg@fifthhorseman.net> | 2017-10-10 01:49:04 -0400 |
---|---|---|
committer | David Bremner <david@tethera.net> | 2017-10-12 22:11:55 -0300 |
commit | bfd669bb2ef01d3142dd8294c7e1326ee7503e39 (patch) | |
tree | 971ee97d3c8c0e274388a4d45a55effd953bc992 | |
parent | 3e9857a0b98c04973b9b6ad2da7dc5c3992b3b1c (diff) |
crypto: _notmuch_crypto_cleanup should return void
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.
-rw-r--r-- | crypto.c | 7 | ||||
-rw-r--r-- | notmuch-client.h | 2 |
2 files changed, 3 insertions, 6 deletions
@@ -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 |