Re: [PATCH 01/11] emacs: Remove redundant NTH argument from `notmuch-get-bodypart-content'.

Subject: Re: [PATCH 01/11] emacs: Remove redundant NTH argument from `notmuch-get-bodypart-content'.

Date: Thu, 10 Jul 2014 06:46:52 -0300

To: Austin Clements, notmuch@notmuchmail.org

Cc:

From: David Bremner

Austin Clements <amdragon@MIT.EDU> writes:

> This can be derived from the PART argument (which is arguably
> canonical), so there's no sense in giving the caller an extra foot
> gun.

This patch seem ready to go, independant of the rest of the series.

d

• Previous message (by thread): Re: [PATCH 08/11] emacs: Support caching in notmuch-get-bodypart-{binary, text}

Thread:

• Austin Clements—[PATCH 00/11] Improve charset and cid: handling [inbox, unread]
  • Austin Clements—[PATCH 01/11] emacs: Remove redundant NTH argument from `notmuch-get-bodypart-content'. [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
    • David Bremner—Re: [PATCH 01/11] emacs: Remove redundant NTH argument from `notmuch-get-bodypart-content'. [inbox, unread]
  • Austin Clements—[PATCH 02/11] test: New tests for Emacs charset handling [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
    • Mark Walters—Re: [PATCH 02/11] test: New tests for Emacs charset handling [inbox, unread]
      • Austin Clements—Re: [PATCH 02/11] test: New tests for Emacs charset handling [inbox, unread]
        • Mark Walters—Re: [PATCH 02/11] test: New tests for Emacs charset handling [inbox, unread]
        • Tomi Ollila—Re: [PATCH 02/11] test: New tests for Emacs charset handling [inbox, unread]
  • Austin Clements—[PATCH 03/11] emacs: Fix coding system in `notmuch-show-view-raw-message' [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
    • David Bremner—Re: [PATCH 03/11] emacs: Fix coding system in `notmuch-show-view-raw-message' [inbox, unread]
    • David Bremner—Re: [PATCH 03/11] emacs: Fix coding system in `notmuch-show-view-raw-message' [inbox, unread]
  • Austin Clements—[PATCH 04/11] emacs: Track full message and part descriptor in w3m CID store [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    • David Bremner—Re: [PATCH 04/11] emacs: Track full message and part descriptor in w3m CID store [inbox, unread]
      • Austin Clements—Re: [PATCH 04/11] emacs: Track full message and part descriptor in w3m CID store [inbox, unread]
  • Austin Clements—[PATCH 05/11] emacs: Create an API for fetching parts as undecoded binary [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH 06/11] emacs: Remove broken `notmuch-get-bodypart-content' API [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    • David Bremner—Re: [PATCH 06/11] emacs: Remove broken `notmuch-get-bodypart-content' API [inbox, unread]
      • Austin Clements—Re: [PATCH 06/11] emacs: Remove broken `notmuch-get-bodypart-content' API [inbox, unread]
  • Austin Clements—[PATCH 07/11] emacs: Return unibyte strings for binary part data [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH 08/11] emacs: Support caching in notmuch-get-bodypart-{binary, text} [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    • Mark Walters—Re: [PATCH 08/11] emacs: Support caching in notmuch-get-bodypart-{binary, text} [inbox, unread]
      • Austin Clements—Re: [PATCH 08/11] emacs: Support caching in notmuch-get-bodypart-{binary, text} [inbox, unread]
        • Mark Walters—Re: [PATCH 08/11] emacs: Support caching in notmuch-get-bodypart-{binary, text} [inbox, unread]
          • Austin Clements—Re: [PATCH 08/11] emacs: Support caching in notmuch-get-bodypart-{binary, text} [inbox, unread]
  • Austin Clements—[PATCH 09/11] emacs: Use generalized content caching in w3m CID code [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH 10/11] emacs: Rewrite content ID handling [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH 11/11] emacs: Support cid: references with shr renderer [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    • David Edmondson—Re: [PATCH 11/11] emacs: Support cid: references with shr renderer [inbox, unread]
      • Austin Clements—Re: [PATCH 11/11] emacs: Support cid: references with shr renderer [inbox, unread]
  • Tomi Ollila—Re: [PATCH 00/11] Improve charset and cid: handling [inbox, notmuch::review, unread]
  • Mark Walters—Re: [PATCH 00/11] Improve charset and cid: handling [inbox, unread]
    • Austin Clements—Re: [PATCH 00/11] Improve charset and cid: handling [inbox, unread]