debian/notmuch
1
0
Fork 0
mirror of https://git.notmuchmail.org/git/notmuch synced 2024-11-28 21:54:10 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

notmuch_database_close: Explicitly flush the Xapian database.

Browse source 
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.
This commit is contained in:
Carl Worth 2009-10-27 16:17:22 -07:00
parent b0190e59ac
commit 854f82fb91
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
database.cc
View file

@ -505,6 +505,8 @@ notmuch_database_open (const char *path)
void void
notmuch_database_close (notmuch_database_t *notmuch) notmuch_database_close (notmuch_database_t *notmuch)
{ {
notmuch->xapian_db->flush ();
delete notmuch->query_parser; delete notmuch->query_parser;
delete notmuch->xapian_db; delete notmuch->xapian_db;
talloc_free (notmuch); talloc_free (notmuch);