Re: [PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

Subject: Re: [PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

Date: Thu, 27 Jan 2011 12:32:42 +0100

To: Thomas Schwinge, notmuch@notmuchmail.org

Cc: Thomas Schwinge

From: Michal Sojka


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

Thread: