On Thu, Jun 09, 2016 at 08:09:37PM -0400, Damien Riegel wrote:
> diff -r 9556f8bbb5a9 -r 9eebf8f22c4d mx.c
> --- a/mx.c    Tue Jun 07 18:34:14 2016 -0400
> +++ b/mx.c    Tue Jun 07 18:41:40 2016 -0400
> @@ -1332,8 +1332,12 @@
>  
>  int mx_commit_message (MESSAGE *msg, CONTEXT *ctx)

> +  r = ops->commit_msg (ctx, msg);
>
>    if (r == 0 && (ctx->magic == MUTT_MBOX || ctx->magic == MUTT_MMDF)
>        && (fflush (msg->fp) == EOF || fsync (fileno (msg->fp)) == -1))
>    {
>      mutt_perror _("Can't write message");
>      r = -1;
>    }

I think it would be good to move the fflush and fsync inside
mmdf_commit_message() and mbox_commit_message().

-- 
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA

Attachment: signature.asc
Description: PGP signature

Reply via email to