[PATCH v4 7/7] complete ghost-on-removal-when-shared-thread-exists

David Bremner david at tethera.net
Sat Apr 9 04:31:47 PDT 2016


Daniel Kahn Gillmor <dkg at fifthhorseman.net> writes:


> +		status = _notmuch_message_delete (message);
> +		if (status) /* we'll report the last failure we see;
> +			     * if there is more than one failure, we
> +			     * forget about previous ones */
> +		    last_error = status;

I was initially worried/paranoid that there might be some risk of data
loss by continuing deleting after the first bad status; that doesn't
seem to be the case, but there doesn't seem to be much advantage in
continuing either, since the only error currently returned from
_notmuch_message_delete is from _notmuch_database_ensure_writable, which
seems likely to persist.  So perhaps exiting the loop on the first error
might be less confusing.

Other than that, and my bug in ghost-report, the series looks good to
me.

d



More information about the notmuch mailing list