Re: [PATCH] CLI: print extra headers only for non-replies

Subject: Re: [PATCH] CLI: print extra headers only for non-replies

Date: Wed, 19 Jan 2022 20:00:33 -0400

To: Tomi Ollila, notmuch@notmuchmail.org

Cc:

From: David Bremner


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

> On Tue, Jan 18 2022, David Bremner wrote:
>
>> If in the future we decide to output extra headers for replies, this
>> should be controlled by a separate configuration option.
>> ---
>>  notmuch-show.c     | 3 ++-
>>  test/T310-emacs.sh | 1 -
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/notmuch-show.c b/notmuch-show.c
>> index 136f4439..6a54d9c1 100644
>> --- a/notmuch-show.c
>> +++ b/notmuch-show.c
>> @@ -294,7 +294,8 @@ format_headers_sprinter (sprinter_t *sp, GMimeMessage *message,
>>      }
>>  
>>      /* Output extra headers the user has configured, if any */
>> -    format_extra_headers_sprinter (sp, message);
>> +    if (! reply)
>> +	format_extra_headers_sprinter (sp, message);
>
> Looks simple enough for LGTM.
>

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

Thread: