[PATCH v4 06/12] test: add tests for insert
David Bremner
david at tethera.net
Fri Mar 29 16:59:56 PDT 2013
It took longer than I thought (of course) but I finally finished looking
at the first 6 patches.
I already mentioned a minor man page issue in a seperate message.
I took a second pass through 03/12, and I think I would prefer thethe
control flow of insert_message be closer to the standard style in
notmuch of using a return value variable and a single cleanup block at
the end. Reasonable people can disagree about issues of style, but in
the end consistency of the code base is also important.
d
More information about the notmuch
mailing list