[notmuch] [PATCH] Make addresses case insensitive for the purpose of constructing replies.

2009-11-23 Thread Jed Brown
The domain is alway case insensitive, but in principle the username is
case sensitive.  Few systems actually enforce this so I think a good
default is to treat the entire address as case insensitive, it will
eliminate a lot of superfluous self-addressed messages and reply from
the correct address in these cases.

Signed-off-by: Jed Brown 
---
 notmuch-reply.c |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/notmuch-reply.c b/notmuch-reply.c
index 44e1766..cd81e76 100644
--- a/notmuch-reply.c
+++ b/notmuch-reply.c
@@ -97,12 +97,12 @@ address_is_users (const char *address, notmuch_config_t 
*config)
 size_t i, other_len;

 primary = notmuch_config_get_user_primary_email (config);
-if (strcmp (primary, address) == 0)
+if (strcasecmp (primary, address) == 0)
return 1;

 other = notmuch_config_get_user_other_email (config, &other_len);
 for (i = 0; i < other_len; i++)
-   if (strcmp (other[i], address) == 0)
+   if (strcasecmp (other[i], address) == 0)
return 1;

 return 0;
-- 
1.6.5.3



[notmuch] [PATCH] Make addresses case insensitive for the purpose of constructing replies.

2009-11-23 Thread Carl Worth
On Mon, 23 Nov 2009 19:29:52 +0100, Jed Brown  wrote:
> The domain is alway case insensitive, but in principle the username is
> case sensitive.  Few systems actually enforce this so I think a good
> default is to treat the entire address as case insensitive, it will
> eliminate a lot of superfluous self-addressed messages and reply from
> the correct address in these cases.

Good call. I've pushed this out now.

-Carl