Re: [PATCH 2/2] lib/message-property: sync removed properties to the database

Subject: Re: [PATCH 2/2] lib/message-property: sync removed properties to the database

Date: Wed, 29 Mar 2023 08:33:12 -0300

To: Kevin Boulain, Tomi Ollila

Cc: notmuch@notmuchmail.org

From: David Bremner

Kevin Boulain <kevin@boula.in> writes:

>
> Yeah, talloc's documentation confirms that when the context is free'd
> every child is also free'd. Not quite sure what style is preferred
> (explicit or implicit, implicit sure leads to nicer code here).

In general implicit de-allocation is fine. 
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

• Previous message (by thread): Re: [PATCH v2 5/5] test: add test for notmuch_message_remove_all_properties_with_prefix

Thread:

• Kevin Boulain—[PATCH 1/2] test: display key name in property tests [inbox, notmuch::obsolete, notmuch::patch, notmuch::pushed, unread]
  • Kevin Boulain—[PATCH 2/2] lib/message-property: sync removed properties to the database [inbox, notmuch::obsolete, notmuch::patch, notmuch::pushed, unread]
    • Tomi Ollila—Re: [PATCH 2/2] lib/message-property: sync removed properties to the database [inbox, unread]
      • Kevin Boulain—Re: [PATCH 2/2] lib/message-property: sync removed properties to the database [inbox, unread]
        • David Bremner—Re: [PATCH 2/2] lib/message-property: sync removed properties to the database [inbox, unread]
          • Kevin Boulain—[PATCH v2 1/5] test: display key name in property tests [inbox, notmuch::patch, notmuch::pushed, unread]
          • Kevin Boulain—[PATCH v2 2/5] test: remove unnecessary sorting [inbox, notmuch::patch, notmuch::pushed, unread]
          • Kevin Boulain—[PATCH v2 3/5] test: reorganize tests and mark a few of them as broken [inbox, notmuch::patch, notmuch::pushed, unread]
          • Kevin Boulain—[PATCH v2 4/5] lib/message-property: sync removed properties to the database [inbox, notmuch::patch, notmuch::pushed, unread]
          • Kevin Boulain—[PATCH v2 5/5] test: add test for notmuch_message_remove_all_properties_with_prefix [inbox, notmuch::patch, notmuch::pushed, unread]
            • David Bremner—Re: [PATCH v2 5/5] test: add test for notmuch_message_remove_all_properties_with_prefix [inbox, unread]
          • Kevin Boulain—Re: [PATCH 2/2] lib/message-property: sync removed properties to the database [inbox, unread]
        • David Bremner—Re: [PATCH 2/2] lib/message-property: sync removed properties to the database [inbox, unread]