Looks good to me, but it appears that both callers of
notmuch_message_tags_to_maildir_flags ignore the return value.  Both
callers synchronize maildir flags immediately after thawing the tag
changes on a message.  Perhaps they should instead synchronize
*before* thawing and abort if the sync fails?

On Tue, Feb 15, 2011 at 1:07 AM, Rob Browning <rlb at defaultvalue.org> wrote:
> Signed-off-by: Rob Browning <rlb at defaultvalue.org>
> ---
> ?lib/message.cc | ? ?2 +-
> ?1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/lib/message.cc b/lib/message.cc
> index 0590f76..979fad5 100644
> --- a/lib/message.cc
> +++ b/lib/message.cc
> @@ -1252,7 +1252,7 @@ notmuch_message_tags_to_maildir_flags 
> (notmuch_message_t *message)
> ? ? talloc_free (to_set);
> ? ? talloc_free (to_clear);
>
> - ? ?return NOTMUCH_STATUS_SUCCESS;
> + ? ?return status;
> ?}
>
> ?notmuch_status_t
> --
> 1.7.2.3
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
>

Reply via email to