← Back to team overview

zeitgeist team mailing list archive

Re: [Merge] lp:~zeitgeist/zeitgeist/fts++ into lp:zeitgeist

 

Review: Needs Fixing

1583	+void Indexer::Flush ()
1584	+{
1585	+ db->flush ();
1586	+}

This need to be Commit() and db->commit(). See http://xapian.org/docs/apidoc/html/classXapian_1_1WritableDatabase.html#cbea2163142de795024880a7123bc693. You should probably also surround it with a try/catch.
-- 
https://code.launchpad.net/~zeitgeist/zeitgeist/fts++/+merge/92022
Your team Zeitgeist Framework Team is subscribed to branch lp:zeitgeist.


References