From 854f82fb91ef9560259e4597d475747e948204d5 Mon Sep 17 00:00:00 2001 From: Carl Worth Date: Tue, 27 Oct 2009 16:17:22 -0700 Subject: [PATCH 1/1] notmuch_database_close: Explicitly flush the Xapian database. This would have helped with the recent bug causing "notmuch new" to not record any results in the database. I'm not sure why the explicit flush would be required, (shouldn't the destructor always ensure that things flush?), but perhaps some outstanding references from the leak prevented that. In any case, an explicit flush on close() seems to make sense. --- database.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/database.cc b/database.cc index 9831d79c..8d477eca 100644 --- a/database.cc +++ b/database.cc @@ -505,6 +505,8 @@ notmuch_database_open (const char *path) void notmuch_database_close (notmuch_database_t *notmuch) { + notmuch->xapian_db->flush (); + delete notmuch->query_parser; delete notmuch->xapian_db; talloc_free (notmuch); -- 2.43.0