X-Git-Url: http://git.madism.org/?p=apps%2Fmadmutt.git;a=blobdiff_plain;f=editmsg.c;h=41895aac8700c92f3646a451fca20b2223aabbb5;hp=a11a5e4b843b84bba1bd80c947ec5e93e88d24e8;hb=88d239144bf6f50ca1eda6db7742281f0ad0f97f;hpb=a264c7bd834b6a72c90e9f672a2e0fdb06c18083;ds=sidebyside diff --git a/editmsg.c b/editmsg.c index a11a5e4..41895aa 100644 --- a/editmsg.c +++ b/editmsg.c @@ -9,21 +9,12 @@ /* simple, editor-based message editing */ -#if HAVE_CONFIG_H -# include "config.h" -#endif +#include +#include +#include #include "mutt.h" #include "copy.h" -#include "mx.h" - -#include "lib/intl.h" -#include "lib/str.h" - -#include -#include - -#include /* * return value: @@ -37,7 +28,6 @@ static int edit_one_message (CONTEXT * ctx, HEADER * cur) { char tmp[_POSIX_PATH_MAX]; char buff[STRING]; - int omagic; int oerrno; int rc; @@ -53,16 +43,11 @@ static int edit_one_message (CONTEXT * ctx, HEADER * cur) struct stat sb; time_t mtime = 0; - size_t size = 0; + ssize_t size = 0; mutt_mktemp (tmp); - omagic = DefaultMagic; - DefaultMagic = M_MBOX; - - rc = (mx_open_mailbox (tmp, M_NEWFOLDER, &tmpctx) == NULL) ? -1 : 0; - - DefaultMagic = omagic; + rc = (mx_open_mailbox(tmp, M_NEWFOLDER, &tmpctx) == NULL) ? -1 : 0; if (rc == -1) { mutt_error (_("could not create temporary folder: %s"), strerror (errno)); @@ -70,8 +55,7 @@ static int edit_one_message (CONTEXT * ctx, HEADER * cur) } rc = mutt_append_message (&tmpctx, ctx, cur, 0, CH_NOLEN | - ((ctx->magic == M_MBOX - || ctx->magic == M_MMDF) ? 0 : CH_NOSTATUS)); + (ctx->magic == M_MBOX ? 0 : CH_NOSTATUS)); oerrno = errno; mx_close_mailbox (&tmpctx, NULL); @@ -95,13 +79,13 @@ static int edit_one_message (CONTEXT * ctx, HEADER * cur) * remove it, the message will grow by one line each time the user edits * the message. */ - if (sb.st_size != 0 && truncate (tmp, sb.st_size - 1) == -1) { + if (size != 0 && truncate (tmp, --size) == -1) { mutt_error (_("could not truncate temporary mail folder: %s"), strerror (errno)); goto bail; } - mutt_edit_file (NONULL (Editor), tmp); + mutt_edit_file(tmp); if ((rc = stat (tmp, &sb)) == -1) { mutt_error (_("Can't stat %s: %s"), tmp, strerror (errno)); @@ -133,14 +117,14 @@ static int edit_one_message (CONTEXT * ctx, HEADER * cur) } of = 0; - cf = ((tmpctx.magic == M_MBOX || tmpctx.magic == M_MMDF) ? 0 : CH_NOSTATUS); + cf = (tmpctx.magic == M_MBOX ? 0 : CH_NOSTATUS); if (fgets (buff, sizeof (buff), fp) && is_from (buff, NULL, 0, NULL)) { - if (tmpctx.magic == M_MBOX || tmpctx.magic == M_MMDF) + if (tmpctx.magic == M_MBOX) cf = CH_FROM | CH_FORCE_FROM; - } - else + } else { of = M_ADD_FROM; + } /* * XXX - we have to play games with the message flags to avoid @@ -174,8 +158,7 @@ static int edit_one_message (CONTEXT * ctx, HEADER * cur) mx_close_mailbox (&tmpctx, NULL); bail: - if (fp) - fclose (fp); + m_fclose(&fp); if (rc >= 0) unlink (tmp);