[PATCH 1/2] cli: crypto: abstract gpg context creation for clarity
Jani Nikula
jani at nikula.org
Sun Mar 3 06:10:10 PST 2013
The code filled with #ifdef GMIME_ATLEAST_26 is difficult to
read. Abstract gpg context creation into a function, with separate
implementations for GMime 2.4 and 2.6, to clarify the code.
There should be no functional changes.
---
crypto.c | 60 +++++++++++++++++++++++++++++++++++++++++-------------------
1 file changed, 41 insertions(+), 19 deletions(-)
diff --git a/crypto.c b/crypto.c
index cb361e1..7d1dfe4 100644
--- a/crypto.c
+++ b/crypto.c
@@ -20,6 +20,42 @@
#include "notmuch-client.h"
+#ifdef GMIME_ATLEAST_26
+static notmuch_crypto_context_t *
+create_gpg_context (void)
+{
+ notmuch_crypto_context_t *gpgctx;
+
+ /* TODO: GMimePasswordRequestFunc */
+ gpgctx = g_mime_gpg_context_new (NULL, "gpg");
+ if (! gpgctx)
+ return NULL;
+
+ g_mime_gpg_context_set_use_agent ((GMimeGpgContext *) gpgctx, TRUE);
+ g_mime_gpg_context_set_always_trust ((GMimeGpgContext *) gpgctx, FALSE);
+
+ return gpgctx;
+}
+#else
+static notmuch_crypto_context_t *
+create_gpg_context (void)
+{
+ GMimeSession *session;
+ notmuch_crypto_context_t *gpgctx;
+
+ session = g_object_new (g_mime_session_get_type (), NULL);
+ gpgctx = g_mime_gpg_context_new (session, "gpg");
+ g_object_unref (session);
+
+ if (! gpgctx)
+ return NULL;
+
+ g_mime_gpg_context_set_always_trust ((GMimeGpgContext *) gpgctx, FALSE);
+
+ return gpgctx;
+}
+#endif
+
/* for the specified protocol return the context pointer (initializing
* if needed) */
notmuch_crypto_context_t *
@@ -33,28 +69,14 @@ notmuch_crypto_get_context (notmuch_crypto_t *crypto, const char *protocol)
* parameter names as defined in this document are
* case-insensitive." Thus, we use strcasecmp for the protocol.
*/
- if ((strcasecmp (protocol, "application/pgp-signature") == 0)
- || (strcasecmp (protocol, "application/pgp-encrypted") == 0)) {
- if (!crypto->gpgctx) {
-#ifdef GMIME_ATLEAST_26
- /* TODO: GMimePasswordRequestFunc */
- crypto->gpgctx = g_mime_gpg_context_new (NULL, "gpg");
-#else
- GMimeSession* session = g_object_new (g_mime_session_get_type(), NULL);
- crypto->gpgctx = g_mime_gpg_context_new (session, "gpg");
- g_object_unref (session);
-#endif
- if (crypto->gpgctx) {
-#ifdef GMIME_ATLEAST_26
- g_mime_gpg_context_set_use_agent ((GMimeGpgContext*) crypto->gpgctx, TRUE);
-#endif
- g_mime_gpg_context_set_always_trust ((GMimeGpgContext*) crypto->gpgctx, FALSE);
- } else {
+ if (strcasecmp (protocol, "application/pgp-signature") == 0 ||
+ strcasecmp (protocol, "application/pgp-encrypted") == 0) {
+ if (! crypto->gpgctx) {
+ crypto->gpgctx = create_gpg_context ();
+ if (! crypto->gpgctx)
fprintf (stderr, "Failed to construct gpg context.\n");
- }
}
cryptoctx = crypto->gpgctx;
-
} else {
fprintf (stderr, "Unknown or unsupported cryptographic protocol.\n");
}
--
1.7.10.4
More information about the notmuch
mailing list