[PATCH] Use stock GMimeSession by default

2011-06-03 Thread Daniel Kahn Gillmor 
Our use of GMimeSession was unneeded boilerplate, and we weren't doing
anything with it.  This simplifies and clarifies that assumption.

If we want to do anything fancier later, the examples in the gmime
source are a reasonable source to work from in defining a new
GMimeSession derivative.

Since GMimeSession is going away in GMime 2.6, though, i don't
recommend using it.
---
 Makefile.local  |1 -
 notmuch-client.h|3 --
 notmuch-gmime-session.c |   49 ---
 notmuch-reply.c |2 +-
 notmuch-show.c  |2 +-
 5 files changed, 2 insertions(+), 55 deletions(-)
 delete mode 100644 notmuch-gmime-session.c

diff --git a/Makefile.local b/Makefile.local
index f726f1f..8a8832d 100644
--- a/Makefile.local
+++ b/Makefile.local
@@ -246,7 +246,6 @@ notmuch_client_srcs =   \
notmuch-show.c  \
notmuch-tag.c   \
notmuch-time.c  \
-   notmuch-gmime-session.c \
query-string.c  \
show-message.c  \
json.c  \
diff --git a/notmuch-client.h b/notmuch-client.h
index 8a27260..63be337 100644
--- a/notmuch-client.h
+++ b/notmuch-client.h
@@ -237,7 +237,4 @@ notmuch_config_set_maildir_synchronize_flags 
(notmuch_config_t *config,
 notmuch_bool_t
 debugger_is_active (void);

-GType
-notmuch_gmime_session_get_type (void);
-
 #endif
diff --git a/notmuch-gmime-session.c b/notmuch-gmime-session.c
deleted file mode 100644
index d83d9b3..000
--- a/notmuch-gmime-session.c
+++ /dev/null
@@ -1,49 +0,0 @@
-#include "notmuch-client.h"
-
-/* CRUFTY BOILERPLATE for GMimeSession (dkg thinks this will go away once 
GMime 2.6 comes out) */
-typedef struct _NotmuchGmimeSession NotmuchGmimeSession;
-typedef struct _NotmuchGmimeSessionClass NotmuchGmimeSessionClass;
-
-struct _NotmuchGmimeSession {
-GMimeSession parent_object;
-};
-
-struct _NotmuchGmimeSessionClass {
-GMimeSessionClass parent_class;
-};
-
-static void notmuch_gmime_session_class_init (NotmuchGmimeSessionClass *klass);
-
-static GMimeSessionClass *parent_class = NULL;
-
-GType
-notmuch_gmime_session_get_type (void)
-{
-static GType type = 0;
-
-if (!type) {
-   static const GTypeInfo info = {
-   sizeof (NotmuchGmimeSessionClass),
-   NULL, /* base_class_init */
-   NULL, /* base_class_finalize */
-   (GClassInitFunc) notmuch_gmime_session_class_init,
-   NULL, /* class_finalize */
-   NULL, /* class_data */
-   sizeof (NotmuchGmimeSession),
-   0,/* n_preallocs */
-   NULL, /* object_init */
-   NULL, /* value_table */
-   };
-   type = g_type_register_static (GMIME_TYPE_SESSION, 
"NotmuchGmimeSession", &info, 0);
-}
-return type;
-}
-
-static void
-notmuch_gmime_session_class_init (NotmuchGmimeSessionClass *klass)
-{
-GMimeSessionClass *session_class = GMIME_SESSION_CLASS (klass);
-parent_class = g_type_class_ref (GMIME_TYPE_SESSION);
-session_class->request_passwd = NULL;
-}
-/* END CRUFTY BOILERPLATE */
diff --git a/notmuch-reply.c b/notmuch-reply.c
index 5265af6..514bbc6 100644
--- a/notmuch-reply.c
+++ b/notmuch-reply.c
@@ -607,7 +607,7 @@ notmuch_reply_command (void *ctx, int argc, char *argv[])
}
} else if ((STRNCMP_LITERAL (argv[i], "--decrypt") == 0)) {
if (params.cryptoctx == NULL) {
-   GMimeSession* session = 
g_object_new(notmuch_gmime_session_get_type(), NULL);
+   GMimeSession* session = g_object_new(g_mime_session_get_type(), 
NULL);
if (NULL == (params.cryptoctx = g_mime_gpg_context_new(session, 
"gpg")))
fprintf (stderr, "Failed to construct gpg context.\n");
else
diff --git a/notmuch-show.c b/notmuch-show.c
index 9267d02..dda83a1 100644
--- a/notmuch-show.c
+++ b/notmuch-show.c
@@ -899,7 +899,7 @@ notmuch_show_command (void *ctx, unused (int argc), unused 
(char *argv[]))
} else if ((STRNCMP_LITERAL (argv[i], "--verify") == 0) ||
   (STRNCMP_LITERAL (argv[i], "--decrypt") == 0)) {
if (params.cryptoctx == NULL) {
-   GMimeSession* session = 
g_object_new(notmuch_gmime_session_get_type(), NULL);
+   GMimeSession* session = g_object_new(g_mime_session_get_type(), 
NULL);
if (NULL == (params.cryptoctx = g_mime_gpg_context_new(session, 
"gpg")))
fprintf (stderr, "Failed to construct gpg context.\n");
else
-- 
1.7.4.4

[PATCH] Use stock GMimeSession by default

2011-06-03 Thread Daniel Kahn Gillmor 

This patch supercedes my patch from
1307142188-6551-1-git-send-email-dkg at fifthhorseman.net , and provides
what appears to be a much cleaner approach to achieve the same result.

 --dkg

Re: [PATCH] Use stock GMimeSession by default

2011-06-03 Thread Carl Worth 
On Fri,  3 Jun 2011 19:57:46 -0400, Daniel Kahn Gillmor 
 wrote:
> Our use of GMimeSession was unneeded boilerplate, and we weren't doing
> anything with it.  This simplifies and clarifies that assumption.

Very nice. This is pushed now.

-Carl


pgpMIWmM0YltL.pgp
Description: PGP signature
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch

[PATCH] Use stock GMimeSession by default

2011-06-03 Thread Carl Worth 
On Fri,  3 Jun 2011 19:57:46 -0400, Daniel Kahn Gillmor  wrote:
> Our use of GMimeSession was unneeded boilerplate, and we weren't doing
> anything with it.  This simplifies and clarifies that assumption.

Very nice. This is pushed now.

-Carl
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL:

[PATCH] Use stock GMimeSession by default

2011-06-03 Thread Daniel Kahn Gillmor 

This patch supercedes my patch from
1307142188-6551-1-git-send-email-...@fifthhorseman.net , and provides
what appears to be a much cleaner approach to achieve the same result.

 --dkg
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch

[PATCH] Use stock GMimeSession by default

2011-06-03 Thread Daniel Kahn Gillmor 
Our use of GMimeSession was unneeded boilerplate, and we weren't doing
anything with it.  This simplifies and clarifies that assumption.

If we want to do anything fancier later, the examples in the gmime
source are a reasonable source to work from in defining a new
GMimeSession derivative.

Since GMimeSession is going away in GMime 2.6, though, i don't
recommend using it.
---
 Makefile.local  |1 -
 notmuch-client.h|3 --
 notmuch-gmime-session.c |   49 ---
 notmuch-reply.c |2 +-
 notmuch-show.c  |2 +-
 5 files changed, 2 insertions(+), 55 deletions(-)
 delete mode 100644 notmuch-gmime-session.c

diff --git a/Makefile.local b/Makefile.local
index f726f1f..8a8832d 100644
--- a/Makefile.local
+++ b/Makefile.local
@@ -246,7 +246,6 @@ notmuch_client_srcs =   \
notmuch-show.c  \
notmuch-tag.c   \
notmuch-time.c  \
-   notmuch-gmime-session.c \
query-string.c  \
show-message.c  \
json.c  \
diff --git a/notmuch-client.h b/notmuch-client.h
index 8a27260..63be337 100644
--- a/notmuch-client.h
+++ b/notmuch-client.h
@@ -237,7 +237,4 @@ notmuch_config_set_maildir_synchronize_flags 
(notmuch_config_t *config,
 notmuch_bool_t
 debugger_is_active (void);
 
-GType
-notmuch_gmime_session_get_type (void);
-
 #endif
diff --git a/notmuch-gmime-session.c b/notmuch-gmime-session.c
deleted file mode 100644
index d83d9b3..000
--- a/notmuch-gmime-session.c
+++ /dev/null
@@ -1,49 +0,0 @@
-#include "notmuch-client.h"
-
-/* CRUFTY BOILERPLATE for GMimeSession (dkg thinks this will go away once 
GMime 2.6 comes out) */
-typedef struct _NotmuchGmimeSession NotmuchGmimeSession;
-typedef struct _NotmuchGmimeSessionClass NotmuchGmimeSessionClass;
-
-struct _NotmuchGmimeSession {
-GMimeSession parent_object;
-};
-
-struct _NotmuchGmimeSessionClass {
-GMimeSessionClass parent_class;
-};
-
-static void notmuch_gmime_session_class_init (NotmuchGmimeSessionClass *klass);
-
-static GMimeSessionClass *parent_class = NULL;
-
-GType
-notmuch_gmime_session_get_type (void)
-{
-static GType type = 0;
-
-if (!type) {
-   static const GTypeInfo info = {
-   sizeof (NotmuchGmimeSessionClass),
-   NULL, /* base_class_init */
-   NULL, /* base_class_finalize */
-   (GClassInitFunc) notmuch_gmime_session_class_init,
-   NULL, /* class_finalize */
-   NULL, /* class_data */
-   sizeof (NotmuchGmimeSession),
-   0,/* n_preallocs */
-   NULL, /* object_init */
-   NULL, /* value_table */
-   };
-   type = g_type_register_static (GMIME_TYPE_SESSION, 
"NotmuchGmimeSession", &info, 0);
-}
-return type;
-}
-
-static void
-notmuch_gmime_session_class_init (NotmuchGmimeSessionClass *klass)
-{
-GMimeSessionClass *session_class = GMIME_SESSION_CLASS (klass);
-parent_class = g_type_class_ref (GMIME_TYPE_SESSION);
-session_class->request_passwd = NULL;
-}
-/* END CRUFTY BOILERPLATE */
diff --git a/notmuch-reply.c b/notmuch-reply.c
index 5265af6..514bbc6 100644
--- a/notmuch-reply.c
+++ b/notmuch-reply.c
@@ -607,7 +607,7 @@ notmuch_reply_command (void *ctx, int argc, char *argv[])
}
} else if ((STRNCMP_LITERAL (argv[i], "--decrypt") == 0)) {
if (params.cryptoctx == NULL) {
-   GMimeSession* session = 
g_object_new(notmuch_gmime_session_get_type(), NULL);
+   GMimeSession* session = g_object_new(g_mime_session_get_type(), 
NULL);
if (NULL == (params.cryptoctx = g_mime_gpg_context_new(session, 
"gpg")))
fprintf (stderr, "Failed to construct gpg context.\n");
else
diff --git a/notmuch-show.c b/notmuch-show.c
index 9267d02..dda83a1 100644
--- a/notmuch-show.c
+++ b/notmuch-show.c
@@ -899,7 +899,7 @@ notmuch_show_command (void *ctx, unused (int argc), unused 
(char *argv[]))
} else if ((STRNCMP_LITERAL (argv[i], "--verify") == 0) ||
   (STRNCMP_LITERAL (argv[i], "--decrypt") == 0)) {
if (params.cryptoctx == NULL) {
-   GMimeSession* session = 
g_object_new(notmuch_gmime_session_get_type(), NULL);
+   GMimeSession* session = g_object_new(g_mime_session_get_type(), 
NULL);
if (NULL == (params.cryptoctx = g_mime_gpg_context_new(session, 
"gpg")))
fprintf (stderr, "Failed to construct gpg context.\n");
else
-- 
1.7.4.4

___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch