[PATCH 3/4] lib: Only synchronize maildir flags for messages in maildirs
Jani Nikula
jani at nikula.org
Mon Jun 4 00:01:57 PDT 2012
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 at 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 at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list