Renamed -version switch to -Version to remove the conflict with -verbose.
[mmh] / uip / sortm.c
index 7d5a35c..d7dae61 100644 (file)
@@ -28,7 +28,7 @@ static struct swit switches[] = {
 #define NVERBSW  7
         { "noverbose", 0 },
 #define VERSIONSW  8
-        { "version", 0 },
+        { "Version", 0 },
 #define HELPSW  9
         { "help", 0 },
         { NULL, 0 }
@@ -43,8 +43,8 @@ struct smsg {
 static struct smsg *smsgs;
 int nmsgs;
 
-char *subjsort = (char *) 0;  /* sort on subject if != 0 */
-unsigned long datelimit = 0;
+char *subjsort = NULL;  /* sort on subject if != 0 */
+time_t datelimit = 0;
 int submajor = 0;  /* if true, sort on subject-major */
 int verbose;
 
@@ -54,17 +54,17 @@ typedef int (*qsort_comp) (const void *, const void *);
 /*
 ** static prototypes
 */
-static int read_hdrs (struct msgs *, char *);
-static int get_fields (char *, int, struct smsg *);
-static int dsort (struct smsg **, struct smsg **);
-static int subsort (struct smsg **, struct smsg **);
-static int txtsort (struct smsg **, struct smsg **);
-static void rename_chain (struct msgs *, struct smsg **, int, int);
-static void rename_msgs (struct msgs *, struct smsg **);
+static int read_hdrs(struct msgs *, char *);
+static int get_fields(char *, int, struct smsg *);
+static int dsort(struct smsg **, struct smsg **);
+static int subsort(struct smsg **, struct smsg **);
+static int txtsort(struct smsg **, struct smsg **);
+static void rename_chain(struct msgs *, struct smsg **, int, int);
+static void rename_msgs(struct msgs *, struct smsg **);
 
 
 int
-main (int argc, char **argv)
+main(int argc, char **argv)
 {
        int i, msgnum;
        unsigned char *cp;
@@ -78,12 +78,12 @@ main (int argc, char **argv)
 #ifdef LOCALE
        setlocale(LC_ALL, "");
 #endif
-       invo_name = r1bindex (argv[0], '/');
+       invo_name = mhbasename(argv[0]);
 
        /* read user profile/context */
        context_read();
 
-       arguments = getarguments (invo_name, argc, argv, 1);
+       arguments = getarguments(invo_name, argc, argv, 1);
        argp = arguments;
 
        /*
@@ -91,34 +91,34 @@ main (int argc, char **argv)
        */
        while ((cp = *argp++)) {
                if (*cp == '-') {
-                       switch (smatch (++cp, switches)) {
+                       switch (smatch(++cp, switches)) {
                        case AMBIGSW:
-                               ambigsw (cp, switches);
-                               done (1);
+                               ambigsw(cp, switches);
+                               done(1);
                        case UNKWNSW:
-                               adios (NULL, "-%s unknown", cp);
+                               adios(NULL, "-%s unknown", cp);
 
                        case HELPSW:
                                snprintf(buf, sizeof(buf), "%s [+folder] [msgs] [switches]", invo_name);
-                               print_help (buf, switches, 1);
-                               done (1);
+                               print_help(buf, switches, 1);
+                               done(1);
                        case VERSIONSW:
                                print_version(invo_name);
-                               done (1);
+                               done(1);
 
                        case DATESW:
                                if (datesw)
-                                       adios (NULL, "only one date field at a time");
+                                       adios(NULL, "only one date field at a time");
                                if (!(datesw = *argp++) || *datesw == '-')
-                                       adios (NULL, "missing argument to %s",
+                                       adios(NULL, "missing argument to %s",
                                                        argp[-2]);
                                continue;
 
                        case TEXTSW:
                                if (subjsort)
-                                       adios (NULL, "only one text field at a time");
+                                       adios(NULL, "only one text field at a time");
                                if (!(subjsort = *argp++) || *subjsort == '-')
-                                       adios (NULL, "missing argument to %s",
+                                       adios(NULL, "missing argument to %s",
                                                        argp[-2]);
                                continue;
 
@@ -126,12 +126,12 @@ main (int argc, char **argv)
                                subjsort = "subject";
                                continue;
                        case NSUBJSW:
-                               subjsort = (char *)0;
+                               subjsort = NULL;
                                continue;
 
                        case LIMSW:
                                if (!(cp = *argp++) || *cp == '-')
-                                               adios (NULL, "missing argument to %s", argp[-2]);
+                                               adios(NULL, "missing argument to %s", argp[-2]);
                                while (*cp == '0')
                                        cp++;  /* skip any leading zeros */
                                if (!*cp) {  /* hit end of string */
@@ -139,7 +139,7 @@ main (int argc, char **argv)
                                        continue;
                                }
                                if (!isdigit(*cp) || !(datelimit = atoi(cp)))
-                                       adios (NULL, "impossible limit %s", cp);
+                                       adios(NULL, "impossible limit %s", cp);
                                datelimit *= 60*60*24;
                                continue;
                        case NLIMSW:
@@ -157,62 +157,60 @@ main (int argc, char **argv)
                }
                if (*cp == '+' || *cp == '@') {
                        if (folder)
-                               adios (NULL, "only one folder at a time!");
+                               adios(NULL, "only one folder at a time!");
                        else
-                               folder = pluspath (cp);
+                               folder = getcpy(expandfol(cp));
                } else
                        app_msgarg(&msgs, cp);
        }
 
-       if (!context_find ("path"))
-               free (path ("./", TFOLDER));
        if (!msgs.size)
-               app_msgarg(&msgs, "all");
+               app_msgarg(&msgs, seq_all);
        if (!datesw)
                datesw = "date";
        if (!folder)
-               folder = getfolder (1);
-       maildir = m_maildir (folder);
+               folder = getcurfol();
+       maildir = toabsdir(folder);
 
-       if (chdir (maildir) == NOTOK)
-               adios (maildir, "unable to change directory to");
+       if (chdir(maildir) == NOTOK)
+               adios(maildir, "unable to change directory to");
 
        /* read folder and create message structure */
-       if (!(mp = folder_read (folder)))
-               adios (NULL, "unable to read folder %s", folder);
+       if (!(mp = folder_read(folder)))
+               adios(NULL, "unable to read folder %s", folder);
 
        /* check for empty folder */
        if (mp->nummsg == 0)
-               adios (NULL, "no messages in %s", folder);
+               adios(NULL, "no messages in %s", folder);
 
        /* parse all the message ranges/sequences and set SELECTED */
        for (msgnum = 0; msgnum < msgs.size; msgnum++)
-               if (!m_convert (mp, msgs.msgs[msgnum]))
-                       done (1);
-       seq_setprev (mp);  /* set the previous sequence */
+               if (!m_convert(mp, msgs.msgs[msgnum]))
+                       done(1);
+       seq_setprev(mp);  /* set the previous sequence */
 
-       if ((nmsgs = read_hdrs (mp, datesw)) <= 0)
-               adios (NULL, "no messages to sort");
+       if ((nmsgs = read_hdrs(mp, datesw)) <= 0)
+               adios(NULL, "no messages to sort");
 
        /*
        ** sort a list of pointers to our "messages to be sorted".
        */
-       dlist = (struct smsg **) mh_xmalloc ((nmsgs+1) * sizeof(*dlist));
+       dlist = (struct smsg **) mh_xmalloc((nmsgs+1) * sizeof(*dlist));
        for (i = 0; i < nmsgs; i++)
                dlist[i] = &smsgs[i];
        dlist[nmsgs] = 0;
 
        if (verbose) {  /* announce what we're doing */
                if (subjsort)
-                       printf ("sorting by %s-major %s-minor\n",
+                       printf("sorting by %s-major %s-minor\n",
                                submajor ? subjsort : datesw,
                                submajor ? datesw : subjsort);
                else
-                       printf ("sorting by datefield %s\n", datesw);
+                       printf("sorting by datefield %s\n", datesw);
        }
 
        /* first sort by date, or by subject-major, date-minor */
-       qsort ((char *) dlist, nmsgs, sizeof(*dlist),
+       qsort((char *) dlist, nmsgs, sizeof(*dlist),
                        (qsort_comp) (submajor && subjsort ? txtsort : dsort));
 
        /*
@@ -225,7 +223,7 @@ main (int argc, char **argv)
                register struct smsg ***il, **fp, **dp;
 
                slist = (struct smsg **)
-                               mh_xmalloc ((nmsgs+1) * sizeof(*slist));
+                               mh_xmalloc((nmsgs+1) * sizeof(*slist));
                memcpy((char *)slist, (char *)dlist, (nmsgs+1)*sizeof(*slist));
                qsort((char *)slist, nmsgs, sizeof(*slist),
                                (qsort_comp) subsort);
@@ -235,9 +233,9 @@ main (int argc, char **argv)
                ** the collection of messages with the same subj
                ** given a message number.
                */
-               il = (struct smsg ***) calloc (mp->hghsel+1, sizeof(*il));
+               il = (struct smsg ***) calloc(mp->hghsel+1, sizeof(*il));
                if (! il)
-                       adios (NULL, "couldn't allocate msg list");
+                       adios(NULL, "couldn't allocate msg list");
                for (i = 0; i < nmsgs; i++)
                        il[slist[i]->s_msg] = &slist[i];
                /*
@@ -245,7 +243,7 @@ main (int argc, char **argv)
                ** all the same subjects grouped together.
                */
                flist = (struct smsg **)
-                               mh_xmalloc ((nmsgs+1) * sizeof(*flist));
+                               mh_xmalloc((nmsgs+1) * sizeof(*flist));
                fp = flist;
                for (dp = dlist; *dp;) {
                        register struct smsg **s = il[(*dp++)->s_msg];
@@ -267,8 +265,8 @@ main (int argc, char **argv)
                        }
                }
                *fp = 0;
-               free (slist);
-               free (dlist);
+               free(slist);
+               free(dlist);
                dlist = flist;
        }
 
@@ -277,35 +275,34 @@ main (int argc, char **argv)
        ** structures, each of which contains a message number.
        */
 
-       rename_msgs (mp, dlist);
+       rename_msgs(mp, dlist);
 
-       context_replace (pfolder, folder);  /* update current folder */
-       seq_save (mp);  /* synchronize message sequences */
-       context_save ();  /* save the context file */
-       folder_free (mp);  /* free folder/message structure */
-       done (0);
+       context_replace(curfolder, folder);  /* update current folder */
+       seq_save(mp);  /* synchronize message sequences */
+       context_save();  /* save the context file */
+       folder_free(mp);  /* free folder/message structure */
+       done(0);
        return 1;
 }
 
 static int
-read_hdrs (struct msgs *mp, char *datesw)
+read_hdrs(struct msgs *mp, char *datesw)
 {
        int msgnum;
        struct tws tb;
        register struct smsg *s;
 
-       twscopy (&tb, dlocaltimenow ());
+       twscopy(&tb, dlocaltimenow());
 
-       smsgs = (struct smsg *)
-               calloc ((size_t) (mp->hghsel - mp->lowsel + 2),
+       smsgs = (struct smsg *) calloc((size_t) (mp->hghsel - mp->lowsel + 2),
                        sizeof(*smsgs));
        if (smsgs == NULL)
-               adios (NULL, "unable to allocate sort storage");
+               adios(NULL, "unable to allocate sort storage");
 
        s = smsgs;
        for (msgnum = mp->lowsel; msgnum <= mp->hghsel; msgnum++) {
                if (is_selected(mp, msgnum)) {
-                       if (get_fields (datesw, msgnum, s)) {
+                       if (get_fields(datesw, msgnum, s)) {
                                s->s_msg = msgnum;
                                s++;
                        }
@@ -322,7 +319,7 @@ read_hdrs (struct msgs *mp, char *datesw)
 */
 
 static int
-get_fields (char *datesw, int msg, struct smsg *smsg)
+get_fields(char *datesw, int msg, struct smsg *smsg)
 {
        register int state;
        int compnum;
@@ -331,38 +328,38 @@ get_fields (char *datesw, int msg, struct smsg *smsg)
        register char *datecomp = NULL, *subjcomp = NULL;
        register FILE *in;
 
-       if ((in = fopen (msgnam = m_name (msg), "r")) == NULL) {
-               admonish (msgnam, "unable to read message");
+       if ((in = fopen(msgnam = m_name(msg), "r")) == NULL) {
+               admonish(msgnam, "unable to read message");
                return (0);
        }
        for (compnum = 1, state = FLD;;) {
-               switch (state = m_getfld (state, nam, buf, sizeof(buf), in)) {
+               switch (state = m_getfld(state, nam, buf, sizeof(buf), in)) {
                case FLD:
                case FLDEOF:
                case FLDPLUS:
                        compnum++;
-                       if (!mh_strcasecmp (nam, datesw)) {
-                               datecomp = add (buf, datecomp);
+                       if (!mh_strcasecmp(nam, datesw)) {
+                               datecomp = add(buf, datecomp);
                                while (state == FLDPLUS) {
-                                       state = m_getfld (state, nam, buf,
+                                       state = m_getfld(state, nam, buf,
                                                        sizeof(buf), in);
-                                       datecomp = add (buf, datecomp);
+                                       datecomp = add(buf, datecomp);
                                }
                                if (!subjsort || subjcomp)
                                        break;
                        } else if (subjsort && !mh_strcasecmp(nam, subjsort)) {
-                               subjcomp = add (buf, subjcomp);
+                               subjcomp = add(buf, subjcomp);
                                while (state == FLDPLUS) {
-                                       state = m_getfld (state, nam, buf,
+                                       state = m_getfld(state, nam, buf,
                                                        sizeof(buf), in);
-                                       subjcomp = add (buf, subjcomp);
+                                       subjcomp = add(buf, subjcomp);
                                }
                                if (datecomp)
                                        break;
                        } else {
                                /* just flush this guy */
                                while (state == FLDPLUS)
-                                       state = m_getfld (state, nam, buf,
+                                       state = m_getfld(state, nam, buf,
                                                        sizeof(buf), in);
                        }
                        continue;
@@ -375,16 +372,16 @@ get_fields (char *datesw, int msg, struct smsg *smsg)
                case LENERR:
                case FMTERR:
                        if (state == LENERR || state == FMTERR)
-                               admonish (NULL, "format error in message %d (header #%d)", msg, compnum);
+                               admonish(NULL, "format error in message %d (header #%d)", msg, compnum);
                        if (datecomp)
-                               free (datecomp);
+                               free(datecomp);
                        if (subjcomp)
-                               free (subjcomp);
-                       fclose (in);
+                               free(subjcomp);
+                       fclose(in);
                        return (0);
 
                default:
-                       adios (NULL, "internal error -- you lose");
+                       adios(NULL, "internal error -- you lose");
                }
                break;
        }
@@ -393,15 +390,15 @@ get_fields (char *datesw, int msg, struct smsg *smsg)
        ** If no date component, then use the modification
        ** time of the file as its date
        */
-       if (!datecomp || (tw = dparsetime (datecomp)) == NULL) {
+       if (!datecomp || (tw = dparsetime(datecomp)) == NULL) {
                struct stat st;
 
-               admonish (NULL, "can't parse %s field in message %d",
+               admonish(NULL, "can't parse %s field in message %d",
                                datesw, msg);
-               fstat (fileno (in), &st);
+               fstat(fileno(in), &st);
                smsg->s_clock = st.st_mtime;
        } else {
-               smsg->s_clock = dmktime (tw);
+               smsg->s_clock = dmktime(tw);
        }
 
        if (subjsort) {
@@ -416,7 +413,7 @@ get_fields (char *datesw, int msg, struct smsg *smsg)
 
                        cp = subjcomp;
                        cp2 = subjcomp;
-                       if (strcmp (subjsort, "subject") == 0) {
+                       if (strcmp(subjsort, "subject") == 0) {
                                while ((c = *cp)) {
                                        if (! isspace(c)) {
                                                if(uprf(cp, "re:"))
@@ -439,9 +436,9 @@ get_fields (char *datesw, int msg, struct smsg *smsg)
 
                smsg->s_subj = subjcomp;
        }
-       fclose (in);
+       fclose(in);
        if (datecomp)
-               free (datecomp);
+               free(datecomp);
 
        return (1);
 }
@@ -450,7 +447,7 @@ get_fields (char *datesw, int msg, struct smsg *smsg)
 ** sort on dates.
 */
 static int
-dsort (struct smsg **a, struct smsg **b)
+dsort(struct smsg **a, struct smsg **b)
 {
        if ((*a)->s_clock < (*b)->s_clock)
                return (-1);
@@ -466,22 +463,22 @@ dsort (struct smsg **a, struct smsg **b)
 ** sort on subjects.
 */
 static int
-subsort (struct smsg **a, struct smsg **b)
+subsort(struct smsg **a, struct smsg **b)
 {
        register int i;
 
-       if ((i = strcmp ((*a)->s_subj, (*b)->s_subj)))
+       if ((i = strcmp((*a)->s_subj, (*b)->s_subj)))
                return (i);
 
-       return (dsort (a, b));
+       return (dsort(a, b));
 }
 
 static int
-txtsort (struct smsg **a, struct smsg **b)
+txtsort(struct smsg **a, struct smsg **b)
 {
        register int i;
 
-       if ((i = strcmp ((*a)->s_subj, (*b)->s_subj)))
+       if ((i = strcmp((*a)->s_subj, (*b)->s_subj)))
                return (i);
        else if ((*a)->s_msg < (*b)->s_msg)
                return (-1);
@@ -490,7 +487,7 @@ txtsort (struct smsg **a, struct smsg **b)
 }
 
 static void
-rename_chain (struct msgs *mp, struct smsg **mlist, int msg, int endmsg)
+rename_chain(struct msgs *mp, struct smsg **mlist, int msg, int endmsg)
 {
        int nxt, old, new;
        char *newname, oldname[BUFSIZ];
@@ -501,22 +498,22 @@ rename_chain (struct msgs *mp, struct smsg **mlist, int msg, int endmsg)
                mlist[msg] = (struct smsg *)0;
                old = smsgs[nxt].s_msg;
                new = smsgs[msg].s_msg;
-               strncpy (oldname, m_name (old), sizeof(oldname));
-               newname = m_name (new);
+               strncpy(oldname, m_name(old), sizeof(oldname));
+               newname = m_name(new);
                if (verbose)
-                       printf ("message %d becomes message %d\n", old, new);
+                       printf("message %d becomes message %d\n", old, new);
 
                snprintf(oldname, sizeof (oldname), "%s/%d",
                                mp->foldpath, old);
                snprintf(newbuf, sizeof (newbuf), "%s/%d", mp->foldpath, new);
                ext_hook("ref-hook", oldname, newbuf);
 
-               if (rename (oldname, newname) == NOTOK)
-                       adios (newname, "unable to rename %s to", oldname);
+               if (rename(oldname, newname) == NOTOK)
+                       adios(newname, "unable to rename %s to", oldname);
 
-               copy_msg_flags (mp, new, old);
+               copy_msg_flags(mp, new, old);
                if (mp->curmsg == old)
-                       seq_setcur (mp, new);
+                       seq_setcur(mp, new);
 
                if (nxt == endmsg)
                        break;
@@ -524,11 +521,11 @@ rename_chain (struct msgs *mp, struct smsg **mlist, int msg, int endmsg)
                msg = nxt;
        }
 /* if (nxt != endmsg); */
-/* rename_chain (mp, mlist, nxt, endmsg); */
+/* rename_chain(mp, mlist, nxt, endmsg); */
 }
 
 static void
-rename_msgs (struct msgs *mp, struct smsg **mlist)
+rename_msgs(struct msgs *mp, struct smsg **mlist)
 {
        int i, j, old, new;
        seqset_t tmpset;
@@ -536,7 +533,7 @@ rename_msgs (struct msgs *mp, struct smsg **mlist)
        char newbuf[MAXPATHLEN + 1];
        struct smsg *sp;
 
-       strncpy (tmpfil, m_name (mp->hghmsg + 1), sizeof(tmpfil));
+       strncpy(tmpfil, m_name(mp->hghmsg + 1), sizeof(tmpfil));
 
        for (i = 0; i < nmsgs; i++) {
                if (! (sp = mlist[i]))
@@ -553,10 +550,10 @@ rename_msgs (struct msgs *mp, struct smsg **mlist)
                */
                old = smsgs[j].s_msg;
                new = smsgs[i].s_msg;
-               strncpy (f1, m_name (old), sizeof(f1));
+               strncpy(f1, m_name(old), sizeof(f1));
 
                if (verbose)
-                       printf ("renaming message chain from %d to %d\n",
+                       printf("renaming message chain from %d to %d\n",
                                        old, new);
 
                /*
@@ -565,30 +562,30 @@ rename_msgs (struct msgs *mp, struct smsg **mlist)
                ** messages in the folder.
                */
 
-               (void)snprintf(f1, sizeof (f1), "%s/%d", mp->foldpath, old);
-               (void)snprintf(newbuf, sizeof (newbuf), "%s/%d",
+               snprintf(f1, sizeof (f1), "%s/%d", mp->foldpath, old);
+               snprintf(newbuf, sizeof (newbuf), "%s/%d",
                                mp->foldpath, mp->hghmsg + 1);
                ext_hook("ref-hook", f1, newbuf);
 
-               if (rename (f1, tmpfil) == NOTOK)
-                       adios (tmpfil, "unable to rename %s to ", f1);
+               if (rename(f1, tmpfil) == NOTOK)
+                       adios(tmpfil, "unable to rename %s to ", f1);
 
-               get_msg_flags (mp, &tmpset, old);
+               get_msg_flags(mp, &tmpset, old);
 
-               rename_chain (mp, mlist, j, i);
+               rename_chain(mp, mlist, j, i);
 
                /*
                ** Run the external hook to refile the temorary message number
                ** to the real place.
                */
 
-               (void)snprintf(f1, sizeof (f1), "%s/%d", mp->foldpath, new);
+               snprintf(f1, sizeof (f1), "%s/%d", mp->foldpath, new);
                ext_hook("ref-hook", newbuf, f1);
 
-               if (rename (tmpfil, m_name(new)) == NOTOK)
-                       adios (m_name(new), "unable to rename %s to", tmpfil);
+               if (rename(tmpfil, m_name(new)) == NOTOK)
+                       adios(m_name(new), "unable to rename %s to", tmpfil);
 
-               set_msg_flags (mp, &tmpset, new);
+               set_msg_flags(mp, &tmpset, new);
                mp->msgflags |= SEQMOD;
        }
 }