notmuch reply: Use strstr instead of strcasestr for portability.
authorCarl Worth <cworth@cworth.org>
Thu, 4 Feb 2010 20:29:04 +0000 (12:29 -0800)
committerCarl Worth <cworth@cworth.org>
Thu, 4 Feb 2010 20:42:09 +0000 (12:42 -0800)
This change was already recommended in a comment in the original
implementation of this patch. If someone really wants to support
un-munging in the case of To: and Reply-To: having the same address
but different case, then they can provide a portable approach for
that.

notmuch-reply.c

index 671a915589421cf63cac451eaa553202235d6eb9..885bedb14871f79894853f0e3ba47e41ffc5c7cb 100644 (file)
@@ -219,9 +219,8 @@ mailing_list_munged_reply_to (notmuch_message_t *message)
     mailbox = INTERNET_ADDRESS_MAILBOX (address);
     addr = internet_address_mailbox_get_addr (mailbox);
 
     mailbox = INTERNET_ADDRESS_MAILBOX (address);
     addr = internet_address_mailbox_get_addr (mailbox);
 
-    /* Note that strcasestr() is a GNU extension, strstr() might be sufficient */
-    if (strcasestr (notmuch_message_get_header (message, "to"), addr) != 0 ||
-       strcasestr (notmuch_message_get_header (message, "cc"), addr) != 0)
+    if (strstr (notmuch_message_get_header (message, "to"), addr) != 0 ||
+       strstr (notmuch_message_get_header (message, "cc"), addr) != 0)
     {
        return 1;
     }
     {
        return 1;
     }