[PATCH 00/11] notmuch insert updates
David Bremner
david at tethera.net
Thu Sep 25 01:13:40 PDT 2014
Jani Nikula <jani at nikula.org> writes:
> This series refactors and cleans up insert, improves error handling and
> reporting, and adds post-insert hook. I intend to add documentation and
> more tests, but the code is ready for review. Also, at least some of the
> cleanups and fixes in the beginning of the series could go in without
> additional tests or documentation.
The first 4 seem trivial, and I marked them so.
5-8 don't introduce new functionality, and don't break
any existing tests. Subject to a little more review, these could
probably be merged now.
5 is almost trivial; it wouldn't hurt to explicitly document the return
values of sync_dir, while touching this code.
6 and 8 seem ok, but should probably have another pair of eyes.
7 seems ok. At first I worried about the usual issue of races between
name definition and open, but it seems like nothing changes here.
More information about the notmuch
mailing list