[PATCH v7 2/2] emacs: Tests for user-defined sections
Daniel Schoepe
daniel at schoepe.org
Sat Jan 28 14:54:38 PST 2012
Hi Dmitry.
On Sun, 29 Jan 2012 02:48, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> I only meant trailing whitespaces in test/emacs file in the lines added
> by the patch. Trailing whitespaces in expected results are obviously
> fine.
Ah, okay. I fixed those with delete-trailing-whitespace, but then
noticed that this also corrected some trailing whitespace in unrelated
tests, reverted that and then forgot about the trailing whitespace in my
tests by the time I responded.
> I think splitting emacs tests based on -hello, -show and -search views
> is a good idea. Splitting existing tests is out of scope of this
> patch. But I think it may be a good opportunity to add a new file with
> -hello tests.
You're right; I'll do that if you find other flaws in your next
review. Otherwise, I'll just put it in another patch.
Cheers,
Daniel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120128/444230f5/attachment.pgp>
More information about the notmuch
mailing list