Re: [PATCH 2/3] show: output Reply-To headers

Subject: Re: [PATCH 2/3] show: output Reply-To headers

Date: Tue, 03 Jul 2012 18:27:23 -0700

To: Peter Wang, notmuch@notmuchmail.org

Cc:

From: Jameson Graef Rollins

On Sat, Jun 23 2012, Peter Wang <novalazy@gmail.com> wrote:
> Output Reply-To headers when present in a message.
> These were missing in text and json output formats.

I don't see why the Reply-To header is needed, given that we are
constructing reply templates, but I guess if someone thinks it's needed
I don't see much harm in including it.

I didn't test it, but it looks ok to me.

jamie.

• Previous message (by thread): Re: [PATCH 1/3] test: add basic show, search --format=text tests

Thread:

• Peter Wang—[PATCH 1/3] test: add basic show, search --format=text tests [inbox, notmuch::patch, notmuch::pushed, unread]
  • Peter Wang—[PATCH 2/3] show: output Reply-To headers [inbox, notmuch::moreinfo, notmuch::patch, notmuch::stale, unread]
    • Jameson Graef Rollins—Re: [PATCH 2/3] show: output Reply-To headers [inbox, notmuch::review, signed, unread]
      • Peter Wang—Re: [PATCH 2/3] show: output Reply-To headers [inbox, unread]
        • Jameson Graef Rollins—Re: [PATCH 2/3] show: output Reply-To headers [inbox, signed, unread]
          • Peter Wang—Re: [PATCH 2/3] show: output Reply-To headers [inbox, unread]
            • Mark Walters—Re: [PATCH 2/3] show: output Reply-To headers [inbox, unread]
              • Austin Clements—Re: [PATCH 2/3] show: output Reply-To headers [inbox, unread]
  • Peter Wang—[PATCH 3/3] test: add test for showing Reply-To headers [inbox, notmuch::moreinfo, notmuch::patch, notmuch::stale, unread]
  • Jameson Graef Rollins—Re: [PATCH 1/3] test: add basic show, search --format=text tests [inbox, notmuch::review, signed, unread]
  • Austin Clements—Re: [PATCH 1/3] test: add basic show, search --format=text tests [inbox, notmuch::review, unread]
  • David Bremner—Re: [PATCH 1/3] test: add basic show, search --format=text tests [inbox, unread]