I've applied changes in accordance with all of the feedback in this mail and the others, except for... On Wed, 25 Jan 2012 21:53:08 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote: > > + (let ((notmuch-show-insert-text/plain-hook '(notmuch-wash-excerpt-citations)) > > Do we have to override the default value here? I thought > notmuch-wash-excerpt-citations was enabled by default. It avoids potential confusion over where the problem lies. > [...] > > IMO writing the test in lisp does not give any benefit in this case. > Quite the opposite: a simple test is split in two files and becomes more > complex. > > But since we accepted this way of writing tests and you seem to prefer > it, I would not argue about changing it. You're right - it's simpler in the original style here, so v3 switches to that.