Simplications go on.
[apps/madmutt.git] / lib-mime / rfc822parse.c
index 5078f19..5509b8c 100644 (file)
  * please see the file GPL in the top level source directory.
  */
 
+#if HAVE_CONFIG_H
+# include "config.h"
+#endif
+
 #include <stdio.h>
 
 #include <lib-lib/mem.h>
 #include <lib-lib/buffer.h>
 #include <lib-lib/date.h>
 
-#include <lib-crypt/crypt.h>
-
-#include "enter.h"
 #include "recvattach.h"
-#include "mx.h"
 #include "url.h"
 
-#include "lib/rx.h"
 #include "lib/debug.h"
 
 #include "mime.h"
  * lines.  ``line'' must point to a dynamically allocated string; it is
  * increased if more space is required to fit the whole line.
  */
-char *mutt_read_rfc822_line (FILE * f, char *line, size_t * linelen)
+ssize_t mutt_read_rfc822_line(FILE *f, char **line, ssize_t *n)
 {
-  char *buf = line;
-  char ch;
-  size_t offset = 0;
-
-  for (;;) {
-    if (fgets (buf, *linelen - offset, f) == NULL ||    /* end of file or */
-        (ISSPACE (*line) && !offset)) { /* end of headers */
-      *line = 0;
-      return (line);
-    }
+    ssize_t pos = 0;
 
-    buf += m_strlen(buf) - 1;
-    if (*buf == '\n') {
-      /* we did get a full line. remove trailing space */
-      while (ISSPACE (*buf))
-        *buf-- = 0;             /* we cannot come beyond line's beginning because
-                                 * it begins with a non-space */
-
-      /* check to see if the next line is a continuation line */
-      if ((ch = fgetc (f)) != ' ' && ch != '\t') {
-        ungetc (ch, f);
-        return (line);          /* next line is a separate header field or EOH */
-      }
+    for (;;) {
+        char *p = *line;
 
-      /* eat tabs and spaces from the beginning of the continuation line */
-      while ((ch = fgetc (f)) == ' ' || ch == '\t');
-      ungetc (ch, f);
-      *++buf = ' ';             /* string is still terminated because we removed
-                                   at least one whitespace char above */
-    }
+        /* end of file or end of headers */
+        if (!fgets(p + pos, *n - pos, f) || (ISSPACE(*p) && pos == 0)) {
+            *p = '\0';
+            return 0;
+        }
+
+        pos += m_strlen(p + pos);
+        if (p[pos - 1] == '\n') {
+            int c;
+
+            /* remove trailing spaces. safe: p[0] is not a space */
+            do {
+                p[--pos] = '\0';
+            } while (ISSPACE(p[pos]));
+
+            /* check to see if the next line is a continuation line */
+            c = fgetc(f);
+            if (c != ' ' && c != '\t') {
+                /* next line is a separate header field or EOH */
+                ungetc(c, f);
+                return pos;
+            }
 
-    buf++;
-    offset = buf - line;
-    if (*linelen < offset + STRING) {
-      /* grow the buffer */
-      *linelen += STRING;
-      p_realloc(&line, *linelen);
-      buf = line + offset;
+            /* eat tabs and spaces from the beginning of the continuation line */
+            do {
+                c = fgetc(f);
+            } while (c == ' ' || c == '\t');
+            ungetc(c, f);
+
+            /* string is still terminated because we removed at least one
+               whitespace char above */
+            p[pos++] = ' ';
+        }
+
+        if (*n < pos + STRING) {
+            /* grow the buffer */
+            *n += STRING;
+            p_realloc(line, *n);
+        }
     }
-  }
-  /* not reached */
 }
 
-LIST *mutt_parse_references (char *s, int in_reply_to)
+/* TODO: Make that a string list somehow */
+LIST *mutt_parse_references(char *s, int in_reply_to)
 {
-  LIST *t, *lst = NULL;
-  int m, n = 0;
-  char *o = NULL, *new, *at;
+    LIST *lst = NULL;
+    int n = 0;
+    char *o = NULL;
 
-  while ((s = strtok (s, " \t;")) != NULL) {
-    /*
-     * some mail clients add other garbage besides message-ids, so do a quick
+    /* some mail clients add other garbage besides message-ids, so do a quick
      * check to make sure this looks like a valid message-id
      * some idiotic clients also break their message-ids between lines, deal
      * with that too (give up if it's more than two lines, though)
      */
-    t = NULL;
-    new = NULL;
-
-    if (*s == '<') {
-      n = m_strlen(s);
-      if (s[n - 1] != '>') {
-        o = s;
-        s = NULL;
-        continue;
-      }
 
-      new = m_strdup(s);
-    }
-    else if (o) {
-      m = m_strlen(s);
-      if (s[m - 1] == '>') {
-        new = p_new(char, n + m + 1);
-        strcpy (new, o);        /* __STRCPY_CHECKED__ */
-        strcpy (new + n, s);    /* __STRCPY_CHECKED__ */
-      }
-    }
-    if (new) {
-      /* make sure that this really does look like a message-id.
-       * it should have exactly one @, and if we're looking at
-       * an in-reply-to header, make sure that the part before
-       * the @ has more than eight characters or it's probably
-       * an email address
-       */
-      if (!(at = strchr (new, '@')) || strchr (at + 1, '@')
-          || (in_reply_to && at - new <= 8))
-        p_delete(&new);
-      else {
-        t = p_new(LIST, 1);
-        t->data = new;
-        t->next = lst;
-        lst = t;
-      }
+    for (s = strtok(s, " \t;"); s; s = strtok(NULL, " \t;")) {
+        char *new = NULL;
+
+        if (*s == '<') {
+            n = m_strlen(s);
+            if (s[n - 1] != '>') {
+                o = s;
+                continue;
+            }
+
+            new = m_strdup(s);
+        } else if (o) {
+            ssize_t m = m_strlen(s);
+
+            if (s[m - 1] != '>') {
+                o = NULL;
+            } else {
+                new = p_new(char, n + m + 1);
+                strcpy(new, o);
+                strcpy(new + n, s);
+            }
+        }
+
+        /* make sure that this really does look like a message-id.
+         * it should have exactly one @, and if we're looking at
+         * an in-reply-to header, make sure that the part before
+         * the @ has more than eight characters or it's probably
+         * an email address
+         */
+        if (new) {
+            char *at = strchr(new, '@');
+            LIST *tmp;
+
+            if (!at || strchr(at + 1, '@') || (in_reply_to && at - new <= 8)) {
+                p_delete(&new);
+                continue;
+            }
+
+            tmp = p_new(LIST, 1);
+            tmp->data = new;
+            tmp->next = lst;
+            lst = tmp;
+        }
     }
-    o = NULL;
-    s = NULL;
-  }
 
-  return (lst);
+    return lst;
 }
 
-int mutt_check_encoding (const char *c)
+int mutt_check_encoding(const char *s)
 {
-  if (ascii_strncasecmp ("7bit", c, sizeof ("7bit") - 1) == 0)
-    return (ENC7BIT);
-  else if (ascii_strncasecmp ("8bit", c, sizeof ("8bit") - 1) == 0)
-    return (ENC8BIT);
-  else if (ascii_strncasecmp ("binary", c, sizeof ("binary") - 1) == 0)
-    return (ENCBINARY);
-  else
-    if (ascii_strncasecmp
-        ("quoted-printable", c, sizeof ("quoted-printable") - 1) == 0)
-    return (ENCQUOTEDPRINTABLE);
-  else if (ascii_strncasecmp ("base64", c, sizeof ("base64") - 1) == 0)
-    return (ENCBASE64);
-  else if (ascii_strncasecmp ("x-uuencode", c, sizeof ("x-uuencode") - 1) == 0)
-    return (ENCUUENCODED);
-  else
-    return (ENCOTHER);
+#define COMPARE(tok, value)                             \
+    if (!ascii_strncasecmp(tok, s, sizeof(tok) - 1)) {  \
+        return value;                                   \
+    }
+    COMPARE("7bit", ENC7BIT);
+    COMPARE("8bit", ENC8BIT);
+    COMPARE("binary", ENCBINARY);
+    COMPARE("quoted-printable", ENCQUOTEDPRINTABLE);
+    COMPARE("base64", ENCBASE64);
+    COMPARE("x-uuencode", ENCUUENCODED);
+#undef COMPARE
+
+    return ENCOTHER;
 }
 
-static PARAMETER *parse_parameters (const char *s)
+int mutt_check_mime_type(const char *s)
 {
-  PARAMETER *head = 0, *cur = 0, *new;
-  char buffer[LONG_STRING];
-  const char *p;
-  size_t i;
-
-  debug_print (2, ("`%s'\n", s));
-
-  while (*s) {
-    if ((p = strpbrk (s, "=;")) == NULL) {
-      debug_print (1, ("malformed parameter: %s\n", s));
-      goto bail;
+#define COMPARE(tok, value)                             \
+    if (!ascii_strncasecmp(tok, s, sizeof(tok) - 1)) {  \
+        return value;                                   \
     }
+  COMPARE("text", TYPETEXT);
+  COMPARE("multipart", TYPEMULTIPART);
+  COMPARE("application", TYPEAPPLICATION);
+  COMPARE("message", TYPEMESSAGE);
+  COMPARE("image", TYPEIMAGE);
+  COMPARE("audio", TYPEAUDIO);
+  COMPARE("video", TYPEVIDEO);
+  COMPARE("model", TYPEMODEL);
+  COMPARE("*",  TYPEANY);
+  COMPARE(".*", TYPEANY);
+#undef COMPARE
+
+  return TYPEOTHER;
+}
 
-    /* if we hit a ; now the parameter has no value, just skip it */
-    if (*p != ';') {
-      i = p - s;
+static PARAMETER *parse_parameters(const char *s)
+{
+    PARAMETER *res = NULL;
+    PARAMETER **list = &res;
 
-      new = mutt_new_parameter ();
+    while (*s) {
+        const char *p;
+        PARAMETER *new;
+        int i;
 
-      new->attribute = p_dupstr(s, i);
+        s = skipspaces(s);
+        if (*s == '=')             /* parameters are fucked up, go away */
+            break;
 
-      /* remove whitespace from the end of the attribute name */
-      while (ISSPACE (new->attribute[--i]))
-        new->attribute[i] = 0;
+        p = strpbrk(s, "=;");
+        if (!p)
+            break;
 
-      s = vskipspaces(p + 1);     /* skip over the = */
+        if (*p == ';') {
+            /* if we hit a ; now the parameter has no value, just skip it */
+            s = p + 1;
+            continue;
+        }
 
-      if (*s == '"') {
-        int state_ascii = 1;
+        i = p - s;
+        new = parameter_new();
+        new->attribute = p_dupstr(s, i);
 
-        s++;
-        for (i = 0; *s && i < sizeof (buffer) - 1; i++, s++) {
-          if (!option (OPTSTRICTMIME)) {
-            /* As iso-2022-* has a characer of '"' with non-ascii state,
-             * ignore it. */
-            if (*s == 0x1b && i < sizeof (buffer) - 2) {
-              if (s[1] == '(' && (s[2] == 'B' || s[2] == 'J'))
-                state_ascii = 1;
-              else
-                state_ascii = 0;
-            }
-          }
-          if (state_ascii && *s == '"')
-            break;
-          if (*s == '\\') {
-            /* Quote the next character */
-            buffer[i] = s[1];
-            if (!*++s)
-              break;
-          }
-          else
-            buffer[i] = *s;
+        while (--i >= 0 && ISSPACE(new->attribute[i])) {
+            new->attribute[i] = '\0';
         }
-        buffer[i] = 0;
-        if (*s)
-          s++;                  /* skip over the " */
-      }
-      else {
-        for (i = 0; *s && *s != ' ' && *s != ';' && i < sizeof (buffer) - 1;
-             i++, s++)
-          buffer[i] = *s;
-        buffer[i] = 0;
-      }
+        s = skipspaces(p + 1);                      /* skip over the = */
+
+        if (*s == '"') {
+            char buffer[LONG_STRING];
+            int state_ascii = 1;
+
+            s++;
+            for (i = 0; *s && i < ssizeof(buffer) - 1; i++, s++) {
+                if (!option(OPTSTRICTMIME)) {
+                    /* As iso-2022-* has a characer of '"' with non-ascii state,
+                     * ignore it. */
+                    if (*s == 0x1b && i < ssizeof(buffer) - 2) {
+                        state_ascii = s[1] == '(' && (s[2] == 'B' || s[2] == 'J');
+                    }
+                }
+                if (state_ascii && *s == '"')
+                    break;
+
+                if (*s == '\\') {
+                    buffer[i] = *++s;
+                } else {
+                    buffer[i] = *s;
+                }
+            }
 
-      new->value = m_strdup(buffer);
+            new->value = p_dupstr(buffer, i);
+        } else {
+            for (p = s; *p && *p != ' ' && *p != ';'; p++);
+            new->value = p_dupstr(s, p - s);
+        }
 
-      debug_print (2, ("`%s' = `%s'\n", new->attribute ? new->attribute : "",
-                  new->value ? new->value : ""));
+        *list = new;
+        list = &new->next;
 
-      /* Add this parameter to the list */
-      if (head) {
-        cur->next = new;
-        cur = cur->next;
-      }
-      else
-        head = cur = new;
-    }
-    else {
-      debug_print (1, ("parameter with no value: %s\n", s));
-      s = p;
+        s = strchr(s, ';');           /* Find the next parameter */
+        if (!s)
+            break;                    /* no more parameters */
     }
 
-    /* Find the next parameter */
-    if (*s != ';' && (s = strchr (s, ';')) == NULL)
-      break;                    /* no more parameters */
+    rfc2231_decode_parameters(&res);
+    return res;
+}
 
-    do {
-      /* Move past any leading whitespace */
-      s = vskipspaces(s + 1);
+void mutt_parse_content_type(char *s, BODY *ct)
+{
+    char *pc;
+    char *subtype;
+
+    p_delete(&ct->subtype);
+    parameter_delete(&ct->parameter);
+
+    /* First extract any existing parameters */
+    if ((pc = strchr(s, ';')) != NULL) {
+        *pc++ = '\0';
+        ct->parameter = parse_parameters(vskipspaces(pc));
+
+        /* Some pre-RFC1521 gateways still use the "name=filename" convention,
+         * but if a filename has already been set in the content-disposition,
+         * let that take precedence, and don't set it here */
+        pc = mutt_get_parameter("name", ct->parameter);
+        if (pc && !ct->filename)
+            ct->filename = m_strdup(pc);
     }
-    while (*s == ';');          /* skip empty parameters */
-  }
 
-bail:
+    /* Now get the subtype */
+    if ((subtype = strchr (s, '/'))) {
+        *subtype++ = '\0';
+        for (pc = subtype; *pc && !ISSPACE(*pc) && *pc != ';'; pc++);
+        ct->subtype = p_dupstr(subtype, pc - subtype);
+    }
 
-  rfc2231_decode_parameters (&head);
-  return (head);
-}
+    /* Finally, get the major type */
+    ct->type = mutt_check_mime_type(s);
 
-int mutt_check_mime_type (const char *s)
-{
-  if (ascii_strcasecmp ("text", s) == 0)
-    return TYPETEXT;
-  else if (ascii_strcasecmp ("multipart", s) == 0)
-    return TYPEMULTIPART;
-  else if (ascii_strcasecmp ("application", s) == 0)
-    return TYPEAPPLICATION;
-  else if (ascii_strcasecmp ("message", s) == 0)
-    return TYPEMESSAGE;
-  else if (ascii_strcasecmp ("image", s) == 0)
-    return TYPEIMAGE;
-  else if (ascii_strcasecmp ("audio", s) == 0)
-    return TYPEAUDIO;
-  else if (ascii_strcasecmp ("video", s) == 0)
-    return TYPEVIDEO;
-  else if (ascii_strcasecmp ("model", s) == 0)
-    return TYPEMODEL;
-  else if (ascii_strcasecmp ("*", s) == 0)
-    return TYPEANY;
-  else if (ascii_strcasecmp (".*", s) == 0)
-    return TYPEANY;
-  else
-    return TYPEOTHER;
-}
+    if (ct->type == TYPEOTHER) {
+        ct->xtype = m_strdup(s);
+    }
 
-void mutt_parse_content_type (char *s, BODY * ct)
-{
-  char *pc;
-  char *subtype;
-
-  p_delete(&ct->subtype);
-  mutt_free_parameter (&ct->parameter);
-
-  /* First extract any existing parameters */
-  if ((pc = strchr (s, ';')) != NULL) {
-    *pc++ = 0;
-    while (*pc && ISSPACE (*pc))
-      pc++;
-    ct->parameter = parse_parameters (pc);
-
-    /* Some pre-RFC1521 gateways still use the "name=filename" convention,
-     * but if a filename has already been set in the content-disposition,
-     * let that take precedence, and don't set it here */
-    if ((pc = mutt_get_parameter ("name", ct->parameter)) != 0
-        && !ct->filename)
-      ct->filename = m_strdup(pc);
-  }
+    if (!ct->subtype) {
+        /* Some older non-MIME mailers (i.e., mailtool, elm) have a content-type
+         * field, so we can attempt to convert the type to BODY here.
+         */
+        switch (ct->type) {
+            char buffer[SHORT_STRING];
 
-  /* Now get the subtype */
-  if ((subtype = strchr (s, '/'))) {
-    *subtype++ = '\0';
-    for (pc = subtype; *pc && !ISSPACE (*pc) && *pc != ';'; pc++);
-    *pc = '\0';
-    ct->subtype = m_strdup(subtype);
-  }
+          case TYPETEXT:
+            ct->subtype = m_strdup("plain");
+            break;
 
-  /* Finally, get the major type */
-  ct->type = mutt_check_mime_type (s);
+          case TYPEAUDIO:
+            ct->subtype = m_strdup("basic");
+            break;
 
-  if (ct->type == TYPEOTHER) {
-    ct->xtype = m_strdup(s);
-  }
+          case TYPEMESSAGE:
+            ct->subtype = m_strdup("rfc822");
+            break;
 
-  if (ct->subtype == NULL) {
-    /* Some older non-MIME mailers (i.e., mailtool, elm) have a content-type
-     * field, so we can attempt to convert the type to BODY here.
-     */
-    if (ct->type == TYPETEXT)
-      ct->subtype = m_strdup("plain");
-    else if (ct->type == TYPEAUDIO)
-      ct->subtype = m_strdup("basic");
-    else if (ct->type == TYPEMESSAGE)
-      ct->subtype = m_strdup("rfc822");
-    else if (ct->type == TYPEOTHER) {
-      char buffer[SHORT_STRING];
-
-      ct->type = TYPEAPPLICATION;
-      snprintf (buffer, sizeof (buffer), "x-%s", s);
-      ct->subtype = m_strdup(buffer);
-    }
-    else
-      ct->subtype = m_strdup("x-unknown");
-  }
+          case TYPEOTHER:
+            ct->type = TYPEAPPLICATION;
+            snprintf(buffer, sizeof(buffer), "x-%s", s);
+            ct->subtype = m_strdup(buffer);
+            break;
 
-  /* Default character set for text types. */
-  if (ct->type == TYPETEXT) {
-    if (!(pc = mutt_get_parameter ("charset", ct->parameter)))
-      mutt_set_parameter ("charset", option (OPTSTRICTMIME) ? "us-ascii" :
-                          (const char *)
-                          mutt_get_first_charset (AssumedCharset),
-                          &ct->parameter);
-  }
+          default:
+            ct->subtype = m_strdup("x-unknown");
+            break;
+        }
+    }
 
+    /* Default character set for text types. */
+    if (ct->type == TYPETEXT) {
+        pc = mutt_get_parameter("charset", ct->parameter);
+        if (!pc) {
+            mutt_set_parameter("charset",
+                               option(OPTSTRICTMIME) ? "us-ascii" :
+                               mutt_get_first_charset(AssumedCharset),
+                               &ct->parameter);
+        }
+    }
 }
 
-static void parse_content_disposition (char *s, BODY * ct)
+static void parse_content_disposition(char *s, BODY *ct)
 {
-  PARAMETER *parms;
+    if (!ascii_strncasecmp(s, "inline", 6)) {
+        ct->disposition = DISPINLINE;
+    } else if (!ascii_strncasecmp(s, "form-data", 9)) {
+        ct->disposition = DISPFORMDATA;
+    } else {
+        ct->disposition = DISPATTACH;
+    }
 
-  if (!ascii_strncasecmp ("inline", s, 6))
-    ct->disposition = DISPINLINE;
-  else if (!ascii_strncasecmp ("form-data", s, 9))
-    ct->disposition = DISPFORMDATA;
-  else
-    ct->disposition = DISPATTACH;
-
-  /* Check to see if a default filename was given */
-  if ((s = strchr (s, ';')) != NULL) {
-    s = vskipspaces(s + 1);
-    if ((s = mutt_get_parameter("filename",
-                                (parms = parse_parameters (s)))) != 0)
-      m_strreplace(&ct->filename, s);
-    if ((s = mutt_get_parameter ("name", parms)) != 0)
-      ct->form_name = m_strdup(s);
-    mutt_free_parameter (&parms);
-  }
+    /* Check to see if a default filename was given */
+    if ((s = strchr (s, ';'))) {
+        PARAMETER *parms = parse_parameters(vskipspaces(s));
+
+        if ((s = mutt_get_parameter("filename", parms)))
+            m_strreplace(&ct->filename, s);
+        if ((s = mutt_get_parameter ("name", parms)))
+            ct->form_name = m_strdup(s);
+
+        parameter_delete(&parms);
+    }
 }
 
 /* args:
@@ -400,94 +379,92 @@ static void parse_content_disposition (char *s, BODY * ct)
  *     digest  1 if reading subparts of a multipart/digest, 0
  *             otherwise
  */
-
-BODY *mutt_read_mime_header (FILE * fp, int digest)
+BODY *mutt_read_mime_header(FILE *fp, int digest)
 {
-  BODY *p = mutt_new_body ();
-  char *c;
-  char *line = p_new(char, LONG_STRING);
-  size_t linelen = LONG_STRING;
-
-  p->hdr_offset = ftello (fp);
-
-  p->encoding = ENC7BIT;        /* default from RFC1521 */
-  p->type = digest ? TYPEMESSAGE : TYPETEXT;
-  p->disposition = DISPINLINE;
-
-  while (*(line = mutt_read_rfc822_line (fp, line, &linelen)) != 0) {
-    /* Find the value of the current header */
-    if ((c = strchr (line, ':'))) {
-      *c++ = 0;
-      c = vskipspaces(c);
-      if (!*c) {
-        debug_print (1, ("skipping empty header field: %s\n", line));
-        continue;
-      }
-    }
-    else {
-      debug_print (1, ("bogus MIME header: %s\n", line));
-      break;
-    }
+    BODY *body = mutt_new_body ();
+    char *line = p_new(char, LONG_STRING);
+    ssize_t linelen = LONG_STRING;
+    char *p;
+
+    body->hdr_offset  = ftello(fp);
+    body->encoding    = ENC7BIT;    /* default from RFC1521 */
+    body->disposition = DISPINLINE;
+    body->type        = digest ? TYPEMESSAGE : TYPETEXT;
+
+    while (mutt_read_rfc822_line(fp, &line, &linelen)) {
+        /* Find the value of the current header */
+        if ((p = strchr(line, ':'))) {
+            *p++ = '\0';
+            p = vskipspaces(p);
+            if (!*p)
+                continue;
+        } else {
+            debug_print (1, ("bogus MIME header: %s\n", line));
+            break;
+        }
 
-    if (!ascii_strncasecmp ("content-", line, 8)) {
-      if (!ascii_strcasecmp ("type", line + 8))
-        mutt_parse_content_type (c, p);
-      else if (!ascii_strcasecmp ("transfer-encoding", line + 8))
-        p->encoding = mutt_check_encoding (c);
-      else if (!ascii_strcasecmp ("disposition", line + 8))
-        parse_content_disposition (c, p);
-      else if (!ascii_strcasecmp ("description", line + 8)) {
-        m_strreplace(&p->description, c);
-        rfc2047_decode (&p->description);
-      }
+        if (!ascii_strncasecmp(line, "content-", 8)) {
+            if (!ascii_strcasecmp("type", line + 8))
+                mutt_parse_content_type (p, body);
+            else if (!ascii_strcasecmp ("transfer-encoding", line + 8))
+                body->encoding = mutt_check_encoding (p);
+            else if (!ascii_strcasecmp ("disposition", line + 8))
+                parse_content_disposition(p, body);
+            else if (!ascii_strcasecmp ("description", line + 8)) {
+                m_strreplace(&body->description, p);
+                rfc2047_decode(&body->description);
+            }
+        }
     }
-  }
-  p->offset = ftello (fp);       /* Mark the start of the real data */
-  if (p->type == TYPETEXT && !p->subtype)
-    p->subtype = m_strdup("plain");
-  else if (p->type == TYPEMESSAGE && !p->subtype)
-    p->subtype = m_strdup("rfc822");
 
-  p_delete(&line);
+    body->offset = ftello(fp);       /* Mark the start of the real data */
+    if (!body->subtype) {
+        if (body->type == TYPETEXT)
+            body->subtype = m_strdup("plain");
+        if (body->type == TYPEMESSAGE)
+            body->subtype = m_strdup("rfc822");
+    }
 
-  return (p);
+    p_delete(&line);
+    return (body);
 }
 
-void mutt_parse_part (FILE * fp, BODY * b)
+void mutt_parse_part(FILE *fp, BODY *b)
 {
-  char *bound = 0;
-
-  switch (b->type) {
-  case TYPEMULTIPART:
-    bound = mutt_get_parameter ("boundary", b->parameter);
-    fseeko (fp, b->offset, SEEK_SET);
-    b->parts = mutt_parse_multipart (fp, bound,
-                                     b->offset + b->length,
-                                     ascii_strcasecmp ("digest",
-                                                       b->subtype) == 0);
-    break;
-
-  case TYPEMESSAGE:
-    if (b->subtype) {
-      fseeko (fp, b->offset, SEEK_SET);
-      if (mutt_is_message_type (b->type, b->subtype))
-        b->parts = mutt_parse_messageRFC822 (fp, b);
-      else if (ascii_strcasecmp (b->subtype, "external-body") == 0)
-        b->parts = mutt_read_mime_header (fp, 0);
-      else
+    char *bound = 0;
+
+    switch (b->type) {
+      case TYPEMULTIPART:
+        bound = mutt_get_parameter("boundary", b->parameter);
+        fseeko(fp, b->offset, SEEK_SET);
+        b->parts = mutt_parse_multipart(fp, bound, b->offset + b->length,
+                           !ascii_strcasecmp("digest", b->subtype));
+        break;
+
+      case TYPEMESSAGE:
+        if (b->subtype) {
+            fseeko(fp, b->offset, SEEK_SET);
+
+            if (mutt_is_message_type(b->type, b->subtype)) {
+                b->parts = mutt_parse_messageRFC822(fp, b);
+            } else
+            if (!ascii_strcasecmp(b->subtype, "external-body") == 0) {
+                b->parts = mutt_read_mime_header(fp, 0);
+            } else {
+                return;
+            }
+        }
+        break;
+
+      default:
         return;
     }
-    break;
-
-  default:
-    return;
-  }
 
-  /* try to recover from parsing error */
-  if (!b->parts) {
-    b->type = TYPETEXT;
-    m_strreplace(&b->subtype, "plain");
-  }
+    /* try to recover from parsing error */
+    if (!b->parts) {
+        b->type = TYPETEXT;
+        m_strreplace(&b->subtype, "plain");
+    }
 }
 
 /* parse a MESSAGE/RFC822 body
@@ -500,27 +477,25 @@ void mutt_parse_part (FILE * fp, BODY * b)
  *
  * NOTE: this assumes that `parent->length' has been set!
  */
-
-BODY *mutt_parse_messageRFC822 (FILE * fp, BODY * parent)
+BODY *mutt_parse_messageRFC822(FILE * fp, BODY * parent)
 {
-  BODY *msg;
+    BODY *msg;
+
+    parent->hdr = header_new();
+    parent->hdr->offset = ftello(fp);
+    parent->hdr->env    = mutt_read_rfc822_header(fp, parent->hdr, 0, 0);
 
-  parent->hdr = mutt_new_header ();
-  parent->hdr->offset = ftello (fp);
-  parent->hdr->env = mutt_read_rfc822_header (fp, parent->hdr, 0, 0);
-  msg = parent->hdr->content;
+    msg = parent->hdr->content;
 
-  /* ignore the length given in the content-length since it could be wrong
-     and we already have the info to calculate the correct length */
-  /* if (msg->length == -1) */
-  msg->length = parent->length - (msg->offset - parent->offset);
+    /* ignore the length given in the content-length since it could be wrong
+       and we already have the info to calculate the correct length */
+    /* if (msg->length == -1) */
+    /* if body of this message is empty, we can end up with a negative length */
+    msg->length = MAX(0, parent->length - (msg->offset - parent->offset));
 
-  /* if body of this message is empty, we can end up with a negative length */
-  if (msg->length < 0)
-    msg->length = 0;
+    mutt_parse_part(fp, msg);
 
-  mutt_parse_part (fp, msg);
-  return (msg);
+    return msg;
 }
 
 /* parse a multipart structure
@@ -528,7 +503,7 @@ BODY *mutt_parse_messageRFC822 (FILE * fp, BODY * parent)
  * args:
  *     fp              stream to read from
  *
- *     boundary        body separator
+ *     bound           body separator
  *
  *     end_off         length of the multipart body (used when the final
  *                     boundary is missing to avoid reading too far)
@@ -536,188 +511,154 @@ BODY *mutt_parse_messageRFC822 (FILE * fp, BODY * parent)
  *     digest          1 if reading a multipart/digest, 0 otherwise
  */
 
-BODY *mutt_parse_multipart (FILE * fp, const char *boundary, off_t end_off,
-                            int digest)
+BODY *
+mutt_parse_multipart(FILE *fp, const char *bound, off_t end_off, int digest)
 {
-  int blen, len, crlf = 0;
-  char buffer[LONG_STRING];
-  BODY *head = 0, *last = 0, *new = 0;
-  int i;
-  int final = 0;                /* did we see the ending boundary? */
+    char buffer[LONG_STRING];
+    BODY *head = NULL;
+    BODY **last = &head;
+    int blen = m_strlen(bound);
+    int final = 0;                /* did we see the ending boundary? */
+
+    if (!blen) {
+        mutt_error _("multipart message has no boundary parameter!");
+        return NULL;
+    }
 
-  if (!boundary) {
-    mutt_error _("multipart message has no boundary parameter!");
+    while (ftello(fp) < end_off && fgets(buffer, sizeof(buffer), fp)) {
+        int len, crlf, i;
 
-    return (NULL);
-  }
+        len  = m_strlen(buffer);
+        crlf = len > 1 && buffer[len - 2] == '\r';
 
-  blen = m_strlen(boundary);
-  while (ftello (fp) < end_off && fgets (buffer, LONG_STRING, fp) != NULL) {
-    len = m_strlen(buffer);
-
-    crlf = (len > 1 && buffer[len - 2] == '\r') ? 1 : 0;
-
-    if (buffer[0] == '-' && buffer[1] == '-' &&
-        m_strncmp(buffer + 2, boundary, blen) == 0) {
-      if (last) {
-        last->length = ftello (fp) - last->offset - len - 1 - crlf;
-        if (last->parts && last->parts->length == 0)
-          last->parts->length =
-            ftello (fp) - last->parts->offset - len - 1 - crlf;
-        /* if the body is empty, we can end up with a -1 length */
-        if (last->length < 0)
-          last->length = 0;
-      }
+        if (buffer[0] == '-' && buffer[1] == '-'
+        && !m_strncmp(buffer + 2, bound, blen))
+        {
+            if (*last) {
+                BODY *b = *last;
 
-      /* Remove any trailing whitespace, up to the length of the boundary */
-      for (i = len - 1; ISSPACE (buffer[i]) && i >= blen + 2; i--)
-        buffer[i] = 0;
+                /* if the body is empty, we can end up with a -1 length */
+                b->length = MAX(0, ftello(fp) - b->offset - len - 1 - crlf);
+                if (b->parts && b->parts->length == 0) {
+                    b->parts->length = ftello(fp) - b->parts->offset
+                                     - len - 1 - crlf;
+                }
+            }
 
-      /* Check for the end boundary */
-      if (m_strcmp(buffer + blen + 2, "--") == 0) {
-        final = 1;
-        break;                  /* done parsing */
-      }
-      else if (buffer[2 + blen] == 0) {
-        new = mutt_read_mime_header (fp, digest);
+            /* Remove any trailing whitespace, up to the length of the boundary */
+            for (i = len - 1; ISSPACE(buffer[i]) && i >= blen + 2; i--)
+                buffer[i] = '\0';
 
-        /*
-         * Consistency checking - catch
-         * bad attachment end boundaries
-         */
+            /* Check for the end boundary */
+            final = buffer[blen + 3] == '-' && buffer[blen + 4] == '-';
+            if (final)
+                break;
 
-        if (new->offset > end_off) {
-          mutt_free_body (&new);
-          break;
-        }
-        if (head) {
-          last->next = new;
-          last = new;
+            if (buffer[2 + blen] == '\0') {
+                BODY *new = mutt_read_mime_header(fp, digest);
+
+                /*
+                 * Consistency checking - catch
+                 * bad attachment end boundaries
+                 */
+
+                if (new->offset > end_off) {
+                    mutt_free_body(&new);
+                    break;
+                }
+
+                if (*last)
+                    last = &(*last)->next;
+                *last = new;
+            }
         }
-        else
-          last = head = new;
-      }
     }
-  }
 
-  /* in case of missing end boundary, set the length to something reasonable */
-  if (last && last->length == 0 && !final)
-    last->length = end_off - last->offset;
+    /* in case of missing end boundary, set the length to something reasonable */
+    if (*last && (*last)->length == 0 && !final)
+        (*last)->length = end_off - (*last)->offset;
 
-  /* parse recursive MIME parts */
-  for (last = head; last; last = last->next)
-    mutt_parse_part (fp, last);
+    /* parse recursive MIME parts */
+    {
+        BODY *b;
+        for (b = head; b; b = b->next)
+            mutt_parse_part(fp, b);
+    }
 
-  return (head);
+    return (head);
 }
 
-static const char *uncomment_timezone (char *buf, size_t buflen,
-                                       const char *tz)
+static const char *
+uncomment_timezone(char *buf, size_t buflen, const char *tz)
 {
-  char *p;
-  size_t len;
-
-  if (*tz != '(')
-    return tz;                  /* no need to do anything */
-  tz = vskipspaces(tz + 1);
-  if ((p = strpbrk (tz, " )")) == NULL)
-    return tz;
-  len = p - tz;
-  if (len > buflen - 1)
-    len = buflen - 1;
-  memcpy (buf, tz, len);
-  buf[len] = 0;
-  return buf;
+    char *p;
+
+    if (*tz != '(')
+        return tz;                  /* no need to do anything */
+
+    tz = vskipspaces(tz + 1);
+    p = strpbrk(tz, " )");
+    if (!p)
+        return tz;
+
+    m_strncpy(buf, buflen, tz, p - tz);
+    return buf;
 }
 
 static struct tz_t {
-  char tzname[5];
-  unsigned char zhours;
-  unsigned char zminutes;
-  unsigned char zoccident;      /* west of UTC? */
+    char tzname[5];
+    unsigned char zhours;
+    unsigned char zminutes;
+    unsigned char zoccident;      /* west of UTC? */
 } TimeZones[] = {
-  {
-  "aat", 1, 0, 1},              /* Atlantic Africa Time */
-  {
-  "adt", 4, 0, 0},              /* Arabia DST */
-  {
-  "ast", 3, 0, 0},              /* Arabia */
+    {"aat", 1, 0, 1},             /* Atlantic Africa Time */
+    {"adt", 4, 0, 0},             /* Arabia DST */
+    {"ast", 3, 0, 0},             /* Arabia */
     /*{ "ast",   4,  0, 1 }, *//* Atlantic */
-  {
-  "bst", 1, 0, 0},              /* British DST */
-  {
-  "cat", 1, 0, 0},              /* Central Africa */
-  {
-  "cdt", 5, 0, 1}, {
-  "cest", 2, 0, 0},             /* Central Europe DST */
-  {
-  "cet", 1, 0, 0},              /* Central Europe */
-  {
-  "cst", 6, 0, 1},
+    {"bst", 1, 0, 0},             /* British DST */
+    {"cat", 1, 0, 0},             /* Central Africa */
+    {"cdt", 5, 0, 1},
+    {"cest", 2, 0, 0},            /* Central Europe DST */
+    {"cet", 1, 0, 0},             /* Central Europe */
+    {"cst", 6, 0, 1},
     /*{ "cst",   8,  0, 0 }, *//* China */
     /*{ "cst",   9, 30, 0 }, *//* Australian Central Standard Time */
-  {
-  "eat", 3, 0, 0},              /* East Africa */
-  {
-  "edt", 4, 0, 1}, {
-  "eest", 3, 0, 0},             /* Eastern Europe DST */
-  {
-  "eet", 2, 0, 0},              /* Eastern Europe */
-  {
-  "egst", 0, 0, 0},             /* Eastern Greenland DST */
-  {
-  "egt", 1, 0, 1},              /* Eastern Greenland */
-  {
-  "est", 5, 0, 1}, {
-  "gmt", 0, 0, 0}, {
-  "gst", 4, 0, 0},              /* Presian Gulf */
-  {
-  "hkt", 8, 0, 0},              /* Hong Kong */
-  {
-  "ict", 7, 0, 0},              /* Indochina */
-  {
-  "idt", 3, 0, 0},              /* Israel DST */
-  {
-  "ist", 2, 0, 0},              /* Israel */
+    {"eat", 3, 0, 0},             /* East Africa */
+    {"edt", 4, 0, 1},
+    {"eest", 3, 0, 0},            /* Eastern Europe DST */
+    {"eet", 2, 0, 0},             /* Eastern Europe */
+    {"egst", 0, 0, 0},            /* Eastern Greenland DST */
+    {"egt", 1, 0, 1},             /* Eastern Greenland */
+    {"est", 5, 0, 1},
+    {"gmt", 0, 0, 0},
+    {"gst", 4, 0, 0},             /* Presian Gulf */
+    {"hkt", 8, 0, 0},             /* Hong Kong */
+    {"ict", 7, 0, 0},             /* Indochina */
+    {"idt", 3, 0, 0},             /* Israel DST */
+    {"ist", 2, 0, 0},             /* Israel */
     /*{ "ist",   5, 30, 0 }, *//* India */
-  {
-  "jst", 9, 0, 0},              /* Japan */
-  {
-  "kst", 9, 0, 0},              /* Korea */
-  {
-  "mdt", 6, 0, 1}, {
-  "met", 1, 0, 0},              /* this is now officially CET */
-  {
-  "msd", 4, 0, 0},              /* Moscow DST */
-  {
-  "msk", 3, 0, 0},              /* Moscow */
-  {
-  "mst", 7, 0, 1}, {
-  "nzdt", 13, 0, 0},            /* New Zealand DST */
-  {
-  "nzst", 12, 0, 0},            /* New Zealand */
-  {
-  "pdt", 7, 0, 1}, {
-  "pst", 8, 0, 1}, {
-  "sat", 2, 0, 0},              /* South Africa */
-  {
-  "smt", 4, 0, 0},              /* Seychelles */
-  {
-  "sst", 11, 0, 1},             /* Samoa */
+    {"jst", 9, 0, 0},             /* Japan */
+    {"kst", 9, 0, 0},             /* Korea */
+    {"mdt", 6, 0, 1},
+    {"met", 1, 0, 0},             /* this is now officially CET */
+    {"msd", 4, 0, 0},             /* Moscow DST */
+    {"msk", 3, 0, 0},             /* Moscow */
+    {"mst", 7, 0, 1},
+    {"nzdt", 13, 0, 0},           /* New Zealand DST */
+    {"nzst", 12, 0, 0},           /* New Zealand */
+    {"pdt", 7, 0, 1},
+    {"pst", 8, 0, 1},
+    {"sat", 2, 0, 0},             /* South Africa */
+    {"smt", 4, 0, 0},             /* Seychelles */
+    {"sst", 11, 0, 1},            /* Samoa */
     /*{ "sst",   8,  0, 0 }, *//* Singapore */
-  {
-  "utc", 0, 0, 0}, {
-  "wat", 0, 0, 0},              /* West Africa */
-  {
-  "west", 1, 0, 0},             /* Western Europe DST */
-  {
-  "wet", 0, 0, 0},              /* Western Europe */
-  {
-  "wgst", 2, 0, 1},             /* Western Greenland DST */
-  {
-  "wgt", 3, 0, 1},              /* Western Greenland */
-  {
-  "wst", 8, 0, 0},              /* Western Australia */
+    {"utc", 0, 0, 0},
+    {"wat", 0, 0, 0},             /* West Africa */
+    {"west", 1, 0, 0},            /* Western Europe DST */
+    {"wet", 0, 0, 0},             /* Western Europe */
+    {"wgst", 2, 0, 1},            /* Western Greenland DST */
+    {"wgt", 3, 0, 1},             /* Western Greenland */
+    {"wst", 8, 0, 0},             /* Western Australia */
 };
 
 /* parses a date string in RFC822 format:
@@ -727,503 +668,397 @@ static struct tz_t {
  * This routine assumes that `h' has been initialized to 0.  the `timezone'
  * field is optional, defaulting to +0000 if missing.
  */
-time_t mutt_parse_date (const char *s, HEADER * h)
+time_t mutt_parse_date(const char *s, HEADER *h)
 {
-  int count = 0;
-  char *t;
-  int hour, min, sec;
-  struct tm tm;
-  int i;
-  int tz_offset = 0;
-  int zhours = 0;
-  int zminutes = 0;
-  int zoccident = 0;
-  const char *ptz;
-  char tzstr[SHORT_STRING];
-  char scratch[SHORT_STRING];
-
-  /* Don't modify our argument. Fixed-size buffer is ok here since
-   * the date format imposes a natural limit.
-   */
-
-  m_strcpy(scratch, sizeof(scratch), s);
-
-  /* kill the day of the week, if it exists. */
-  if ((t = strchr (scratch, ',')))
-    t++;
-  else
-    t = scratch;
-  t = vskipspaces(t);
-
-  p_clear(&tm, 1);
-
-  while ((t = strtok (t, " \t")) != NULL) {
-    switch (count) {
-    case 0:                    /* day of the month */
-      if (!isdigit ((unsigned char) *t))
-        return (-1);
-      tm.tm_mday = atoi (t);
-      if (tm.tm_mday > 31)
-        return (-1);
-      break;
-
-    case 1:                    /* month of the year */
-      if ((i = mutt_check_month (t)) < 0)
-        return (-1);
-      tm.tm_mon = i;
-      break;
-
-    case 2:                    /* year */
-      tm.tm_year = atoi (t);
-      if (tm.tm_year < 50)
-        tm.tm_year += 100;
-      else if (tm.tm_year >= 1900)
-        tm.tm_year -= 1900;
-      break;
-
-    case 3:                    /* time of day */
-      if (sscanf (t, "%d:%d:%d", &hour, &min, &sec) == 3);
-      else if (sscanf (t, "%d:%d", &hour, &min) == 2)
-        sec = 0;
-      else {
-        debug_print (1, ("could not process time format: %s\n", t));
-        return (-1);
-      }
-      tm.tm_hour = hour;
-      tm.tm_min = min;
-      tm.tm_sec = sec;
-      break;
-
-    case 4:                    /* timezone */
-      /* sometimes we see things like (MST) or (-0700) so attempt to
-       * compensate by uncommenting the string if non-RFC822 compliant
-       */
-      ptz = uncomment_timezone (tzstr, sizeof (tzstr), t);
-
-      if (*ptz == '+' || *ptz == '-') {
-        if (ptz[1] && ptz[2] && ptz[3] && ptz[4]
-            && isdigit ((unsigned char) ptz[1])
-            && isdigit ((unsigned char) ptz[2])
-            && isdigit ((unsigned char) ptz[3])
-            && isdigit ((unsigned char) ptz[4])) {
-          zhours = (ptz[1] - '0') * 10 + (ptz[2] - '0');
-          zminutes = (ptz[3] - '0') * 10 + (ptz[4] - '0');
-
-          if (ptz[0] == '-')
-            zoccident = 1;
-        }
-      }
-      else {
-        struct tz_t *tz;
-
-        tz = bsearch (ptz, TimeZones, sizeof TimeZones / sizeof (struct tz_t),
-                      sizeof (struct tz_t),
-                      (int (*)(const void *, const void *)) ascii_strcasecmp
-                      /* This is safe to do: A pointer to a struct equals
-                       * a pointer to its first element*/ );
-
-        if (tz) {
-          zhours = tz->zhours;
-          zminutes = tz->zminutes;
-          zoccident = tz->zoccident;
-        }
+    int zhours = 0, zminutes = 0, zoccident = 0;
+    char scratch[SHORT_STRING];
+    struct tm tm;
+    int count = 0;
+    char *p;
+
+    /* Don't modify our argument. Fixed-size buffer is ok here since
+       the date format imposes a natural limit.  */
+
+    m_strcpy(scratch, sizeof(scratch), s);
+
+    /* kill the day of the week, if it exists. */
+    p = strchr(scratch, ',');
+    p = vskipspaces(p ? p + 1 : scratch);
+
+    p_clear(&tm, 1);
+
+    while ((p = strtok (p, " \t")) != NULL) {
+        char tzstr[SHORT_STRING];
+        const char *ptz;
+
+        switch (count) {
+          case 0:                    /* day of the month */
+            if (!isdigit((unsigned char)*p))
+                return -1;
+            tm.tm_mday = atoi(p);
+            if (tm.tm_mday > 31)
+                return -1;
+            break;
+
+          case 1:                    /* month of the year */
+            tm.tm_mon = mutt_check_month(p);
+            if (tm.tm_mon < 0)
+                return -1;
+            break;
+
+          case 2:                    /* year */
+            tm.tm_year = atoi(p);
+            if (tm.tm_year < 50)
+                tm.tm_year += 100;
+            else if (tm.tm_year >= 1900)
+                tm.tm_year -= 1900;
+            break;
 
-        /* ad hoc support for the European MET (now officially CET) TZ */
-        if (ascii_strcasecmp (t, "MET") == 0) {
-          if ((t = strtok (NULL, " \t")) != NULL) {
-            if (!ascii_strcasecmp (t, "DST"))
-              zhours++;
-          }
+          case 3:                    /* time of day */
+            tm.tm_hour = strtol(p, &p, 10);
+            if (*p++ != ':')
+                return -1;
+            tm.tm_min  = strtol(p, &p, 10);
+            if (*p++ == ':') {
+                tm.tm_sec = strtol(p, &p, 10);
+            } else {
+                tm.tm_sec = 0;
+            }
+            break;
+
+          case 4:                    /* timezone */
+            /* sometimes we see things like (MST) or (-0700) so attempt to
+             * compensate by uncommenting the string if non-RFC822 compliant
+             */
+            ptz = uncomment_timezone(tzstr, sizeof(tzstr), p);
+
+            if (*ptz == '+' || *ptz == '-') {
+                if (isdigit((unsigned char)ptz[1])
+                &&  isdigit((unsigned char)ptz[2])
+                &&  isdigit((unsigned char)ptz[3])
+                &&  isdigit((unsigned char)ptz[4]))
+                {
+                    zoccident = ptz[0] == '-';
+                    zhours    = (ptz[1] - '0') * 10 + (ptz[2] - '0');
+                    zminutes  = (ptz[3] - '0') * 10 + (ptz[4] - '0');
+                }
+            } else {
+                struct tz_t *tz;
+
+                /* This is safe to do: A pointer to a struct equals a pointer to its
+                 * first element*/
+                tz = bsearch(ptz, TimeZones, countof(TimeZones), sizeof(TimeZones[0]),
+                             (int (*)(const void *, const void *))ascii_strcasecmp);
+
+                if (tz) {
+                    zhours = tz->zhours;
+                    zminutes = tz->zminutes;
+                    zoccident = tz->zoccident;
+                }
+
+                /* ad hoc support for the European MET (now officially CET) TZ */
+                if (ascii_strcasecmp(p, "MET") == 0) {
+                    if ((p = strtok (NULL, " \t")) && !ascii_strcasecmp(p, "DST")) {
+                        zhours++;
+                    }
+                }
+            }
+            break;
         }
-      }
-      tz_offset = zhours * 3600 + zminutes * 60;
-      if (!zoccident)
-        tz_offset = -tz_offset;
-      break;
+        count++;
+        p = NULL;
     }
-    count++;
-    t = 0;
-  }
 
-  if (count < 4) {              /* don't check for missing timezone */
-    debug_print (1, ("error parsing date format, using received time\n"));
-    return (-1);
-  }
+    if (count < 4) {  /* don't check for missing timezone */
+        debug_print (1, ("error parsing date format, using received time\n"));
+        return -1;
+    }
 
-  if (h) {
-    h->zhours = zhours;
-    h->zminutes = zminutes;
-    h->zoccident = zoccident;
-  }
+    if (h) {
+        h->zhours    = zhours;
+        h->zminutes  = zminutes;
+        h->zoccident = zoccident;
+    }
 
-  return (mutt_mktime (&tm, 0) + tz_offset);
+    return mutt_mktime(&tm, 0) + (zoccident ? 1 : -1) * (zhours * 3600 + zminutes * 60);
 }
 
-/* extract the first substring that looks like a message-id */
-static char *extract_message_id(const char *s)
+#include "rfc822hdrs.h"
+
+LIST **mutt_parse_rfc822_line(ENVELOPE *e, HEADER *hdr, char *line, char *p,
+                              short weed, short do_2047, LIST **user_hdrs)
 {
-    const char *p;
+    switch (rfc822_well_known(line)) {
+      case HDR_APPARENTLY_FROM:
+        e->from = rfc822_parse_adrlist (e->from, p);
+        break;
 
-    if ((s = strchr(s, '<')) == NULL || (p = strchr(s, '>')) == NULL)
-        return NULL;
-    return p_dupstr(s, (p - s) + 1);
-}
+      case HDR_APPARENTLY_TO:
+        e->to = rfc822_parse_adrlist (e->to, p);
+        break;
 
-void mutt_parse_mime_message (CONTEXT * ctx, HEADER * cur)
-{
-  MESSAGE *msg;
-  int flags = 0;
+      case HDR_BCC:
+        e->bcc = rfc822_parse_adrlist (e->bcc, p);
+        break;
 
-  do {
-    if (cur->content->type != TYPEMESSAGE
-        && cur->content->type != TYPEMULTIPART)
-      break;                     /* nothing to do */
+      case HDR_CC:
+        e->cc = rfc822_parse_adrlist (e->cc, p);
+        break;
+
+      case HDR_CONTENT_DESCRIPTION:
+        if (hdr) {
+            m_strreplace(&hdr->content->description, p);
+            rfc2047_decode(&hdr->content->description);
+        }
+        break;
 
-    if (cur->content->parts)
-      break;                     /* The message was parsed earlier. */
+      case HDR_CONTENT_DISPOSITION:
+        if (hdr)
+            parse_content_disposition(p, hdr->content);
+        break;
 
-    if ((msg = mx_open_message (ctx, cur->msgno))) {
-      mutt_parse_part (msg->fp, cur->content);
+      case HDR_CONTENT_LENGTH:
+        if (hdr) {
+            if ((hdr->content->length = atoi(p)) < 0)
+                hdr->content->length = -1;
+        }
+        break;
 
-      cur->security = crypt_query (cur->content);
+      case HDR_CONTENT_TRANSFER_ENCODING:
+        if (hdr)
+            hdr->content->encoding = mutt_check_encoding(p);
+        break;
 
-      mx_close_message (&msg);
-    }
-  } while (0);
-  mutt_count_body_parts (cur, flags | M_PARTS_RECOUNT);
-}
+      case HDR_CONTENT_TYPE:
+        if (hdr)
+            mutt_parse_content_type (p, hdr->content);
+        break;
 
-int mutt_parse_rfc822_line (ENVELOPE * e, HEADER * hdr, char *line, char *p,
-                            short user_hdrs, short weed, short do_2047,
-                            LIST ** lastp)
-{
-  int matched = 0;
-  LIST *last = NULL;
+      case HDR_DATE:
+        m_strreplace(&e->date, p);
+        if (hdr)
+            hdr->date_sent = mutt_parse_date (p, hdr);
+        break;
 
-  if (lastp)
-    last = *lastp;
+      case HDR_EXPIRES:
+        if (hdr && mutt_parse_date (p, NULL) < time (NULL))
+            hdr->expired = 1;
+        break;
 
-  switch (ascii_tolower (line[0])) {
-  case 'a':
-    if (ascii_strcasecmp (line + 1, "pparently-to") == 0) {
-      e->to = rfc822_parse_adrlist (e->to, p);
-      matched = 1;
-    }
-    else if (ascii_strcasecmp (line + 1, "pparently-from") == 0) {
-      e->from = rfc822_parse_adrlist (e->from, p);
-      matched = 1;
-    }
-    break;
+#ifdef USE_NNTP
+      case HDR_FOLLOWUP_TO:
+        if (!e->followup_to) {
+            m_strrtrim(p);
+            e->followup_to = m_strdup(skipspaces(p));
+        }
+        break;
+#endif
 
-  case 'b':
-    if (ascii_strcasecmp (line + 1, "cc") == 0) {
-      e->bcc = rfc822_parse_adrlist (e->bcc, p);
-      matched = 1;
-    }
-    break;
+      case HDR_FROM:
+        e->from = rfc822_parse_adrlist(e->from, p);
+        /* don't leave from info NULL if there's an invalid address (or
+         * whatever) in From: field; mutt would just display it as empty
+         * and mark mail/(esp.) news article as your own. aaargh! this
+         * bothered me for _years_ */
+        if (!e->from) {
+            e->from = address_new();
+            e->from->personal = m_strdup(p);
+        }
+        break;
 
-  case 'c':
-    if (ascii_strcasecmp (line + 1, "c") == 0) {
-      e->cc = rfc822_parse_adrlist (e->cc, p);
-      matched = 1;
-    }
-    else if (ascii_strncasecmp (line + 1, "ontent-", 7) == 0) {
-      if (ascii_strcasecmp (line + 8, "type") == 0) {
-        if (hdr)
-          mutt_parse_content_type (p, hdr->content);
-        matched = 1;
-      }
-      else if (ascii_strcasecmp (line + 8, "transfer-encoding") == 0) {
-        if (hdr)
-          hdr->content->encoding = mutt_check_encoding (p);
-        matched = 1;
-      }
-      else if (ascii_strcasecmp (line + 8, "length") == 0) {
+      case HDR_IN_REPLY_TO:
+        mutt_free_list(&e->in_reply_to);
+        e->in_reply_to = mutt_parse_references(p, 1);
+        break;
+
+      case HDR_LINES:
         if (hdr) {
-          if ((hdr->content->length = atoi (p)) < 0)
-            hdr->content->length = -1;
+            /* HACK - mutt has, for a very short time, produced negative
+               Lines header values.  Ignore them. */
+            hdr->lines = MAX(0, atoi(p));
         }
-        matched = 1;
-      }
-      else if (ascii_strcasecmp (line + 8, "description") == 0) {
-        if (hdr) {
-          m_strreplace(&hdr->content->description, p);
-          rfc2047_decode (&hdr->content->description);
+        break;
+
+      case HDR_LIST_POST:
+        /* RFC 2369.  FIXME: We should ignore whitespace, but don't. */
+        if (strncmp(p, "NO", 2)) {
+            char *beg, *end;
+
+            for (beg = strchr (p, '<'); beg; beg = strchr (end, ',')) {
+                ++beg;
+                if (!(end = strchr (beg, '>')))
+                    break;
+
+                /* Take the first mailto URL */
+                if (url_check_scheme (beg) == U_MAILTO) {
+                    p_delete(&e->list_post);
+                    e->list_post = p_dupstr(beg, end - beg);
+                    break;
+                }
+            }
         }
-        matched = 1;
-      }
-      else if (ascii_strcasecmp (line + 8, "disposition") == 0) {
+        break;
+
+      case HDR_MAIL_FOLLOWUP_TO:
+        e->mail_followup_to = rfc822_parse_adrlist(e->mail_followup_to, p);
+        break;
+
+      case HDR_MAIL_REPLY_TO:
+        address_delete (&e->reply_to);
+        e->reply_to = rfc822_parse_adrlist(e->reply_to, p);
+        break;
+
+      case HDR_MESSAGE_ID:
+        {
+            const char *beg, *end;
+
+            /* We add a new "Message-ID:" when building a message */
+            p_delete(&e->message_id);
+
+            if ((beg = strchr(p, '<')) && (end = strchr(beg, '>')))
+                e->message_id = p_dupstr(beg, (end - beg) + 1);
+        }
+        break;
+
+      case HDR_MIME_VERSION:
         if (hdr)
-          parse_content_disposition (p, hdr->content);
-        matched = 1;
-      }
-    }
-    break;
-
-  case 'd':
-    if (!ascii_strcasecmp ("ate", line + 1)) {
-      m_strreplace(&e->date, p);
-      if (hdr)
-        hdr->date_sent = mutt_parse_date (p, hdr);
-      matched = 1;
-    }
-    break;
-
-  case 'e':
-    if (!ascii_strcasecmp ("xpires", line + 1) &&
-        hdr && mutt_parse_date (p, NULL) < time (NULL))
-      hdr->expired = 1;
-    break;
-
-  case 'f':
-    if (!ascii_strcasecmp ("rom", line + 1)) {
-      e->from = rfc822_parse_adrlist (e->from, p);
-      /* don't leave from info NULL if there's an invalid address (or
-       * whatever) in From: field; mutt would just display it as empty
-       * and mark mail/(esp.) news article as your own. aaargh! this
-       * bothered me for _years_ */
-      if (!e->from) {
-        e->from = address_new ();
-        e->from->personal = m_strdup(p);
-      }
-      matched = 1;
-    }
+            hdr->mime = 1;
+        break;
+
 #ifdef USE_NNTP
-    else if (!m_strcasecmp(line + 1, "ollowup-to")) {
-      if (!e->followup_to) {
+      case HDR_NEWSGROUPS:
+        p_delete(&e->newsgroups);
         m_strrtrim(p);
-        e->followup_to = m_strdup(skipspaces(p));
-      }
-      matched = 1;
-    }
+        e->newsgroups = m_strdup(skipspaces(p));
+        break;
 #endif
-    break;
 
-  case 'i':
-    if (!ascii_strcasecmp (line + 1, "n-reply-to")) {
-      mutt_free_list (&e->in_reply_to);
-      e->in_reply_to = mutt_parse_references (p, 1);
-      matched = 1;
-    }
-    break;
+      case HDR_ORGANIZATION:
+        if (!e->organization && m_strcasecmp(p, "unknown"))
+            e->organization = m_strdup(p);
+        break;
+
+      case HDR_RECEIVED:
+        if (hdr && !hdr->received) {
+            char *d = strchr(p, ';');
+            if (d)
+                hdr->received = mutt_parse_date(d + 1, NULL);
+        }
+        break;
 
-  case 'l':
-    if (!ascii_strcasecmp (line + 1, "ines")) {
-      if (hdr) {
-        hdr->lines = atoi (p);
+      case HDR_REFERENCES:
+        mutt_free_list(&e->references);
+        e->references = mutt_parse_references(p, 0);
+        break;
 
-        /*
-         * HACK - mutt has, for a very short time, produced negative
-         * Lines header values.  Ignore them.
-         */
-        if (hdr->lines < 0)
-          hdr->lines = 0;
-      }
+      case HDR_REPLY_TO:
+        e->reply_to = rfc822_parse_adrlist(e->reply_to, p);
+        break;
 
-      matched = 1;
-    }
-    else if (!ascii_strcasecmp (line + 1, "ist-Post")) {
-      /* RFC 2369.  FIXME: We should ignore whitespace, but don't. */
-      if (strncmp (p, "NO", 2)) {
-        char *beg, *end;
-
-        for (beg = strchr (p, '<'); beg; beg = strchr (end, ',')) {
-          ++beg;
-          if (!(end = strchr (beg, '>')))
-            break;
+      case HDR_RETURN_PATH:
+        e->return_path = rfc822_parse_adrlist(e->return_path, p);
+        break;
 
-          /* Take the first mailto URL */
-          if (url_check_scheme (beg) == U_MAILTO) {
-            p_delete(&e->list_post);
-            e->list_post = p_dupstr(beg, end - beg);
-            break;
-          }
+      case HDR_SENDER:
+        e->sender = rfc822_parse_adrlist (e->sender, p);
+        break;
+
+      case HDR_STATUS:
+        if (hdr) {
+            while (*p) {
+                switch (*p) {
+                  case 'r':
+                    hdr->replied = 1;
+                    break;
+                  case 'O':
+                    hdr->old = 1;
+                    break;
+                  case 'R':
+                    hdr->read = 1;
+                    break;
+                }
+                p++;
+            }
         }
-      }
-      matched = 1;
-    }
-    break;
+        break;
 
-  case 'm':
-    if (!ascii_strcasecmp (line + 1, "ime-version")) {
-      if (hdr)
-        hdr->mime = 1;
-      matched = 1;
-    }
-    else if (!ascii_strcasecmp (line + 1, "essage-id")) {
-      /* We add a new "Message-ID:" when building a message */
-      p_delete(&e->message_id);
-      e->message_id = extract_message_id (p);
-      matched = 1;
-    }
-    else if (!ascii_strncasecmp (line + 1, "ail-", 4)) {
-      if (!ascii_strcasecmp (line + 5, "reply-to")) {
-        /* override the Reply-To: field */
-        address_delete (&e->reply_to);
-        e->reply_to = rfc822_parse_adrlist (e->reply_to, p);
-        matched = 1;
-      }
-      else if (!ascii_strcasecmp (line + 5, "followup-to")) {
-        e->mail_followup_to = rfc822_parse_adrlist (e->mail_followup_to, p);
-        matched = 1;
-      }
-    }
-    break;
+      case HDR_SUBJECT:
+        if (!e->subject)
+            e->subject = m_strdup(p);
+        break;
 
-#ifdef USE_NNTP
-  case 'n':
-    if (!m_strcasecmp(line + 1, "ewsgroups")) {
-      p_delete(&e->newsgroups);
-      m_strrtrim(p);
-      e->newsgroups = m_strdup(skipspaces(p));
-      matched = 1;
-    }
-    break;
-#endif
+      case HDR_SUPERCEDES:
+      case HDR_SUPERSEDES:
+        if (hdr)
+            e->supersedes = m_strdup(p);
+        break;
 
-  case 'o':
-    /* field `Organization:' saves only for pager! */
-    if (!m_strcasecmp(line + 1, "rganization")) {
-      if (!e->organization && m_strcasecmp(p, "unknown"))
-        e->organization = m_strdup(p);
-    }
-    break;
+      case HDR_TO:
+        e->to = rfc822_parse_adrlist(e->to, p);
+        break;
 
-  case 'r':
-    if (!ascii_strcasecmp (line + 1, "eferences")) {
-      mutt_free_list (&e->references);
-      e->references = mutt_parse_references (p, 0);
-      matched = 1;
-    }
-    else if (!ascii_strcasecmp (line + 1, "eply-to")) {
-      e->reply_to = rfc822_parse_adrlist (e->reply_to, p);
-      matched = 1;
-    }
-    else if (!ascii_strcasecmp (line + 1, "eturn-path")) {
-      e->return_path = rfc822_parse_adrlist (e->return_path, p);
-      matched = 1;
-    }
-    else if (!ascii_strcasecmp (line + 1, "eceived")) {
-      if (hdr && !hdr->received) {
-        char *d = strchr (p, ';');
+#ifdef USE_NNTP
+      case HDR_X_COMMENT_TO:
+        if (!e->x_comment_to)
+            e->x_comment_to = m_strdup(p);
+        break;
+#endif
 
-        if (d)
-          hdr->received = mutt_parse_date (d + 1, NULL);
-      }
-    }
-    break;
+      case HDR_X_LABEL:
+        e->x_label = m_strdup(p);
+        break;
 
-  case 's':
-    if (!ascii_strcasecmp (line + 1, "ubject")) {
-      if (!e->subject)
-        e->subject = m_strdup(p);
-      matched = 1;
-    }
-    else if (!ascii_strcasecmp (line + 1, "ender")) {
-      e->sender = rfc822_parse_adrlist (e->sender, p);
-      matched = 1;
-    }
-    else if (!ascii_strcasecmp (line + 1, "tatus")) {
-      if (hdr) {
-        while (*p) {
-          switch (*p) {
-          case 'r':
-            hdr->replied = 1;
-            break;
-          case 'O':
-            hdr->old = 1;
-            break;
-          case 'R':
-            hdr->read = 1;
-            break;
-          }
-          p++;
-        }
-      }
-      matched = 1;
-    }
-    else if ((!ascii_strcasecmp ("upersedes", line + 1) ||
-              !ascii_strcasecmp ("upercedes", line + 1)) && hdr)
-      e->supersedes = m_strdup(p);
-    break;
-
-  case 't':
-    if (ascii_strcasecmp (line + 1, "o") == 0) {
-      e->to = rfc822_parse_adrlist (e->to, p);
-      matched = 1;
-    }
-    break;
-
-  case 'x':
-    if (ascii_strcasecmp (line + 1, "-status") == 0) {
-      if (hdr) {
-        while (*p) {
-          switch (*p) {
-          case 'A':
-            hdr->replied = 1;
-            break;
-          case 'D':
-            hdr->deleted = 1;
-            break;
-          case 'F':
-            hdr->flagged = 1;
-            break;
-          default:
-            break;
-          }
-          p++;
-        }
-      }
-      matched = 1;
-    }
-    else if (ascii_strcasecmp (line + 1, "-label") == 0) {
-      e->x_label = m_strdup(p);
-      matched = 1;
-    }
 #ifdef USE_NNTP
-    else if (!m_strcasecmp(line + 1, "-comment-to")) {
-      if (!e->x_comment_to)
-        e->x_comment_to = m_strdup(p);
-      matched = 1;
-    }
-    else if (!m_strcasecmp(line + 1, "ref")) {
-      if (!e->xref)
-        e->xref = m_strdup(p);
-      matched = 1;
-    }
+      case HDR_XREF:
+        if (!e->xref)
+            e->xref = m_strdup(p);
+        break;
 #endif
 
-  default:
-    break;
-  }
+      case HDR_X_STATUS:
+        if (hdr) {
+            while (*p) {
+                switch (*p) {
+                  case 'A':
+                    hdr->replied = 1;
+                    break;
+                  case 'D':
+                    hdr->deleted = 1;
+                    break;
+                  case 'F':
+                    hdr->flagged = 1;
+                    break;
+                  default:
+                    break;
+                }
+                p++;
+            }
+        }
+        break;
+
+      default:
+        if (!user_hdrs)
+            break;
 
-  /* Keep track of the user-defined headers */
-  if (!matched && user_hdrs) {
-    /* restore the original line */
-    line[m_strlen(line)] = ':';
+        /* restore the original line */
+        line[m_strlen(line)] = ':';
 
-    if (weed && option (OPTWEED) && mutt_matches_ignore (line, Ignore)
-        && !mutt_matches_ignore (line, UnIgnore))
-      goto done;
+        if (weed && option(OPTWEED) && mutt_matches_ignore(line, Ignore)
+        && !mutt_matches_ignore(line, UnIgnore)) {
+            break;
+        }
 
-    if (last) {
-      last->next = mutt_new_list ();
-      last = last->next;
+        *user_hdrs = mutt_new_list();
+        (*user_hdrs)->data = m_strdup(line);
+        if (do_2047)
+            rfc2047_decode(&(*user_hdrs)->data);
+        (*user_hdrs)->next = mutt_new_list();
+        return &(*user_hdrs)->next;
     }
-    else
-      last = e->userhdrs = mutt_new_list ();
-    last->data = m_strdup(line);
-    if (do_2047)
-      rfc2047_decode (&last->data);
-  }
-
-done:
 
-  *lastp = last;
-  return matched;
+    return user_hdrs;
 }
 
-
 /* mutt_read_rfc822_header() -- parses a RFC822 header
  *
  * Args:
@@ -1240,168 +1075,133 @@ done:
  *             Used for recall-message.
  *
  * Returns:     newly allocated envelope structure.  You should free it by
- *              mutt_free_envelope() when envelope stay unneeded.
+ *              envelope_delete() when envelope stay unneeded.
  */
-ENVELOPE *mutt_read_rfc822_header (FILE * f, HEADER * hdr, short user_hdrs,
-                                   short weed)
+ENVELOPE *
+mutt_read_rfc822_header(FILE *f, HEADER *hdr, short user_hdrs, short weed)
 {
-  ENVELOPE *e = mutt_new_envelope ();
-  LIST *last = NULL;
-  char *line = p_new(char, LONG_STRING);
-  char *p;
-  off_t loc;
-  int matched;
-  size_t linelen = LONG_STRING;
-  char buf[LONG_STRING + 1];
-
-  if (hdr) {
-    if (hdr->content == NULL) {
-      hdr->content = mutt_new_body ();
-
-      /* set the defaults from RFC1521 */
-      hdr->content->type = TYPETEXT;
-      hdr->content->subtype = m_strdup("plain");
-      hdr->content->encoding = ENC7BIT;
-      hdr->content->length = -1;
-
-      /* RFC 2183 says this is arbitrary */
-      hdr->content->disposition = DISPINLINE;
-    }
-  }
+    ENVELOPE *e = envelope_new();
+    LIST **last = user_hdrs ? &e->userhdrs : NULL;
 
-  while ((loc = ftello (f)),
-         *(line = mutt_read_rfc822_line (f, line, &linelen)) != 0) {
-    matched = 0;
-
-    if ((p = strpbrk (line, ": \t")) == NULL || *p != ':') {
-      char return_path[LONG_STRING];
-      time_t t;
-
-      /* some bogus MTAs will quote the original "From " line */
-      if (m_strncmp(">From ", line, 6) == 0)
-        continue;               /* just ignore */
-      else if (is_from (line, return_path, sizeof (return_path), &t)) {
-        /* MH somtimes has the From_ line in the middle of the header! */
-        if (hdr && !hdr->received)
-          hdr->received = t - mutt_local_tz (t);
-        continue;
-      }
+    char *line = p_new(char, LONG_STRING);
+    ssize_t linelen = LONG_STRING;
+    off_t loc;
 
-      fseeko (f, loc, 0);
-      break;                    /* end of header */
+    if (hdr && !hdr->content) {
+        hdr->content = mutt_new_body ();
+
+        /* set the defaults from RFC1521 */
+        hdr->content->type     = TYPETEXT;
+        hdr->content->subtype  = m_strdup("plain");
+        hdr->content->encoding = ENC7BIT;
+        hdr->content->length   = -1;
+
+        /* RFC 2183 says this is arbitrary */
+        hdr->content->disposition = DISPINLINE;
     }
 
-    *buf = '\0';
-
-    if (mutt_match_spam_list (line, SpamList, buf, sizeof (buf))) {
-      if (!rx_list_match (NoSpamList, line)) {
-
-        /* if spam tag already exists, figure out how to amend it */
-        if (e->spam && *buf) {
-          /* If SpamSep defined, append with separator */
-          if (SpamSep) {
-            mutt_buffer_addstr (e->spam, SpamSep);
-            mutt_buffer_addstr (e->spam, buf);
-          }
-
-          /* else overwrite */
-          else {
-            e->spam->dptr = e->spam->data;
-            *e->spam->dptr = '\0';
-            mutt_buffer_addstr (e->spam, buf);
-          }
-        }
+    while ((loc = ftello(f)),
+           mutt_read_rfc822_line(f, &line, &linelen))
+    {
+        char buf[LONG_STRING + 1] = "";
+        char *p;
+
+        p = strpbrk(line, ": \t");
+        if (!p || *p != ':') {
+            char return_path[LONG_STRING];
+            time_t t;
+
+            /* some bogus MTAs will quote the original "From " line */
+            if (!m_strncmp(">From ", line, 6))
+                continue;               /* just ignore */
+
+            if (is_from(line, return_path, sizeof(return_path), &t)) {
+                /* MH somtimes has the From_ line in the middle of the header! */
+                if (hdr && !hdr->received)
+                    hdr->received = t - mutt_local_tz(t);
+                continue;
+            }
 
-        /* spam tag is new, and match expr is non-empty; copy */
-        else if (!e->spam && *buf) {
-          e->spam = mutt_buffer_from (NULL, buf);
+            fseeko(f, loc, 0);
+            break;                    /* end of header */
         }
 
-        /* match expr is empty; plug in null string if no existing tag */
-        else if (!e->spam) {
-          e->spam = mutt_buffer_from (NULL, "");
+        if (mutt_match_spam_list(line, SpamList, buf, sizeof(buf))) {
+            if (!rx_list_match(NoSpamList, line)) {
+                /* if spam tag already exists, figure out how to amend it */
+                if (e->spam && *buf) {
+                    if (SpamSep) {
+                        /* If SpamSep defined, append with separator */
+                        mutt_buffer_addstr(e->spam, SpamSep);
+                        mutt_buffer_addstr(e->spam, buf);
+                    } else {
+                        /* else overwrite */
+                        mutt_buffer_reset(e->spam);
+                        mutt_buffer_addstr(e->spam, buf);
+                    }
+                }
+                else if (!e->spam && *buf) {
+                    /* spam tag is new, and match expr is non-empty; copy */
+                    e->spam = mutt_buffer_from(NULL, buf);
+                }
+                else if (!e->spam) {
+                    /* match expr is empty; plug in null string if no existing tag */
+                    e->spam = mutt_buffer_from(NULL, "");
+                }
+            }
         }
 
-        if (e->spam && e->spam->data)
-          debug_print (5, ("spam = %s\n", e->spam->data));
-      }
-    }
-
-    *p++ = 0;
-    p = vskipspaces(p);
-    if (!*p)
-      continue;                 /* skip empty header fields */
-
-    matched =
-      mutt_parse_rfc822_line (e, hdr, line, p, user_hdrs, weed, 1, &last);
+        *p++ = '\0';
+        p = vskipspaces(p);
+        if (!*p)
+            continue;                 /* skip empty header fields */
 
-  }
-
-  p_delete(&line);
-
-  if (hdr) {
-    hdr->content->hdr_offset = hdr->offset;
-    hdr->content->offset = ftello (f);
-    rfc2047_decode_envelope(e);
-    /* check for missing or invalid date */
-    if (hdr->date_sent <= 0) {
-      debug_print (1, ("no date found, using received "
-                       "time from msg separator\n"));
-      hdr->date_sent = hdr->received;
+        last = mutt_parse_rfc822_line(e, hdr, line, p, weed, 1, last);
     }
-  }
-
-  return (e);
-}
 
-address_t *mutt_parse_adrlist (address_t * p, const char *s)
-{
-  const char *q;
-
-  /* check for a simple whitespace separated list of addresses */
-  if ((q = strpbrk (s, "\"<>():;,\\")) == NULL) {
-    char tmp[HUGE_STRING];
-    char *r;
-
-    m_strcpy(tmp, sizeof(tmp), s);
-    r = tmp;
-    while ((r = strtok (r, " \t")) != NULL) {
-      p = rfc822_parse_adrlist (p, r);
-      r = NULL;
+    p_delete(&line);
+
+    if (hdr) {
+        hdr->content->hdr_offset = hdr->offset;
+        hdr->content->offset     = ftello(f);
+        rfc2047_decode_envelope(e);
+        /* check for missing or invalid date */
+        if (hdr->date_sent <= 0) {
+            debug_print(1, ("no date found, using received "
+                            "time from msg separator\n"));
+            hdr->date_sent = hdr->received;
+        }
     }
-  }
-  else
-    p = rfc822_parse_adrlist (p, s);
 
-  return p;
+    return e;
 }
 
+/* -------------------- XXX: MC READ MARK ------------- */
 
 /* Compares mime types to the ok and except lists */
-int count_body_parts_check(LIST **checklist, BODY *b, int dflt) {
-  LIST *type;
-  ATTACH_MATCH *a;
-
-  /* If list is null, use default behavior. */
-  if (! *checklist) {
-    /*return dflt;*/
-    return 0;
-  }
-
-  for (type = *checklist; type; type = type->next) {
-    a = (ATTACH_MATCH *)type->data;
-    debug_print(5, ("cbpc: %s %d/%s ?? %s/%s [%d]... ",
-               dflt ? "[OK] " : "[EXCL] ",
-               b->type, b->subtype, a->major, a->minor, a->major_int));
-    if ((a->major_int == TYPEANY || a->major_int == b->type) &&
-        !regexec(&a->minor_rx, b->subtype, 0, NULL, 0)) {
-      debug_print(5, ("yes\n"));
-      return 1;
-    } else {
-      debug_print(5, ("no\n"));
+static int count_body_parts_check(LIST **checklist, BODY *b, int dflt)
+{
+    LIST *type;
+    ATTACH_MATCH *a;
+
+    /* If list is null, use default behavior. */
+    if (!*checklist)
+        return 0;
+
+    for (type = *checklist; type; type = type->next) {
+        a = (ATTACH_MATCH *)type->data;
+        debug_print(5, ("cbpc: %s %d/%s ?? %s/%s [%d]... ",
+                        dflt ? "[OK] " : "[EXCL] ",
+                        b->type, b->subtype, a->major, a->minor, a->major_int));
+        if ((a->major_int == TYPEANY || a->major_int == b->type) &&
+            !regexec(&a->minor_rx, b->subtype, 0, NULL, 0)) {
+            debug_print(5, ("yes\n"));
+            return 1;
+        } else {
+            debug_print(5, ("no\n"));
+        }
     }
-  }
-  return 0;
+    return 0;
 }
 
 #define AT_COUNT(why) { shallcount = 1; }