X-Git-Url: http://git.marmaro.de/?p=mmh;a=blobdiff_plain;f=uip%2Fmhlist.c;h=bcc4dd2b7b83f980893d8703b1dc4ec55821d848;hp=10ba1e4c35b8f7aefafca7e30482914443bdd1d9;hb=56a805299de35d8924969138aef4d0f1580daa6d;hpb=6c42153ad9362cc676ea66563bf400d7511b3b68 diff --git a/uip/mhlist.c b/uip/mhlist.c index 10ba1e4..bcc4dd2 100644 --- a/uip/mhlist.c +++ b/uip/mhlist.c @@ -20,18 +20,12 @@ #include #include #include +#include #ifdef HAVE_SYS_WAIT_H # include #endif -/* - * We allocate space for message names (msgs array) - * this number of elements at a time. - */ -#define MAXMSGS 256 - - static struct swit switches[] = { #define CHECKSW 0 { "check", 0 }, @@ -73,8 +67,6 @@ static struct swit switches[] = { }; -extern int errno; - /* mhparse.c */ extern int checksw; extern char *tmp; /* directory to place temp files */ @@ -125,19 +117,23 @@ void free_content (CT); * static prototypes */ static RETSIGTYPE pipeser (int); +static int freectp_done (int); int main (int argc, char **argv) { int sizesw = 1, headsw = 1; - int nummsgs, maxmsgs, msgnum, *icachesw; + int msgnum, *icachesw; char *cp, *file = NULL, *folder = NULL; char *maildir, buf[100], **argp; - char **arguments, **msgs; + char **arguments; + struct msgs_array msgs = { 0, 0, NULL }; struct msgs *mp = NULL; CT ct, *ctp; + done=freectp_done; + #ifdef LOCALE setlocale(LC_ALL, ""); #endif @@ -150,15 +146,6 @@ main (int argc, char **argv) argp = arguments; /* - * Allocate the initial space to record message - * names, ranges, and sequences. - */ - nummsgs = 0; - maxmsgs = MAXMSGS; - if (!(msgs = (char **) malloc ((size_t) (maxmsgs * sizeof(*msgs))))) - adios (NULL, "unable to allocate storage"); - - /* * Parse arguments */ while ((cp = *argp++)) { @@ -258,20 +245,9 @@ do_cache: if (folder) adios (NULL, "only one folder at a time!"); else - folder = path (cp + 1, *cp == '+' ? TFOLDER : TSUBCWF); - } else { - /* - * Check if we need to allocate more space - * for message names/ranges/sequences. - */ - if (nummsgs >= maxmsgs) { - maxmsgs += MAXMSGS; - if (!(msgs = (char **) realloc (msgs, - (size_t) (maxmsgs * sizeof(*msgs))))) - adios (NULL, "unable to reallocate msgs storage"); - } - msgs[nummsgs++] = cp; - } + folder = pluspath (cp); + } else + app_msgarg(&msgs, cp); } /* null terminate the list of acceptable parts/types */ @@ -302,7 +278,7 @@ do_cache: if (!context_find ("path")) free (path ("./", TFOLDER)); - if (file && nummsgs) + if (file && msgs.size) adios (NULL, "cannot specify msg and file at same time!"); /* @@ -319,8 +295,8 @@ do_cache: /* * message(s) are coming from a folder */ - if (!nummsgs) - msgs[nummsgs++] = "cur"; + if (!msgs.size) + app_msgarg(&msgs, "cur"); if (!folder) folder = getfolder (1); maildir = m_maildir (folder); @@ -337,8 +313,8 @@ do_cache: adios (NULL, "no messages in %s", folder); /* parse all the message ranges/sequences and set SELECTED */ - for (msgnum = 0; msgnum < nummsgs; msgnum++) - if (!m_convert (mp, msgs[msgnum])) + for (msgnum = 0; msgnum < msgs.size; msgnum++) + if (!m_convert (mp, msgs.msgs[msgnum])) done (1); seq_setprev (mp); /* set the previous-sequence */ @@ -418,8 +394,8 @@ pipeser (int i) } -int -done (int status) +static int +freectp_done (int status) { CT *ctp;