[Patch v5 05/11] lib: provide config API
David Bremner
david at tethera.net
Tue May 24 04:48:11 PDT 2016
David Bremner <david at tethera.net> writes:
> + notmuch->exception_reported = TRUE;
> + if (! notmuch->exception_reported) {
> + _notmuch_database_log (notmuch, "Error: A Xapian exception occurred setting metadata: %s\n",
> + error.get_msg().c_str());
> + }
Upon revisiting, this looks pretty silly. According to the commit
message in e2341cb the point of exception_reported is to avoid excess
error reporting when closing/flushing after another error. Since we
don't actually print these error messages anymore, I'm not sure it's
even needed. In any case, this is a bad case of copy-pasta, which I've
fixed in git by removing the if (in both places).
More information about the notmuch
mailing list