[PATCH v2 01/10] lib: add return status to database close and destroy
David Bremner
david at tethera.net
Sat Jul 5 05:39:43 PDT 2014
Peter Wang <novalazy at gmail.com> writes:
> From: Jani Nikula <jani at nikula.org>
>
> notmuch_database_close may fail in Xapian ->flush() or ->close(), so
> report the status. Similarly for notmuch_database_destroy which calls
> close.
This patch looks OK, and it makes sense independent of what else
follows with respect to notmuch-insert.
With respect so SONAME bumping, we might be able to get away with out it
here (although that could be un-portable or broken in a way I haven't
seen yet), but we can also delay that decision until closer to the next
release; I don't think it's as important to provide a stable ABI for
people building from git, and I'd prefer to batch ABI changes into one
SONAME bump.
More information about the notmuch
mailing list