Re: [PATCH 3/4] lib: Only synchronize maildir flags for messages in maildirs

Subject: Re: [PATCH 3/4] lib: Only synchronize maildir flags for messages in maildirs

Date: Mon, 04 Jun 2012 07:01:57 +0000

To: Austin Clements, notmuch@notmuchmail.org

Cc:

From: Jani Nikula


The patch makes a lot of sense, and fixes the asymmetry introduced by
commit 95dd5fe5. I think it deserves a more verbose commit message,
along with that commit reference, and (eventually, not necessarily in
this series) a NEWS item of its own.

On Sun, 03 Jun 2012, Austin Clements <amdragon@MIT.EDU> wrote:
> ---
>  lib/message.cc |    6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/lib/message.cc b/lib/message.cc
> index ed96477..bbac2ff 100644
> --- a/lib/message.cc
> +++ b/lib/message.cc
> @@ -1074,7 +1074,7 @@ notmuch_message_maildir_flags_to_tags (notmuch_message_t *message)
>      const char *flags;
>      notmuch_status_t status;
>      notmuch_filenames_t *filenames;
> -    const char *filename;
> +    const char *filename, *dir;
>      char *combined_flags = talloc_strdup (message, "");
>      unsigned i;
>      int seen_maildir_info = 0;
> @@ -1084,6 +1084,10 @@ notmuch_message_maildir_flags_to_tags (notmuch_message_t *message)
>  	 notmuch_filenames_move_to_next (filenames))
>      {
>  	filename = notmuch_filenames_get (filenames);
> +	dir = _filename_is_in_maildir (filename);
> +
> +	if (! dir)
> +	    continue;
>  
>  	flags = strstr (filename, ":2,");
>  	if (! flags)
> -- 
> 1.7.10
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

Thread: