further pop_mx_ng work
[apps/madmutt.git] / pattern.c
index d388924..94c1f9c 100644 (file)
--- a/pattern.c
+++ b/pattern.c
  * please see the file GPL in the top level source directory.
  */
 
-#if HAVE_CONFIG_H
-# include "config.h"
-#endif
-
-#include <lib-lib/mem.h>
-#include <lib-lib/ascii.h>
-#include <lib-lib/str.h>
-#include <lib-lib/macros.h>
-#include <lib-lib/file.h>
-#include <lib-lib/buffer.h>
-#include <lib-lib/mapping.h>
+#include <lib-lib/lib-lib.h>
 
 #include <lib-mime/mime.h>
+#include <lib-ui/lib-ui.h>
+#include <lib-mx/mx.h>
+#include <imap/imap.h>
 
-#include "mutt.h"
+#include "pattern.h"
+#include "alias.h"
+#include "crypt.h"
 #include "handler.h"
-#include "enter.h"
-#include "mx.h"
 #include "keymap.h"
 #include "copy.h"
 
-
-#ifdef USE_IMAP
-#include "mx.h"
-#include "imap/imap.h"
-#endif
-
-#include <string.h>
-#include <stdlib.h>
-#include <ctype.h>
-#include <sys/stat.h>
-#include <unistd.h>
-#include <stdarg.h>
-
-#include "mutt_crypt.h"
-
 static int eat_regexp (pattern_t * pat, BUFFER *, BUFFER *);
 static int eat_date (pattern_t * pat, BUFFER *, BUFFER *);
 static int eat_range (pattern_t * pat, BUFFER *, BUFFER *);
 static int patmatch (const pattern_t* pat, const char* buf);
 
-struct pattern_flags {
+static struct pattern_flags {
   int tag;                      /* character used to represent this op */
   int op;                       /* operation to perform */
   int class;
-  int (*eat_arg) (pattern_t *, BUFFER *, BUFFER *);
-} Flags[] = {
-  {
-  'A', M_ALL, 0, NULL}, {
-  'b', M_BODY, M_FULL_MSG, eat_regexp}, {
-  'B', M_WHOLE_MSG, M_FULL_MSG, eat_regexp}, {
-  'c', M_CC, 0, eat_regexp}, {
-  'C', M_RECIPIENT, 0, eat_regexp}, {
-  'd', M_DATE, 0, eat_date}, {
-  'D', M_DELETED, 0, NULL}, {
-  'e', M_SENDER, 0, eat_regexp}, {
-  'E', M_EXPIRED, 0, NULL}, {
-  'f', M_FROM, 0, eat_regexp}, {
-  'F', M_FLAG, 0, NULL}, {
-  'g', M_CRYPT_SIGN, 0, NULL}, {
-  'G', M_CRYPT_ENCRYPT, 0, NULL}, {
-  'h', M_HEADER, M_FULL_MSG, eat_regexp}, {
-  'H', M_HORMEL, 0, eat_regexp}, {
-  'i', M_ID, 0, eat_regexp}, {
-  'k', M_PGP_KEY, 0, NULL}, {
-  'L', M_ADDRESS, 0, eat_regexp}, {
-  'l', M_LIST, 0, NULL}, {
-  'm', M_MESSAGE, 0, eat_range}, {
-  'M', M_MULTIPART, 0, NULL}, {
-  'n', M_SCORE, 0, eat_range}, {
-  'N', M_NEW, 0, NULL}, {
-  'O', M_OLD, 0, NULL}, {
-  'p', M_PERSONAL_RECIP, 0, NULL}, {
-  'P', M_PERSONAL_FROM, 0, NULL}, {
-  'Q', M_REPLIED, 0, NULL}, {
-  'R', M_READ, 0, NULL}, {
-  'r', M_DATE_RECEIVED, 0, eat_date}, {
-  's', M_SUBJECT, 0, eat_regexp}, {
-  'S', M_SUPERSEDED, 0, NULL}, {
-  'T', M_TAG, 0, NULL}, {
-  't', M_TO, 0, eat_regexp}, {
-  'U', M_UNREAD, 0, NULL}, {
-  'u', M_SUBSCRIBED_LIST, 0, NULL}, {
-  'v', M_COLLAPSED, 0, NULL}, {
-  'V', M_CRYPT_VERIFIED, 0, NULL},
-#ifdef USE_NNTP
-  {
-  'w', M_NEWSGROUPS, 0, eat_regexp},
-#endif
-  {
-  'x', M_REFERENCE, 0, eat_regexp}, {
-  'X', M_MIMEATTACH, 0, eat_range}, {
-  'y', M_XLABEL, 0, eat_regexp}, {
-  'z', M_SIZE, 0, eat_range}, {
-  '=', M_DUPLICATED, 0, NULL}, {
-  '$', M_UNREFERENCED, 0, NULL}, {
-  '*', M_REALNAME, 0, NULL}, {
-  0, 0, 0, NULL}
+  int (*eat_arg)(pattern_t *, BUFFER *, BUFFER *);
+} const Flags[] = {
+  {'A', M_ALL, 0, NULL},
+  {'b', M_BODY, M_FULL_MSG, eat_regexp},
+  {'B', M_WHOLE_MSG, M_FULL_MSG, eat_regexp},
+  {'c', M_CC, 0, eat_regexp},
+  {'C', M_RECIPIENT, 0, eat_regexp},
+  {'d', M_DATE, 0, eat_date},
+  {'D', M_DELETED, 0, NULL},
+  {'e', M_SENDER, 0, eat_regexp},
+  {'E', M_EXPIRED, 0, NULL},
+  {'f', M_FROM, 0, eat_regexp},
+  {'F', M_FLAG, 0, NULL},
+  {'g', M_CRYPT_SIGN, 0, NULL},
+  {'G', M_CRYPT_ENCRYPT, 0, NULL},
+  {'h', M_HEADER, M_FULL_MSG, eat_regexp},
+  {'H', M_HORMEL, 0, eat_regexp},
+  {'i', M_ID, 0, eat_regexp},
+  {'k', M_PGP_KEY, 0, NULL},
+  {'L', M_ADDRESS, 0, eat_regexp},
+  {'l', M_LIST, 0, NULL},
+  {'m', M_MESSAGE, 0, eat_range},
+  {'M', M_MULTIPART, 0, NULL},
+  {'n', M_SCORE, 0, eat_range},
+  {'N', M_NEW, 0, NULL},
+  {'O', M_OLD, 0, NULL},
+  {'p', M_PERSONAL_RECIP, 0, NULL},
+  {'P', M_PERSONAL_FROM, 0, NULL},
+  {'Q', M_REPLIED, 0, NULL},
+  {'R', M_READ, 0, NULL},
+  {'r', M_DATE_RECEIVED, 0, eat_date},
+  {'s', M_SUBJECT, 0, eat_regexp},
+  {'S', M_SUPERSEDED, 0, NULL},
+  {'T', M_TAG, 0, NULL},
+  {'t', M_TO, 0, eat_regexp},
+  {'U', M_UNREAD, 0, NULL},
+  {'u', M_SUBSCRIBED_LIST, 0, NULL},
+  {'v', M_COLLAPSED, 0, NULL},
+  {'V', M_CRYPT_VERIFIED, 0, NULL},
+  {'x', M_REFERENCE, 0, eat_regexp},
+  {'X', M_MIMEATTACH, 0, eat_range},
+  {'y', M_XLABEL, 0, eat_regexp},
+  {'z', M_SIZE, 0, eat_range},
+  {'=', M_DUPLICATED, 0, NULL},
+  {'$', M_UNREFERENCED, 0, NULL},
+  {'*', M_REALNAME, 0, NULL},
+  {0, 0, 0, NULL}
 };
 
 static pattern_t *SearchPattern = NULL; /* current search pattern */
@@ -124,17 +96,14 @@ static char LastSearchExpn[LONG_STRING] = { 0 };        /* expanded version of
 #define M_PDR_ERROR    0x0100
 #define M_PDR_ERRORDONE        (M_PDR_ERROR | M_PDR_DONE)
 
-
-int mutt_getvaluebychar (char ch, struct mapping_t *table)
+void pattern_wipe(pattern_t *pat)
 {
-  int i;
-
-  for (i = 0; table[i].name; i++) {
-    if (ch == table[i].name[0])
-      return table[i].value;
-  }
-
-  return (-1);
+    if (pat->rx) {
+        regfree (pat->rx);
+        p_delete(&pat->rx);
+    }
+    p_delete(&pat->str);
+    pattern_list_wipe(&pat->child);
 }
 
 /* if no uppercase letters are given, do a case-insensitive search */
@@ -160,7 +129,7 @@ msg_search (CONTEXT *ctx, pattern_t* pat, int msgno)
   int match = 0;
   HEADER *h = ctx->hdrs[msgno];
   char* buf;
-  size_t blen;
+  ssize_t blen;
 
   if ((msg = mx_open_message (ctx, msgno)) != NULL) {
     if (option (OPTTHOROUGHSRC)) {
@@ -168,10 +137,10 @@ msg_search (CONTEXT *ctx, pattern_t* pat, int msgno)
       p_clear(&s, 1);
       s.fpin = msg->fp;
       s.flags = M_CHARCONV;
-      mutt_mktemp (tempfile);
-      if ((s.fpout = safe_fopen (tempfile, "w+")) == NULL) {
-        mutt_perror (tempfile);
-        return (0);
+      s.fpout = m_tempfile(tempfile, sizeof(tempfile), NONULL(mod_core.tmpdir), NULL);
+      if (!s.fpout) {
+        mutt_error(_("Could not create temporary file"));
+        return 0;
       }
 
       if (pat->op != M_BODY)
@@ -179,17 +148,6 @@ msg_search (CONTEXT *ctx, pattern_t* pat, int msgno)
 
       if (pat->op != M_HEADER) {
         mutt_parse_mime_message (ctx, h);
-
-        if (WithCrypto && (h->security & ENCRYPT)
-            && !crypt_valid_passphrase (h->security)) {
-          mx_close_message (&msg);
-          if (fp) {
-            fclose (fp);
-            unlink (tempfile);
-          }
-          return (0);
-        }
-
         fseeko (msg->fp, h->offset, 0);
         mutt_body_handler (h->content, &s);
       }
@@ -219,7 +177,7 @@ msg_search (CONTEXT *ctx, pattern_t* pat, int msgno)
     /* search the file "fp" */
     while (lng > 0) {
       if (pat->op == M_HEADER) {
-        if (*(buf = mutt_read_rfc822_line (fp, buf, &blen)) == '\0')
+        if (!mutt_read_rfc822_line(fp, &buf, &blen))
           break;
       } else if (fgets (buf, blen - 1, fp) == NULL)
         break;                  /* don't loop forever */
@@ -235,7 +193,7 @@ msg_search (CONTEXT *ctx, pattern_t* pat, int msgno)
     mx_close_message (&msg);
 
     if (option (OPTTHOROUGHSRC)) {
-      fclose (fp);
+      m_fclose(&fp);
       unlink (tempfile);
     }
   }
@@ -253,20 +211,14 @@ int eat_regexp (pattern_t * pat, BUFFER * s, BUFFER * err)
   if (mutt_extract_token (&buf, s, M_TOKEN_PATTERN | M_TOKEN_COMMENT) != 0 ||
       !buf.data) {
     snprintf (err->data, err->dsize, _("Error in expression: %s"), s->dptr);
-    return (-1);
+    return -1;
   }
 
   if (!*buf.data) {
     snprintf (err->data, err->dsize, _("Empty expression"));
-    return (-1);
+    return -1;
   }
 
-#if 0
- /* If there are no RE metacharacters, use simple search anyway */
-  if (!pat->stringmatch && !strpbrk (buf.data, "|[{.*+?^$"))
-    pat->stringmatch = 1;
-#endif
-
   if (pat->stringmatch) {
     pat->str = m_strdup(buf.data);
     p_delete(&buf.data);
@@ -278,7 +230,7 @@ int eat_regexp (pattern_t * pat, BUFFER * s, BUFFER * err)
       regerror (r, pat->rx, err->data, err->dsize);
       regfree (pat->rx);
       p_delete(&pat->rx);
-      return (-1);
+      return -1;
     }
   }
   return 0;
@@ -291,7 +243,7 @@ static int patmatch (const pattern_t* pat, const char* buf) {
     return regexec (pat->rx, buf, 0, NULL, 0);
 }
 
-int eat_range (pattern_t * pat, BUFFER * s, BUFFER * err)
+int eat_range (pattern_t * pat, BUFFER * s, BUFFER * err __attribute__ ((unused)))
 {
   char *tmp;
   int do_exclusive = 0;
@@ -428,8 +380,8 @@ static const char *get_offset (struct tm *tm, const char *s, int sign)
   default:
     return s;
   }
-  mutt_normalize_time (tm);
-  return (ps + 1);
+  mktime(tm);
+  return ps + 1;
 }
 
 static void adjust_date_range (struct tm *min, struct tm *max)
@@ -529,7 +481,7 @@ static const char *parse_date_range (const char *pc, struct tm *min,
   if ((flag & M_PDR_ERROR) && !(flag & M_PDR_ABSOLUTE)) {       /* getDate has its own error message, don't overwrite it here */
     snprintf (err->data, err->dsize, _("Invalid relative date: %s"), pc - 1);
   }
-  return ((flag & M_PDR_ERROR) ? NULL : pc);
+  return (flag & M_PDR_ERROR) ? NULL : pc;
 }
 
 static int eat_date (pattern_t * pat, BUFFER * s, BUFFER * err)
@@ -541,7 +493,7 @@ static int eat_date (pattern_t * pat, BUFFER * s, BUFFER * err)
   if (mutt_extract_token (&buffer, s, M_TOKEN_COMMENT | M_TOKEN_PATTERN) != 0
       || !buffer.data) {
     m_strcpy(err->data, err->dsize, _("error in expression"));
-    return (-1);
+    return -1;
   }
 
   p_clear(&min, 1);
@@ -605,7 +557,7 @@ static int eat_date (pattern_t * pat, BUFFER * s, BUFFER * err)
       /* mininum date specified */
       if ((pc = getDate (pc, &min, err)) == NULL) {
         p_delete(&buffer.data);
-        return (-1);
+        return -1;
       }
       haveMin = TRUE;
       pc = vskipspaces(pc);
@@ -638,7 +590,7 @@ static int eat_date (pattern_t * pat, BUFFER * s, BUFFER * err)
 
       if (!parse_date_range (pc, &min, &max, haveMin, &baseMin, err)) { /* bail out on any parsing error */
         p_delete(&buffer.data);
-        return (-1);
+        return -1;
       }
     }
   }
@@ -646,60 +598,37 @@ static int eat_date (pattern_t * pat, BUFFER * s, BUFFER * err)
   /* Since we allow two dates to be specified we'll have to adjust that. */
   adjust_date_range (&min, &max);
 
-  pat->min = mutt_mktime (&min, 1);
-  pat->max = mutt_mktime (&max, 1);
+  pat->min = mktime(&min);
+  pat->max = mktime(&max);
 
   p_delete(&buffer.data);
 
   return 0;
 }
 
-static struct pattern_flags *lookup_tag (char tag)
+static const struct pattern_flags *lookup_tag (char tag)
 {
   int i;
 
   for (i = 0; Flags[i].tag; i++)
     if (Flags[i].tag == tag)
-      return (&Flags[i]);
+      return &Flags[i];
   return NULL;
 }
 
-static /* const */ char *find_matching_paren ( /* const */ char *s)
-{
-  int level = 1;
-
-  for (; *s; s++) {
-    if (*s == '(')
-      level++;
-    else if (*s == ')') {
-      level--;
-      if (!level)
-        break;
-    }
-  }
-  return s;
-}
-
-void mutt_pattern_free (pattern_t ** pat)
+static const char *find_matching_paren (const char *s)
 {
-  pattern_t *tmp;
-
-  while (*pat) {
-    tmp = *pat;
-    *pat = (*pat)->next;
+    int level = 1;
 
-    if (tmp->rx) {
-      regfree (tmp->rx);
-      p_delete(&tmp->rx);
+    for (; *s; s++) {
+        level += (*s == '(') - (*s == ')');
+        if (!level)
+            break;
     }
-    p_delete(&tmp->str);
-    if (tmp->child)
-      mutt_pattern_free (&tmp->child);
-    p_delete(&tmp);
-  }
+    return s;
 }
 
-pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
+pattern_t *mutt_pattern_comp(const char *s, int flags, BUFFER *err)
 {
   pattern_t *curlist = NULL;
   pattern_t *tmp;
@@ -708,13 +637,13 @@ pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
   int alladdr = 0;
   int or = 0;
   int implicit = 1;             /* used to detect logical AND operator */
-  struct pattern_flags *entry;
+  const struct pattern_flags *entry;
   char *p;
   char *buf;
   BUFFER ps;
 
   p_clear(&ps, 1);
-  ps.dptr = s;
+  ps.dptr  = s;
   ps.dsize = m_strlen(s);
 
   while (*ps.dptr) {
@@ -737,7 +666,7 @@ pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
         }
         if (curlist->next) {
           /* A & B | C == (A & B) | C */
-          tmp = new_pattern ();
+          tmp = pattern_new();
           tmp->op = M_AND;
           tmp->child = curlist;
 
@@ -757,7 +686,7 @@ pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
     case '~':
       if (implicit && or) {
         /* A | B & C == (A | B) & C */
-        tmp = new_pattern ();
+        tmp = pattern_new();
         tmp->op = M_OR;
         tmp->child = curlist;
         curlist = tmp;
@@ -765,7 +694,7 @@ pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
         or = 0;
       }
 
-      tmp = new_pattern ();
+      tmp = pattern_new();
       tmp->not = not;
       tmp->alladdr = alladdr;
       tmp->stringmatch = (*ps.dptr == '=') ? 1 : 0;
@@ -781,13 +710,13 @@ pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
       ps.dptr++;                /* move past the ~ */
       if ((entry = lookup_tag (*ps.dptr)) == NULL) {
         snprintf (err->data, err->dsize, _("%c: invalid command"), *ps.dptr);
-        mutt_pattern_free (&curlist);
+        pattern_list_wipe(&curlist);
         return NULL;
       }
       if (entry->class && (flags & entry->class) == 0) {
         snprintf (err->data, err->dsize, _("%c: not supported in this mode"),
                   *ps.dptr);
-        mutt_pattern_free (&curlist);
+        pattern_list_wipe(&curlist);
         return NULL;
       }
       tmp->op = entry->op;
@@ -797,11 +726,11 @@ pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
       if (entry->eat_arg) {
         if (!*ps.dptr) {
           snprintf (err->data, err->dsize, _("missing parameter"));
-          mutt_pattern_free (&curlist);
+          pattern_list_wipe(&curlist);
           return NULL;
         }
         if (entry->eat_arg (tmp, &ps, err) == -1) {
-          mutt_pattern_free (&curlist);
+          pattern_list_wipe(&curlist);
           return NULL;
         }
       }
@@ -812,14 +741,14 @@ pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
       if (*p != ')') {
         snprintf (err->data, err->dsize, _("mismatched parenthesis: %s"),
                   ps.dptr);
-        mutt_pattern_free (&curlist);
+        pattern_list_wipe(&curlist);
         return NULL;
       }
       /* compile the sub-expression */
       buf = p_dupstr(ps.dptr + 1, p - ps.dptr - 1);
       if ((tmp = mutt_pattern_comp (buf, flags, err)) == NULL) {
         p_delete(&buf);
-        mutt_pattern_free (&curlist);
+        pattern_list_wipe(&curlist);
         return NULL;
       }
       p_delete(&buf);
@@ -836,7 +765,7 @@ pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
       break;
     default:
       snprintf (err->data, err->dsize, _("error in pattern at: %s"), ps.dptr);
-      mutt_pattern_free (&curlist);
+      pattern_list_wipe(&curlist);
       return NULL;
     }
   }
@@ -845,12 +774,12 @@ pattern_t *mutt_pattern_comp ( /* const */ char *s, int flags, BUFFER * err)
     return NULL;
   }
   if (curlist->next) {
-    tmp = new_pattern ();
+    tmp = pattern_new();
     tmp->op = or ? M_OR : M_AND;
     tmp->child = curlist;
     curlist = tmp;
   }
-  return (curlist);
+  return curlist;
 }
 
 static int
@@ -873,7 +802,7 @@ perform_or (struct pattern_t *pat, pattern_exec_flag flags, CONTEXT * ctx,
   return 0;
 }
 
-static int match_adrlist (pattern_t* pat, int match_personal, int alladdr,
+static int match_adrlist (pattern_t* pat, int match_personal, int alladdr __attribute__ ((unused)),
                           int n, ...)
 {
   va_list ap;
@@ -887,7 +816,7 @@ static int match_adrlist (pattern_t* pat, int match_personal, int alladdr,
            (match_personal && a->personal &&
             patmatch (pat, a->personal) == 0))) {
         va_end (ap);
-        return (!pat->alladdr);      /* Found match, or non-match if alladdr */
+        return !pat->alladdr;      /* Found match, or non-match if alladdr */
       }
     }
   }
@@ -895,7 +824,7 @@ static int match_adrlist (pattern_t* pat, int match_personal, int alladdr,
   return pat->alladdr;               /* No matches, or all matches if alladdr */
 }
 
-static int match_reference (pattern_t* pat, LIST * refs)
+static int match_reference (pattern_t* pat, string_list_t * refs)
 {
   for (; refs; refs = refs->next)
     if (patmatch (pat, refs->data) == 0)
@@ -907,10 +836,10 @@ int mutt_is_list_recipient (int alladdr, address_t * a1, address_t * a2)
 {
   for (; a1; a1 = a1->next)
     if (alladdr ^ mutt_is_subscribed_list (a1))
-      return (!alladdr);
+      return !alladdr;
   for (; a2; a2 = a2->next)
     if (alladdr ^ mutt_is_subscribed_list (a2))
-      return (!alladdr);
+      return !alladdr;
   return alladdr;
 }
 
@@ -918,10 +847,10 @@ int mutt_is_list_cc (int alladdr, address_t * a1, address_t * a2)
 {
   for (; a1; a1 = a1->next)
     if (alladdr ^ mutt_is_mail_list (a1))
-      return (!alladdr);
+      return !alladdr;
   for (; a2; a2 = a2->next)
     if (alladdr ^ mutt_is_mail_list (a2))
-      return (!alladdr);
+      return !alladdr;
   return alladdr;
 }
 
@@ -929,10 +858,10 @@ static int match_user (int alladdr, address_t * a1, address_t * a2)
 {
   for (; a1; a1 = a1->next)
     if (alladdr ^ mutt_addr_is_user (a1))
-      return (!alladdr);
+      return !alladdr;
   for (; a2; a2 = a2->next)
     if (alladdr ^ mutt_addr_is_user (a2))
-      return (!alladdr);
+      return !alladdr;
   return alladdr;
 }
 
@@ -952,7 +881,7 @@ static int valid_realname (const char *name)
       ret--;
     p++;
   }
-  return (ret >= 1);
+  return ret >= 1;
 }
 
 /* flags
@@ -963,31 +892,31 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags,
 {
   switch (pat->op) {
   case M_AND:
-    return (pat->not ^ (perform_and (pat->child, flags, ctx, h) > 0));
+    return pat->not ^ (perform_and (pat->child, flags, ctx, h) > 0);
   case M_OR:
-    return (pat->not ^ (perform_or (pat->child, flags, ctx, h) > 0));
+    return pat->not ^ (perform_or (pat->child, flags, ctx, h) > 0);
   case M_ALL:
-    return (!pat->not);
+    return !pat->not;
   case M_EXPIRED:
-    return (pat->not ^ h->expired);
+    return pat->not ^ h->expired;
   case M_SUPERSEDED:
-    return (pat->not ^ h->superseded);
+    return pat->not ^ h->superseded;
   case M_FLAG:
-    return (pat->not ^ h->flagged);
+    return pat->not ^ h->flagged;
   case M_TAG:
-    return (pat->not ^ h->tagged);
+    return pat->not ^ h->tagged;
   case M_NEW:
-    return (pat->not ? h->old || h->read : !(h->old || h->read));
+    return pat->not ? h->old || h->read : !(h->old || h->read);
   case M_UNREAD:
-    return (pat->not ? h->read : !h->read);
+    return pat->not ? h->read : !h->read;
   case M_REPLIED:
-    return (pat->not ^ h->replied);
+    return pat->not ^ h->replied;
   case M_OLD:
-    return (pat->not ? (!h->old || h->read) : (h->old && !h->read));
+    return pat->not ? (!h->old || h->read) : (h->old && !h->read);
   case M_READ:
-    return (pat->not ^ h->read);
+    return pat->not ^ h->read;
   case M_DELETED:
-    return (pat->not ^ h->deleted);
+    return pat->not ^ h->deleted;
   case M_MESSAGE:
     return (pat->not ^ (h->msgno >= pat->min - 1 && (pat->max == M_MAXRANGE ||
                                                      h->msgno <=
@@ -996,16 +925,14 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags,
     return (pat->
             not ^ (h->date_sent >= pat->min && h->date_sent <= pat->max));
   case M_DATE_RECEIVED:
-    return (pat->not ^ (h->received >= pat->min && h->received <= pat->max));
+    return pat->not ^ (h->received >= pat->min && h->received <= pat->max);
   case M_BODY:
   case M_HEADER:
   case M_WHOLE_MSG:
-#ifdef USE_IMAP
     /* IMAP search sets h->matched at search compile time */
     if (ctx->magic == M_IMAP && pat->stringmatch)
-      return (h->matched);
-#endif
-    return (pat->not ^ msg_search (ctx, pat, h->msgno));
+      return h->matched;
+    return pat->not ^ msg_search (ctx, pat, h->msgno);
   case M_SENDER:
     return (pat->not ^ match_adrlist (pat, flags & M_MATCH_FULL_ADDRESS,
                                       pat->alladdr, 1, h->env->sender));
@@ -1035,7 +962,7 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags,
                    && (pat->max == M_MAXRANGE
                        || h->content->length <= pat->max)));
   case M_REFERENCE:
-    return (pat->not ^ match_reference (pat, h->env->references));
+    return pat->not ^ match_reference (pat, h->env->references);
   case M_ADDRESS:
     return (pat->
             not ^ (h->env
@@ -1067,22 +994,14 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags,
     return (pat->
             not ^ (h->env && match_user (pat->alladdr, h->env->from, NULL)));
   case M_COLLAPSED:
-    return (pat->not ^ (h->collapsed && h->num_hidden > 1));
+    return pat->not ^ (h->collapsed && h->num_hidden > 1);
   case M_CRYPT_SIGN:
-    if (!WithCrypto)
-      break;
-    return (pat->not ^ ((h->security & SIGN) ? 1 : 0));
+    return pat->not ^ ((h->security & SIGN) ? 1 : 0);
   case M_CRYPT_VERIFIED:
-    if (!WithCrypto)
-      break;
-    return (pat->not ^ ((h->security & GOODSIGN) ? 1 : 0));
+    return pat->not ^ ((h->security & GOODSIGN) ? 1 : 0);
   case M_CRYPT_ENCRYPT:
-    if (!WithCrypto)
-      break;
-    return (pat->not ^ ((h->security & ENCRYPT) ? 1 : 0));
+    return pat->not ^ ((h->security & ENCRYPT) ? 1 : 0);
   case M_PGP_KEY:
-    if (!(WithCrypto & APPLICATION_PGP))
-      break;
     return (pat->not ^ ((h->security & APPLICATION_PGP)
                         && (h->security & PGPKEY)));
   case M_XLABEL:
@@ -1094,7 +1013,7 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags,
             not ^ (h->env->spam && h->env->spam->data
                    && patmatch (pat, h->env->spam->data) == 0));
   case M_DUPLICATED:
-    return (pat->not ^ (h->thread && h->thread->duplicate_thread));
+    return pat->not ^ (h->thread && h->thread->duplicate_thread);
 
   case M_MIMEATTACH:
     {
@@ -1105,7 +1024,7 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags,
       else {
         mutt_parse_mime_message(ctx, h);
         count = mutt_count_body_parts(h, 0);
-        mutt_free_body(&h->content->parts);
+        body_list_wipe(&h->content->parts);
       }
       
       return (pat->not ^ (count >= pat->min && (pat->max == M_MAXRANGE ||
@@ -1113,9 +1032,9 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags,
     }
 
   case M_UNREFERENCED:
-    return (pat->not ^ (h->thread && !h->thread->child));
+    return pat->not ^ (h->thread && !h->thread->child);
   case M_MULTIPART:
-    return (pat->not ^ (h->content && h->content->type == TYPEMULTIPART));
+    return pat->not ^ (h->content && h->content->type == TYPEMULTIPART);
   case M_REALNAME:
     /* realname filter:
      * we have a match if
@@ -1123,31 +1042,21 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags,
      * - or we have an alias for current address
      * - or From: contains valid email address _and_ name has >= 2 fields
      */
-    return (pat->
-            not ^ (h->env && h->env->from && (mutt_addr_is_user (h->env->from)
-                                              ||
-                                              (alias_reverse_lookup
-                                               (h->env->from) != NULL)
-                                              || (h->env->from->personal
-                                                  && valid_realname (h->env->
-                                                                     from->
-                                                                     personal)
-                                                  && h->env->from->mailbox)
-                   )));
-#ifdef USE_NNTP
-  case M_NEWSGROUPS:
-    return (pat->
-            not ^ (h->env->newsgroups
-                   && patmatch (pat, h->env->newsgroups) == 0));
-#endif
+    return (h->env && h->env->from
+        && (mutt_addr_is_user(h->env->from)
+            || alias_reverse_lookup(h->env->from)
+            || (h->env->from->personal
+                && valid_realname(h->env-> from->personal)
+                && h->env->from->mailbox)
+           )) ^ pat->not;
   }
   mutt_error (_("error: unknown op %d (report this error)."), pat->op);
-  return (-1);
+  return -1;
 }
 
-static void quote_simple (char *tmp, size_t len, const char *p)
+static void quote_simple (char *tmp, ssize_t len, const char *p)
 {
-  int i = 0;
+  ssize_t i = 0;
 
   tmp[i++] = '"';
   while (*p && i < len - 3) {
@@ -1160,7 +1069,7 @@ static void quote_simple (char *tmp, size_t len, const char *p)
 }
 
 /* convert a simple search into a real request */
-void mutt_check_simple (char *s, size_t len, const char *simple)
+void mutt_check_simple (char *s, ssize_t len, const char *simple)
 {
   char tmp[LONG_STRING];
 
@@ -1190,12 +1099,12 @@ void mutt_check_simple (char *s, size_t len, const char *simple)
       m_strcpy(s, len, "~U");
     else {
       quote_simple (tmp, sizeof (tmp), s);
-      mutt_expand_fmt (s, len, simple, tmp);
+      m_file_fmt(s, len, simple, tmp);
     }
   }
 }
 
-int mutt_pattern_func (int op, char *prompt)
+int mutt_pattern_func (int op, const char *prompt)
 {
   pattern_t *pat;
   char buf[LONG_STRING] = "", *simple, error[STRING];
@@ -1205,7 +1114,7 @@ int mutt_pattern_func (int op, char *prompt)
   m_strcpy(buf, sizeof(buf), NONULL(Context->pattern));
   if (prompt || op != M_LIMIT)
     if (mutt_get_field (prompt, buf, sizeof (buf), M_PATTERN | M_CLEAR) != 0 || !buf[0])
-      return (-1);
+      return -1;
 
   mutt_message _("Compiling search pattern...");
 
@@ -1217,13 +1126,11 @@ int mutt_pattern_func (int op, char *prompt)
   if ((pat = mutt_pattern_comp (buf, M_FULL_MSG, &err)) == NULL) {
     p_delete(&simple);
     mutt_error ("%s", err.data);
-    return (-1);
+    return -1;
   }
 
-#ifdef USE_IMAP
   if (Context->magic == M_IMAP && imap_search (Context, pat) < 0)
     return -1;
-#endif
 
   mutt_message _("Executing command on matching messages...");
 
@@ -1282,7 +1189,7 @@ int mutt_pattern_func (int op, char *prompt)
     /* drop previous limit pattern */
     p_delete(&Context->pattern);
     if (Context->limit_pattern)
-      mutt_pattern_free (&Context->limit_pattern);
+      pattern_list_wipe(&Context->limit_pattern);
     if (Context->msgcount && !Context->vcount) {
       mutt_error _("No messages matched criteria.");
     }
@@ -1295,7 +1202,7 @@ int mutt_pattern_func (int op, char *prompt)
     }
   }
   p_delete(&simple);
-  mutt_pattern_free (&pat);
+  pattern_list_wipe(&pat);
   return 0;
 }
 
@@ -1314,7 +1221,7 @@ int mutt_search_command (int cur, int op)
     if (mutt_get_field ((op == OP_SEARCH) ? _("Search for: ") :
                         _("Reverse search for: "), buf, sizeof (buf),
                         M_CLEAR | M_PATTERN) != 0 || !buf[0])
-      return (-1);
+      return -1;
 
     if (op == OP_SEARCH)
       unset_option (OPTSEARCHREVERSE);
@@ -1331,13 +1238,13 @@ int mutt_search_command (int cur, int op)
       m_strcpy(LastSearch, sizeof(LastSearch), buf);
       mutt_message _("Compiling search pattern...");
 
-      mutt_pattern_free (&SearchPattern);
+      pattern_list_wipe(&SearchPattern);
       err.data = error;
       err.dsize = sizeof (error);
       if ((SearchPattern =
            mutt_pattern_comp (temp, M_FULL_MSG, &err)) == NULL) {
         mutt_error ("%s", error);
-        return (-1);
+        return -1;
       }
       mutt_clear_error ();
     }
@@ -1345,16 +1252,14 @@ int mutt_search_command (int cur, int op)
   else if (!SearchPattern) {
     mutt_error _("No search pattern.");
 
-    return (-1);
+    return -1;
   }
 
   if (option (OPTSEARCHINVALID)) {
     for (i = 0; i < Context->msgcount; i++)
       Context->hdrs[i]->searched = 0;
-#ifdef USE_IMAP
     if (Context->magic == M_IMAP && imap_search (Context, SearchPattern) < 0)
       return -1;
-#endif
     unset_option (OPTSEARCHINVALID);
   }
 
@@ -1371,7 +1276,7 @@ int mutt_search_command (int cur, int op)
       else {
         mutt_message _("Search hit bottom without finding match");
 
-        return (-1);
+        return -1;
       }
     }
     else if (i < 0) {
@@ -1382,7 +1287,7 @@ int mutt_search_command (int cur, int op)
       else {
         mutt_message _("Search hit top without finding match");
 
-        return (-1);
+        return -1;
       }
     }
 
@@ -1405,7 +1310,7 @@ int mutt_search_command (int cur, int op)
       mutt_error _("Search interrupted.");
 
       SigInt = 0;
-      return (-1);
+      return -1;
     }
 
     i += incr;
@@ -1413,5 +1318,5 @@ int mutt_search_command (int cur, int op)
 
   mutt_error _("Not found.");
 
-  return (-1);
+  return -1;
 }