On Tue, 24 Jan 2012 19:45:18 +0400, Dmitry Kurochkin <[email protected]> wrote: > 1. rename notmuch-test-compare to notmuch-test-expect-equal
I'll change this. > 2. move the top level equal check to the non-list branch I'd rather not change this, though I'll write some commentary to explain how things are. > But both of these are subjective and minor, so I leave it to David to > decide whether to change or ignore them. Otherwise, the patch looks > good to me. > > While replying to this email, I noticed a trailing whitespace. I may > have missed them in other patches. David, can you please check for > trailing white spaces and clean them? Yes.
pgp5UqTzP1FuC.pgp
Description: PGP signature
_______________________________________________ notmuch mailing list [email protected] http://notmuchmail.org/mailman/listinfo/notmuch
