lib/cli: replace use of g_mime_message_get_sender

This function changes semantics in gmime-3.0 so make a new function
that provides the same functionality in both

diff --git a/lib/index.cc b/lib/index.cc
index 0c4e232924ce1e139066c02c9f03bb5336f4167d..044e4111dc0b1420139fe0da429c34421891dd1f 100644 (file)
--- a/lib/index.cc
+++ b/lib/index.cc
@@ -490,7 +490,7 @@ _notmuch_message_index_file (notmuch_message_t *message,
 {
     GMimeMessage *mime_message;
     InternetAddressList *addresses;
 {
     GMimeMessage *mime_message;
     InternetAddressList *addresses;

-    const char *from, *subject;
+    const char *subject;

     notmuch_status_t status;
 
     status = _notmuch_message_file_get_mime_message (message_file,
     notmuch_status_t status;
 
     status = _notmuch_message_file_get_mime_message (message_file,


@@ -498,9 +498,7 @@ _notmuch_message_index_file (notmuch_message_t *message,
     if (status)
        return status;
 
     if (status)
        return status;
 

-    from = g_mime_message_get_sender (mime_message);
-
-    addresses = internet_address_list_parse_string (from);
+    addresses = g_mime_message_get_from (mime_message);

     if (addresses) {
        _index_address_list (message, "from", addresses);
        g_object_unref (addresses);
     if (addresses) {
        _index_address_list (message, "from", addresses);
        g_object_unref (addresses);

diff --git a/lib/notmuch-private.h b/lib/notmuch-private.h
index ac315e4c99b77ae316571d3b0d686612100f05c3..5dfebf5dfd0d3a34f2953bbec90aca1dc74e3a29 100644 (file)
--- a/lib/notmuch-private.h
+++ b/lib/notmuch-private.h
@@ -46,7 +46,7 @@ NOTMUCH_BEGIN_DECLS
 
 #include <talloc.h>
 
 
 #include <talloc.h>
 

-#include <gmime/gmime.h>
+#include "gmime-extra.h"

 
 #include "xutil.h"
 #include "error_util.h"
 
 #include "xutil.h"
 #include "error_util.h"

diff --git a/notmuch-reply.c b/notmuch-reply.c
index f789a14083e0fa444a294cd2e4d2c0c559459806..ceb1d231e6c85a7b6d23dedc81a5d8ad861c2e0e 100644 (file)
--- a/notmuch-reply.c
+++ b/notmuch-reply.c
@@ -47,9 +47,9 @@ format_part_reply (GMimeStream *stream, mime_node_t *node)
        InternetAddressList *recipients;
        char *recipients_string;
 
        InternetAddressList *recipients;
        char *recipients_string;
 

-       g_mime_stream_printf (stream, "> From: %s\n", g_mime_message_get_sender (message));

        recipients = g_mime_message_get_recipients (message, GMIME_RECIPIENT_TYPE_TO);
        recipients_string = internet_address_list_to_string (recipients, 0);
        recipients = g_mime_message_get_recipients (message, GMIME_RECIPIENT_TYPE_TO);
        recipients_string = internet_address_list_to_string (recipients, 0);

+       g_mime_stream_printf (stream, "> From: %s\n", g_mime_message_get_from_string (message));

        if (recipients_string)
            g_mime_stream_printf (stream, "> To: %s\n",
                                  recipients_string);
        if (recipients_string)
            g_mime_stream_printf (stream, "> To: %s\n",
                                  recipients_string);


@@ -292,8 +292,7 @@ static InternetAddressList *get_sender(GMimeMessage *message)
        g_object_unref (G_OBJECT (reply_to_list));
     }
 
        g_object_unref (G_OBJECT (reply_to_list));
     }
 

-    return internet_address_list_parse_string (
-       g_mime_message_get_sender (message));
+    return g_mime_message_get_from (message);

 }
 
 static InternetAddressList *get_to(GMimeMessage *message)
 }
 
 static InternetAddressList *get_to(GMimeMessage *message)

diff --git a/notmuch-show.c b/notmuch-show.c
index e0433095da37b76bc127d372ee9784172b12c5ae..1a72d12f68cab3980b4273d8de8b917b39c9dbc4 100644 (file)
--- a/notmuch-show.c
+++ b/notmuch-show.c
@@ -212,7 +212,7 @@ format_headers_sprinter (sprinter_t *sp, GMimeMessage *message,
     sp->string (sp, g_mime_message_get_subject (message));
 
     sp->map_key (sp, "From");
     sp->string (sp, g_mime_message_get_subject (message));
 
     sp->map_key (sp, "From");

-    sp->string (sp, g_mime_message_get_sender (message));
+    sp->string (sp, g_mime_message_get_from_string (message));

 
     recipients = g_mime_message_get_recipients (message, GMIME_RECIPIENT_TYPE_TO);
     recipients_string = internet_address_list_to_string (recipients, 0);
 
     recipients = g_mime_message_get_recipients (message, GMIME_RECIPIENT_TYPE_TO);
     recipients_string = internet_address_list_to_string (recipients, 0);


@@ -520,9 +520,9 @@ format_part_text (const void *ctx, sprinter_t *sp, mime_node_t *node,
        if (node->envelope_file)
            g_mime_stream_printf (stream, "%s\n", _get_one_line_summary (ctx, node->envelope_file));
        g_mime_stream_printf (stream, "Subject: %s\n", g_mime_message_get_subject (message));
        if (node->envelope_file)
            g_mime_stream_printf (stream, "%s\n", _get_one_line_summary (ctx, node->envelope_file));
        g_mime_stream_printf (stream, "Subject: %s\n", g_mime_message_get_subject (message));

-       g_mime_stream_printf (stream, "From: %s\n", g_mime_message_get_sender (message));

        recipients = g_mime_message_get_recipients (message, GMIME_RECIPIENT_TYPE_TO);
        recipients_string = internet_address_list_to_string (recipients, 0);
        recipients = g_mime_message_get_recipients (message, GMIME_RECIPIENT_TYPE_TO);
        recipients_string = internet_address_list_to_string (recipients, 0);

+       g_mime_stream_printf (stream, "From: %s\n", g_mime_message_get_from_string (message));

        if (recipients_string)
            g_mime_stream_printf (stream, "To: %s\n", recipients_string);
        g_free (recipients_string);
        if (recipients_string)
            g_mime_stream_printf (stream, "To: %s\n", recipients_string);
        g_free (recipients_string);

diff --git a/util/gmime-extra.c b/util/gmime-extra.c
index 33751de7d20c4b70e6cfbed56f1128db4c9467ac..8590c55d184b747799f7a30b545963e748f5de10 100644 (file)
--- a/util/gmime-extra.c
+++ b/util/gmime-extra.c
@@ -39,6 +39,17 @@ g_mime_message_get_date_string (void *ctx, GMimeMessage *message)
     return g_string_talloc_strdup (ctx, date);
 }
 
     return g_string_talloc_strdup (ctx, date);
 }
 

+InternetAddressList *
+g_mime_message_get_from (GMimeMessage *message)
+{
+    return internet_address_list_parse_string (g_mime_message_get_sender (message));
+}
+
+const char *
+g_mime_message_get_from_string (GMimeMessage *message) {
+    return  g_mime_message_get_sender (message);
+}
+

 InternetAddressList *
 g_mime_message_get_reply_to_list (GMimeMessage *message)
 {
 InternetAddressList *
 g_mime_message_get_reply_to_list (GMimeMessage *message)
 {


@@ -81,6 +92,12 @@ g_mime_message_get_reply_to_list(GMimeMessage *message)
     return g_mime_message_get_reply_to (message);
 }
 
     return g_mime_message_get_reply_to (message);
 }
 

+const char *
+g_mime_message_get_from_string (GMimeMessage *message)
+{
+    return g_mime_object_get_header (GMIME_OBJECT (message), "From");
+}
+

 char *
 g_mime_message_get_reply_to_string (void *ctx, GMimeMessage *message)
 {
 char *
 g_mime_message_get_reply_to_string (void *ctx, GMimeMessage *message)
 {

diff --git a/util/gmime-extra.h b/util/gmime-extra.h
index 794ffbfdf9bbc13a215cb58ea9e09e4fce51b2f3..04d630260259094ee5e7e527a24b4417d3c86275 100644 (file)
--- a/util/gmime-extra.h
+++ b/util/gmime-extra.h
@@ -9,18 +9,27 @@ GMimeStream *g_mime_stream_stdout_new(void);
 /**
  * return talloc allocated date string
  */
 /**
  * return talloc allocated date string
  */

+

 char *g_mime_message_get_date_string (void *ctx, GMimeMessage *message);
 char *g_mime_message_get_date_string (void *ctx, GMimeMessage *message);

-InternetAddressList * g_mime_message_get_reply_to_list (GMimeMessage *message);

 
 /**
 
 /**

- * return talloc allocated reply-to string
+ * glib allocated list of From: addresses

  */
  */

-char * g_mime_message_get_reply_to_string (void *ctx, GMimeMessage *message);

 
 

+InternetAddressList * g_mime_message_get_from (GMimeMessage *message);

 
 /**
 
 /**

- * Return glib allocated reply-to list
+ * return string for From: address
+ * (owned by gmime)

  */
  */

+const char * g_mime_message_get_from_string (GMimeMessage *message);
+

 InternetAddressList * g_mime_message_get_reply_to_list (GMimeMessage *message);
 
 InternetAddressList * g_mime_message_get_reply_to_list (GMimeMessage *message);
 

+/**
+ * return talloc allocated reply-to string
+ */
+char * g_mime_message_get_reply_to_string (void *ctx, GMimeMessage *message);
+
+

 #endif
 #endif