On Thu, Jun 09, 2016 at 08:09:36PM -0400, Damien Riegel wrote:
> diff -r f81e8aa12041 -r 9556f8bbb5a9 mx.c
> --- a/mx.c Tue Jun 07 18:32:31 2016 -0400
> +++ b/mx.c Tue Jun 07 18:34:14 2016 -0400
> @@ -1345,9 +1345,7 @@
> {
> case MUTT_MMDF:
> {
> - if (fputs (MMDF_SEP, msg->fp) == EOF)
> - r = -1;
> - break;
> + r = mmdf_commit_message (ctx, msg);This is a minor point, since you are about to get rid of the switch statement in the next patch, but you removed the 'break;' line too. Ordinarily, I'd say forget it, but I have a suggestion for the next patch in the series too. -- Kevin J. McCarthy GPG Fingerprint: 8975 A9B3 3AA3 7910 385C 5308 ADEF 7684 8031 6BDA
signature.asc
Description: PGP signature
