Re: [PATCH 0/6] Finish show rewrite

Subject: Re: [PATCH 0/6] Finish show rewrite

Date: Fri, 13 Apr 2012 13:33:33 +0100

To: Austin Clements, notmuch@notmuchmail.org

Cc:

From: Mark Walters

On Sun, 08 Apr 2012, Austin Clements <amdragon@MIT.EDU> wrote:
> The long-awaited and oft-belated conclusion of the show rewrite.  All
> of the formatters have been converted to the new style, so this series
> just rips out unused code and does a little cleanup.

This looks good to me: I don't know the code well but almost all of it
looks like any bugs would give compile time failures. 

Best wishes

Mark

• Previous message (by thread): Re: [PATCH 0/6] Finish show rewrite

Thread:

• Austin Clements—[PATCH 0/6] Finish show rewrite [inbox, unread]
  • Austin Clements—[PATCH 1/6] Sync schemata with current code structure [inbox, notmuch::patch, notmuch::pushed, unread]
  • Austin Clements—[PATCH 2/6] show: Remove support for old-style formatters in show_message [inbox, notmuch::patch, notmuch::pushed, unread]
  • Austin Clements—[PATCH 3/6] Remove show-message.c [inbox, notmuch::patch, notmuch::pushed, unread]
  • Austin Clements—[PATCH 4/6] show: Remove unused fields from notmuch_show_format [inbox, notmuch::patch, notmuch::pushed, unread]
  • Austin Clements—[PATCH 5/6] show: Support NULL values for message_set_{start, sep, end} [inbox, notmuch::patch, notmuch::pushed, unread]
  • Austin Clements—[PATCH 6/6] show: Remove empty message_set_{start,sep,end} fields [inbox, notmuch::patch, notmuch::pushed, unread]
  • Tomi Ollila—Re: [PATCH 0/6] Finish show rewrite [inbox, notmuch::review, unread]
  • Mark Walters—Re: [PATCH 0/6] Finish show rewrite [inbox, unread]
  • Jameson Graef Rollins—Re: [PATCH 0/6] Finish show rewrite [inbox, signed, unread]
  • David Bremner—Re: [PATCH 0/6] Finish show rewrite [inbox, unread]