Re: [PATCH 2/4] test: define test_expect_equal_message_body

Subject: Re: [PATCH 2/4] test: define test_expect_equal_message_body

Date: Fri, 21 Jan 2022 22:56:46 -0400

To: Tomi Ollila, notmuch@notmuchmail.org

Cc:

From: David Bremner


Tomi Ollila <tomi.ollila@iki.fi> writes:

> The above works, as we don't have whitespace ($IFS characters to be
> precise) in the filenames. However, for better example, "$1" and "$2"
> should be used (as is used in many of the functions in test-lib.sh
> already. 

Makes sense, done (in private git)

> The above prints out also the leading newline (which is OK), but if
> one wanted those w/o the leading newline, the sed lines would be:
> sed '1,/^$/d'

Yeah, this a tougher call, but I decided to go with your version since I
was editing the function anyway. It is (maybe) slightly easier to understand
without sed -n.

d

_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: