Re: [PATCH 4/7] util/crypto: _n_m_crypto_potential_payload returns whether part is the payload

Subject: Re: [PATCH 4/7] util/crypto: _n_m_crypto_potential_payload returns whether part is the payload

Date: Thu, 01 Aug 2019 10:10:42 -0400

To: David Bremner, Notmuch Mail

Cc:

From: Daniel Kahn Gillmor

On Thu 2019-08-01 08:32:56 -0300, David Bremner wrote:
> assert is OK, but INTERNAL_ERROR is definded in util/error_util.c, so
> including that header is another sensible option.

gotcha, i think i was doing these updates with not-enough-sleep. I've
gone back to INTERNAL_ERROR for my internal branch now. :)

      --dkg

_______________________________________________
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch

• Previous message (by thread): Re: [PATCH 5/7] util/repair: add _notmuch_repair_crypto_payload_skip_legacy_display

Thread:

• Daniel Kahn Gillmor—v3 of legacy-display cleanup [inbox, unread]
  • Daniel Kahn Gillmor—[PATCH 1/7] mime-node: split out _mime_node_set_up_part [inbox, notmuch::obsolete, notmuch::patch, unread]
    • William Casarin—Re: [PATCH 1/7] mime-node: split out _mime_node_set_up_part [inbox, unread]
      • Daniel Kahn Gillmor—Re: [PATCH 1/7] mime-node: split out _mime_node_set_up_part [inbox, signed, unread]
  • Daniel Kahn Gillmor—[PATCH 2/7] repair: set up codebase for repair functionality [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Daniel Kahn Gillmor—[PATCH 3/7] test: avoid showing legacy-display parts [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Daniel Kahn Gillmor—[PATCH 4/7] util/crypto: _n_m_crypto_potential_payload returns whether part is the payload [inbox, notmuch::obsolete, notmuch::patch, unread]
    • David Bremner—Re: [PATCH 4/7] util/crypto: _n_m_crypto_potential_payload returns whether part is the payload [inbox, unread]
      • Daniel Kahn Gillmor—Re: [PATCH 4/7] util/crypto: _n_m_crypto_potential_payload returns whether part is the payload [inbox, signed, unread]
        • Daniel Kahn Gillmor—Re: [PATCH 4/7] util/crypto: _n_m_crypto_potential_payload returns whether part is the payload [inbox, signed, unread]
          • David Bremner—Re: [PATCH 4/7] util/crypto: _n_m_crypto_potential_payload returns whether part is the payload [inbox, unread]
            • Daniel Kahn Gillmor—Re: [PATCH 4/7] util/crypto: _n_m_crypto_potential_payload returns whether part is the payload [inbox, signed, unread]
        • David Bremner—Re: [PATCH 4/7] util/crypto: _n_m_crypto_potential_payload returns whether part is the payload [inbox, unread]
  • Daniel Kahn Gillmor—[PATCH 5/7] util/repair: add _notmuch_repair_crypto_payload_skip_legacy_display [inbox, notmuch::obsolete, notmuch::patch, unread]
    • William Casarin—Re: [PATCH 5/7] util/repair: add _notmuch_repair_crypto_payload_skip_legacy_display [inbox, unread]
      • Daniel Kahn Gillmor—Re: [PATCH 5/7] util/repair: add _notmuch_repair_crypto_payload_skip_legacy_display [inbox, signed, unread]
    • David Bremner—Re: [PATCH 5/7] util/repair: add _notmuch_repair_crypto_payload_skip_legacy_display [inbox, unread]
      • Daniel Kahn Gillmor—Re: [PATCH 5/7] util/repair: add _notmuch_repair_crypto_payload_skip_legacy_display [inbox, signed, unread]
  • Daniel Kahn Gillmor—[PATCH 6/7] cli/{show,reply}: skip over legacy-display parts [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Daniel Kahn Gillmor—[PATCH 7/7] index: avoid indexing legacy-display parts [inbox, notmuch::obsolete, notmuch::patch, unread]
  • William Casarin—Re: v3 of legacy-display cleanup [inbox, unread]