From: Michal Sojka <sojkam1@fel.cvut.cz>
To: Thomas Schwinge <thomas@schwinge.name>, notmuch@notmuchmail.org
Subject: Re: [PATCH] Clarify usage of `additional_headers' in
	test/test-lib.sh:generate_message.
In-Reply-To: <1296119841-496-1-git-send-email-thomas@schwinge.name>
References: <1296119841-496-1-git-send-email-thomas@schwinge.name>
User-Agent: Notmuch/0.5-87-g9705d00 (http://notmuchmail.org) Emacs/23.2.1
	(x86_64-pc-linux-gnu)
Date: Thu, 27 Jan 2011 12:32:42 +0100
Message-ID: <87wrlq7fad.fsf@steelpick.2x.cz>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Thomas Schwinge <thomas@schwinge.name>
Precedence: list

On Thu, 27 Jan 2011, Thomas Schwinge wrote:
> Signed-off-by: Thomas Schwinge <thomas@schwinge.name>
> ---
>  test/test-lib.sh |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/test/test-lib.sh b/test/test-lib.sh
> index d179426..f536172 100755
> --- a/test/test-lib.sh
> +++ b/test/test-lib.sh
> @@ -351,8 +351,11 @@ ${additional_headers}"
>  ${additional_headers}"
>      fi
>  
> +    # Note that in the way we're setting it above and using it below,
> +    # `additional_headers' will also serve as the header / body separator
> +    # (empty line in between).

Yes, this will definitely help the future readers of this code. Thanks.

-Michal