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