X-Git-Url: http://git.madism.org/?p=apps%2Fmadmutt.git;a=blobdiff_plain;f=pattern.c;h=94c1f9c03cd96a019f9310bd7d6932645eae675f;hp=9b325a239c726aed4de5ecb92c555a3e21294371;hb=7db27b8f24670fd40cf24755f2782a104ad24594;hpb=89a7d4396b35e8047cbe519e81e22d99a1efd018 diff --git a/pattern.c b/pattern.c index 9b325a2..94c1f9c 100644 --- a/pattern.c +++ b/pattern.c @@ -7,100 +7,76 @@ * please see the file GPL in the top level source directory. */ -#if HAVE_CONFIG_H -# include "config.h" -#endif - -#include -#include -#include -#include -#include -#include -#include +#include #include +#include +#include +#include -#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" -#include - -#include -#include -#include -#include -#include -#include - -#include - 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 */ @@ -120,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 */ @@ -156,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)) { @@ -164,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) @@ -175,17 +148,6 @@ msg_search (CONTEXT *ctx, pattern_t* pat, int msgno) if (pat->op != M_HEADER) { mutt_parse_mime_message (ctx, h); - - if ((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); } @@ -231,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); } } @@ -249,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); @@ -274,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; @@ -287,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; @@ -424,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) @@ -525,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) @@ -537,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); @@ -601,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); @@ -634,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; } } } @@ -642,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) +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) -{ - 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; @@ -704,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) { @@ -733,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; @@ -753,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; @@ -761,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; @@ -777,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; @@ -793,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; } } @@ -808,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); @@ -832,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; } } @@ -841,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 @@ -869,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; @@ -883,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 */ } } } @@ -891,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) @@ -903,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; } @@ -914,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; } @@ -925,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; } @@ -948,7 +881,7 @@ static int valid_realname (const char *name) ret--; p++; } - return (ret >= 1); + return ret >= 1; } /* flags @@ -959,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 <= @@ -992,14 +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: /* IMAP search sets h->matched at search compile time */ if (ctx->magic == M_IMAP && pat->stringmatch) - return (h->matched); - 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)); @@ -1029,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 @@ -1061,13 +994,13 @@ 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: - return (pat->not ^ ((h->security & SIGN) ? 1 : 0)); + return pat->not ^ ((h->security & SIGN) ? 1 : 0); case M_CRYPT_VERIFIED: - return (pat->not ^ ((h->security & GOODSIGN) ? 1 : 0)); + return pat->not ^ ((h->security & GOODSIGN) ? 1 : 0); case M_CRYPT_ENCRYPT: - return (pat->not ^ ((h->security & ENCRYPT) ? 1 : 0)); + return pat->not ^ ((h->security & ENCRYPT) ? 1 : 0); case M_PGP_KEY: return (pat->not ^ ((h->security & APPLICATION_PGP) && (h->security & PGPKEY))); @@ -1080,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: { @@ -1091,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 || @@ -1099,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 @@ -1109,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) { @@ -1146,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]; @@ -1176,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]; @@ -1191,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..."); @@ -1203,7 +1126,7 @@ 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; } if (Context->magic == M_IMAP && imap_search (Context, pat) < 0) @@ -1266,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."); } @@ -1279,7 +1202,7 @@ int mutt_pattern_func (int op, char *prompt) } } p_delete(&simple); - mutt_pattern_free (&pat); + pattern_list_wipe(&pat); return 0; } @@ -1298,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); @@ -1315,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 (); } @@ -1329,7 +1252,7 @@ int mutt_search_command (int cur, int op) else if (!SearchPattern) { mutt_error _("No search pattern."); - return (-1); + return -1; } if (option (OPTSEARCHINVALID)) { @@ -1353,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) { @@ -1364,7 +1287,7 @@ int mutt_search_command (int cur, int op) else { mutt_message _("Search hit top without finding match"); - return (-1); + return -1; } } @@ -1387,7 +1310,7 @@ int mutt_search_command (int cur, int op) mutt_error _("Search interrupted."); SigInt = 0; - return (-1); + return -1; } i += incr; @@ -1395,5 +1318,5 @@ int mutt_search_command (int cur, int op) mutt_error _("Not found."); - return (-1); + return -1; }