Re: [PATCH v2 03/12] lib: Handle empty date value

Subject: Re: [PATCH v2 03/12] lib: Handle empty date value

Date: Sat, 11 Oct 2014 07:12:41 +0200

To: Austin Clements, notmuch@notmuchmail.org

Cc:

From: David Bremner

Austin Clements <aclements@csail.mit.edu> writes:

> From: Austin Clements <amdragon@mit.edu>
>
> In the interest of robustness, avoid undefined behavior of
> sortable_unserialise if the date value is missing.  This shouldn't
> happen now, but ghost messages will have blank date values.

Pushed the first 3 patches in the series.

d

• Previous message (by thread): Re: [PATCH v2 12/12] lib: Remove unnecessary thread linking steps when using ghost messages

Thread:

• Austin Clements—[PATCH 00/12] Add ghost messages and fix thread linking [inbox, unread]
  • Austin Clements—[PATCH v2 01/12] lib: Move message ID compression to _notmuch_message_create_for_message_id [inbox, notmuch::patch, notmuch::pushed, unread]
  • Austin Clements—[PATCH v2 02/12] lib: Refactor _notmuch_database_link_message [inbox, notmuch::patch, notmuch::pushed, unread]
  • Austin Clements—[PATCH v2 03/12] lib: Handle empty date value [inbox, notmuch::patch, notmuch::pushed, unread]
    • David Bremner—Re: [PATCH v2 03/12] lib: Handle empty date value [inbox, unread]
  • Austin Clements—[PATCH v2 04/12] lib: Add a ghost messages database feature [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH v2 05/12] lib: Update database schema doc for ghost messages [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH v2 06/12] lib: Introduce macros for bit operations [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH v2 07/12] lib: Internal support for querying and creating ghost messages [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Mark Walters—Re: [PATCH v2 07/12] lib: Internal support for querying and creating ghost messages [inbox, unread]
      • Austin Clements—Re: [PATCH v2 07/12] lib: Internal support for querying and creating ghost messages [inbox, unread]
  • Austin Clements—[PATCH v2 08/12] lib: Implement ghost-based thread linking [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Mark Walters—Re: [PATCH v2 08/12] lib: Implement ghost-based thread linking [inbox, unread]
      • Austin Clements—Re: [PATCH v2 08/12] lib: Implement ghost-based thread linking [inbox, unread]
  • Austin Clements—[PATCH v2 09/12] lib: Implement upgrade to ghost messages feature [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH v2 10/12] lib: Enable ghost messages feature [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH v2 11/12] test: Test upgrade to ghost messages feature [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Austin Clements—[PATCH v2 12/12] lib: Remove unnecessary thread linking steps when using ghost messages [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Mark Walters—Re: [PATCH v2 12/12] lib: Remove unnecessary thread linking steps when using ghost messages [inbox, unread]
      • Austin Clements—Re: [PATCH v2 12/12] lib: Remove unnecessary thread linking steps when using ghost messages [inbox, unread]
  • Mark Walters—Re: [PATCH 00/12] Add ghost messages and fix thread linking [inbox, unread]