On Nov 2, 2013 2:14 PM, "David Bremner" wrote: > > Jani Nikula writes: > > > Adding another level is just useless cruft. > > --- > > lib/database.cc | 14 ++++---------- > > notmuch-compact.c | 4 ++-- > > 2 files changed, 6 insertions(+), 12 deletions(-) > > The first two patches are fine, modulo my comments about documentation. > I think the backup path needs to be updated in notmuch-compact.c, and > possible the test updated (after applying > id:1383350515-24320-1-git-send-email-david@tethera.net) > > d Ah, right, I was juggling the patch order back and forth a bit, and I missed this (as I drop the path completely in a later patch). Jani.