Eliza Velasquez <el...@eliza.sh> writes:

> It becomes very clear why this error only happens when removing a
> non-existent tag if you look at at message.cc:1570...
> --8<---------------cut here---------------start------------->8---
>     try {
>       message->doc.remove_term (term);
>       message->modified = true;
>     } catch (const Xapian::InvalidArgumentError) {
>       /* We'll let the philosophers try to wrestle with the
>        * question of whether failing to remove that which was not
>        * there in the first place is failure. For us, we'll silently
>        * consider it all good. */
>     }
> --8<---------------cut here---------------end--------------->8---

OK, I see why that assignment gets skipped. I think that's not the
actual bug, but rather message->modified should be initialized in
_notmuch_message_create_for_document. I'll have a closer look later

notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org

Reply via email to