Re: [PATCH 1/3] new: Do not defer maildir flag synchronization during the first run

Subject: Re: [PATCH 1/3] new: Do not defer maildir flag synchronization during the first run

Date: Thu, 27 Jan 2011 19:08:38 -0500

To: Carl Worth

Cc: notmuch@notmuchmail.org

From: Austin Clements


Looks like eagerly synchronizing tags is easy and works fine, though I need
to beef up the tests and put some transactions around things before I'm
satisfied.

I added a "notmuch_database_remove_message_get" to the public API that's
just like "notmuch_database_remove_message" except that it also returns a
notmuch_message_t if other instances of the message still exist.  It feels
clunky to have two almost identical variants of this function.  Is this the
preferred way to change the public API?  Or should I just add the argument
to the existing function and fix the other three calls to it?
On Thu, Jan 27, 2011 at 12:43 AM, Austin Clements <amdragon@mit.edu> wrote:

> Sure. I've been wanting to take a crack at notmuch new's atomicity for a
> while. Though you'll have to get through some of my outstanding patches. I
> can only keep so many branches in my head. ]:--8)
>

Thread: