use more ad-hoc list handling function, and avoid to muck with ->next
[apps/madmutt.git] / commands.c
index a14b471..bc05999 100644 (file)
 # include "config.h"
 #endif
 
+#include <errno.h>
+#include <unistd.h>
+#include <stdlib.h>
+#include <string.h>
+#include <sys/wait.h>
+#include <sys/stat.h>
+#include <sys/types.h>
+#include <utime.h>
+
 #include <lib-lib/mem.h>
 #include <lib-lib/file.h>
 #include <lib-lib/macros.h>
 
 #include <lib-mime/mime.h>
 
+#include <lib-sys/exit.h>
+#include <lib-sys/unix.h>
+
+#include <lib-ui/curses.h>
+#include <lib-ui/enter.h>
+#include <lib-ui/menu.h>
+
 #include "mutt.h"
-#include "enter.h"
+#include "alias.h"
 #include "recvattach.h"
-#include "mutt_curses.h"
-#include "mutt_menu.h"
 #include "sort.h"
 #include "copy.h"
 #include "mx.h"
 #include "pager.h"
-#include "mutt_crypt.h"
+#include <lib-crypt/crypt.h>
 #include "mutt_idna.h"
-#include "rfc1524.h"
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <fcntl.h>
 
-#ifdef USE_IMAP
-#include "imap.h"
-#endif
+#include <imap/imap.h>
 
 #ifdef BUFFY_SIZE
 #include "buffy.h"
 #endif
 
-#include "lib/debug.h"
-
-#include <errno.h>
-#include <unistd.h>
-#include <stdlib.h>
-#include <string.h>
-#include <sys/wait.h>
-#include <sys/stat.h>
-#include <sys/types.h>
-#include <utime.h>
-
 /* The folder the user last saved to.  Used by ci_save_message() */
 static char LastSaveFolder[_POSIX_PATH_MAX] = "";
 
@@ -108,7 +108,7 @@ int mutt_display_message (HEADER * cur)
 
   if (res != -1) {
     /* see if crytpo is needed for this message.  if so, we should exit curses */
-    if (WithCrypto && cur->security) {
+    if (cur->security) {
       if (cur->security & ENCRYPT) {
         if (cur->security & APPLICATION_SMIME)
           crypt_smime_getkeys (cur->env);
@@ -142,7 +142,6 @@ int mutt_display_message (HEADER * cur)
                              (option (OPTWEED) ? (CH_WEED | CH_REORDER) : 0) |
                              CH_DECODE | CH_FROM);
     if (res == 0 && (ferror(fpout) || feof(fpout))) {
-      debug_print (1, ("_mutt_copy_message failed to detect EOF!\n"));
       res = -1;
     }
 
@@ -173,21 +172,18 @@ int mutt_display_message (HEADER * cur)
   safe_fclose (&fpfilterout);   /* XXX - check result? */
 
 
-  if (WithCrypto) {
-    /* update crypto information for this message */
-    cur->security &= ~(GOODSIGN|BADSIGN);
-    cur->security |= crypt_query (cur->content);
+  /* update crypto information for this message */
+  cur->security &= ~(GOODSIGN|BADSIGN);
+  cur->security |= crypt_query (cur->content);
 
-    /* Remove color cache for this message, in case there
-       are color patterns for both ~g and ~V */
-    cur->pair = 0;
-  }
+  /* Remove color cache for this message, in case there
+     are color patterns for both ~g and ~V */
+  cur->pair = 0;
 
   if (builtin) {
     pager_t info;
 
-    if (WithCrypto
-        && (cur->security & APPLICATION_SMIME) && (cmflags & M_CM_VERIFY)) {
+    if ((cur->security & APPLICATION_SMIME) && (cmflags & M_CM_VERIFY)) {
       if (cur->security & GOODSIGN) {
         if (!crypt_smime_verify_sender (cur))
           mutt_message (_("S/MIME signature successfully verified."));
@@ -201,8 +197,7 @@ int mutt_display_message (HEADER * cur)
         mutt_error (_("S/MIME signature could NOT be verified."));
     }
 
-    if (WithCrypto
-        && (cur->security & APPLICATION_PGP) && (cmflags & M_CM_VERIFY)) {
+    if ((cur->security & APPLICATION_PGP) && (cmflags & M_CM_VERIFY)) {
       if (cur->security & GOODSIGN)
         mutt_message (_("PGP signature successfully verified."));
       else if (cur->security & PARTSIGN)
@@ -245,7 +240,7 @@ void ci_bounce_message (HEADER * h, int *redraw)
   char prompt[SHORT_STRING];
   char buf[HUGE_STRING] = { 0 };
   address_t *adr = NULL;
-  char *err = NULL;
+  const char *err = NULL;
   int rc;
 
   if (h)
@@ -274,7 +269,7 @@ void ci_bounce_message (HEADER * h, int *redraw)
   if (mutt_addrlist_to_idna (adr, &err) < 0) {
     mutt_error (_("Bad IDN: '%s'"), err);
     p_delete(&err);
-    address_delete (&adr);
+    address_list_wipe(&adr);
     return;
   }
 
@@ -295,7 +290,7 @@ void ci_bounce_message (HEADER * h, int *redraw)
   }
 
   if (query_quadoption (OPT_BOUNCE, prompt) != M_YES) {
-    address_delete (&adr);
+    address_list_wipe(&adr);
     CLEARLINE (LINES - 1);
     mutt_message (h ? _("Message not bounced.") : _("Messages not bounced."));
     return;
@@ -304,7 +299,7 @@ void ci_bounce_message (HEADER * h, int *redraw)
   CLEARLINE (LINES - 1);
 
   rc = mutt_bounce_message (NULL, h, adr);
-  address_delete (&adr);
+  address_list_wipe(&adr);
   /* If no error, or background, display message. */
   if ((rc == 0) || (rc == S_BKG))
     mutt_message (h ? _("Message bounced.") : _("Messages bounced."));
@@ -334,7 +329,7 @@ static void pipe_msg (HEADER * h, FILE * fp, int decode, int print)
 
   pipe_set_flags (decode, print, &cmflags, &chflags);
 
-  if (WithCrypto && decode && h->security & ENCRYPT) {
+  if (decode && h->security & ENCRYPT) {
     if (!crypt_valid_passphrase (h->security))
       return;
     endwin ();
@@ -367,7 +362,7 @@ static int _mutt_pipe_message(HEADER * h, char *cmd, int decode, int print,
 
     mutt_message_hook (Context, h, M_MESSAGEHOOK);
 
-    if (WithCrypto && decode) {
+    if (decode) {
       mutt_parse_mime_message (Context, h);
       if (h->security & ENCRYPT && !crypt_valid_passphrase (h->security))
         return 1;
@@ -386,7 +381,7 @@ static int _mutt_pipe_message(HEADER * h, char *cmd, int decode, int print,
   }
   else {                        /* handle tagged messages */
 
-    if (WithCrypto && decode) {
+    if (decode) {
       for (i = 0; i < Context->vcount; i++)
         if (Context->hdrs[Context->v2r[i]]->tagged) {
           mutt_message_hook (Context, Context->hdrs[Context->v2r[i]],
@@ -621,17 +616,14 @@ static void set_copy_flags (HEADER * hdr, int decode, int decrypt,
   *cmflags = 0;
   *chflags = CH_UPDATE_LEN;
 
-  if (WithCrypto && !decode && decrypt && (hdr->security & ENCRYPT)) {
-    if ((WithCrypto & APPLICATION_PGP)
-        && mutt_is_multipart_encrypted (hdr->content)) {
+  if (!decode && decrypt && (hdr->security & ENCRYPT)) {
+    if (mutt_is_multipart_encrypted (hdr->content)) {
       *chflags = CH_NONEWLINE | CH_XMIT | CH_MIME;
       *cmflags = M_CM_DECODE_PGP;
     }
-    else if ((WithCrypto & APPLICATION_PGP)
-             && mutt_is_application_pgp (hdr->content) & ENCRYPT)
+    else if (mutt_is_application_pgp (hdr->content) & ENCRYPT)
       decode = 1;
-    else if ((WithCrypto & APPLICATION_SMIME)
-             && mutt_is_application_smime (hdr->content) & ENCRYPT) {
+    else if (mutt_is_application_smime (hdr->content) & ENCRYPT) {
       *chflags = CH_NONEWLINE | CH_XMIT | CH_MIME;
       *cmflags = M_CM_DECODE_SMIME;
     }
@@ -702,10 +694,8 @@ int mutt_save_message (HEADER * h, int delete,
 
 
   if (h) {
-    if (WithCrypto) {
-      need_passphrase = h->security & ENCRYPT;
-      app = h->security;
-    }
+    need_passphrase = h->security & ENCRYPT;
+    app = h->security;
     mutt_message_hook (Context, h, M_MESSAGEHOOK);
     mutt_default_save (buf, sizeof (buf), h);
   }
@@ -723,10 +713,8 @@ int mutt_save_message (HEADER * h, int delete,
     if (h) {
       mutt_message_hook (Context, h, M_MESSAGEHOOK);
       mutt_default_save (buf, sizeof (buf), h);
-      if (WithCrypto) {
-        need_passphrase = h->security & ENCRYPT;
-        app = h->security;
-      }
+      need_passphrase = h->security & ENCRYPT;
+      app = h->security;
       h = NULL;
     }
   }
@@ -759,13 +747,12 @@ int mutt_save_message (HEADER * h, int delete,
   if (mutt_save_confirm (buf, &st) != 0)
     return -1;
 
-  if (WithCrypto && need_passphrase && (decode || decrypt)
+  if (need_passphrase && (decode || decrypt)
       && !crypt_valid_passphrase (app))
     return -1;
 
   mutt_message (_("Copying to %s..."), buf);
 
-#ifdef USE_IMAP
   if (Context->magic == M_IMAP && !(decode || decrypt) && mx_get_magic (buf) == M_IMAP) {
     switch (imap_copy_messages (Context, h, buf, delete)) {
       /* success */
@@ -780,7 +767,6 @@ int mutt_save_message (HEADER * h, int delete,
       return -1;
     }
   }
-#endif
 
   if (mx_open_mailbox (buf, M_APPEND, &ctx) != NULL) {
     if (h) {
@@ -840,7 +826,6 @@ void mutt_edit_content_type (HEADER * h, BODY * b, FILE * fp)
   char buf[LONG_STRING];
   char obuf[LONG_STRING];
   char tmp[STRING];
-  PARAMETER *p;
 
   char charset[STRING];
   char *cp;
@@ -854,7 +839,8 @@ void mutt_edit_content_type (HEADER * h, BODY * b, FILE * fp)
   snprintf (buf, sizeof (buf), "%s/%s", TYPE (b), b->subtype);
   m_strcpy(obuf, sizeof(obuf), buf);
   if (b->parameter) {
-    size_t l;
+    PARAMETER *p;
+    ssize_t l;
 
     for (p = b->parameter; p; p = p->next) {
       l = m_strlen(buf);
@@ -869,7 +855,7 @@ void mutt_edit_content_type (HEADER * h, BODY * b, FILE * fp)
     return;
 
   /* clean up previous junk */
-  mutt_free_parameter (&b->parameter);
+  parameter_list_wipe(&b->parameter);
   p_delete(&b->subtype);
 
   mutt_parse_content_type (buf, b);
@@ -906,17 +892,17 @@ void mutt_edit_content_type (HEADER * h, BODY * b, FILE * fp)
 
   b->force_charset |= charset_changed ? 1 : 0;
 
-  if (!is_multipart (b) && b->parts)
+  if (!is_multipart(b) && b->parts)
     mutt_free_body (&b->parts);
   if (!mutt_is_message_type (b->type, b->subtype) && b->hdr) {
     b->hdr->content = NULL;
-    mutt_free_header (&b->hdr);
+    header_delete(&b->hdr);
   }
 
-  if (fp && (is_multipart (b) || mutt_is_message_type (b->type, b->subtype)))
+  if (fp && (is_multipart(b) || mutt_is_message_type(b->type, b->subtype)))
     mutt_parse_part (fp, b);
 
-  if (WithCrypto && h) {
+  if (h) {
     if (h->content == b)
       h->security = 0;