Renamed -version switch to -Version to remove the conflict with -verbose.
[mmh] / uip / mhl.c
index 4c2f06e..328db8f 100644 (file)
--- a/uip/mhl.c
+++ b/uip/mhl.c
 #define QUOTE  '\\'
 
 static struct swit mhlswitches[] = {
-#define FOLDSW  0
-       { "folder +folder", 0 },
-#define FORMSW  1
+#define FORMSW  0
        { "form formfile", 0 },
-#define PROGSW  2
-       { "moreproc program", 0 },
-#define WIDTHSW  3
+#define WIDTHSW  1
        { "width columns", 0 },
-#define SLEEPSW  4
+#define SLEEPSW  2
        { "sleep seconds",  0 },
-#define VERSIONSW  5
-       { "version", 0 },
-#define HELPSW  6
+#define VERSIONSW  3
+       { "Version", 0 },
+#define HELPSW  4
        { "help", 0 },
-#define FORW1SW  7
+#define FORW1SW  5
        { "forward", -7 },
-#define FORW2SW  8
+#define FORW2SW  6
        { "forwall", -7 },
-#define DGSTSW  9
+#define DGSTSW  7
        { "digest list", -6 },
-#define VOLUMSW  10
+#define VOLUMSW  8
        { "volume number", -6 },
-#define ISSUESW  11
+#define ISSUESW  9
        { "issue number", -5 },
-#define NBODYSW  12
+#define NBODYSW  10
        { "nobody", -6 },
        { NULL, 0 }
 };
@@ -91,7 +87,7 @@ struct mcomp {
        char *c_name;  /* component name */
        char *c_text;  /* component text */
        char *c_ovtxt; /* text overflow indicator */
-       char *c_nfs;   /* iff FORMAT */
+       char *c_fstr;   /* iff FORMAT */
        struct format *c_fmt;  /*  .. */
        int c_offset;  /* left margin indentation */
        int c_ovoff;   /* overflow indentation */
@@ -107,11 +103,11 @@ static struct mcomp *fmthd = NULL;
 static struct mcomp *fmttl = NULL;
 
 static struct mcomp global = {
-       NULL, NULL, "", NULL, NULL, 0, -1, 80, -1, 0, 0
+       NULL, NULL, NULL, NULL, NULL, 0, -1, 80, -1, 0, NULL
 };
 
 static struct mcomp holder = {
-       NULL, NULL, NULL, NULL, NULL, 0, 0, 0, 0, NOCOMPONENT, 0
+       NULL, NULL, NULL, NULL, NULL, 0, 0, 0, 0, NOCOMPONENT, NULL
 };
 
 struct pair {
@@ -176,16 +172,12 @@ static int issue = 0;
 static int exitstat = 0;
 static int mhldebug = 0;
 
-#define PITTY  (-1)
-#define NOTTY  0
-static int ontty = NOTTY;
-
-static int column;
+static unsigned int column;
 
 static int lm;
 static int ovoff;
 static int term;
-static int wid;
+static unsigned int wid;
 
 static char *ovtxt;
 
@@ -221,7 +213,7 @@ static int evalvar(struct mcomp *);
 static int ptoi(char *, int *);
 static int ptos(char *, char **);
 static char *parse(void);
-static void process(char *, char *, int, int);
+static void process(char *, int, int);
 static void mhlfile(FILE *, char *, int, int);
 static int mcomp_flags(char *);
 static char *mcomp_add(long, char *, char *);
@@ -233,14 +225,9 @@ static void putcomp(struct mcomp *, struct mcomp *, int);
 static char *oneline(char *, long);
 static void putstr(char *);
 static void putch(char);
-static RETSIGTYPE intrser(int);
-static RETSIGTYPE pipeser(int);
-static RETSIGTYPE quitser(int);
+static void intrser(int);
 static void mhladios(char *, char *, ...);
 static void mhldone(int);
-static void m_popen(char *);
-
-void m_pclose(void);
 
 int sc_width(void);  /* from termsbr.c */
 
@@ -249,10 +236,9 @@ int
 main(int argc, char **argv)
 {
        int i, width = 0, vecp = 0;
-       char *cp, *folder = NULL, *form = NULL;
+       char *cp, *form = NULL;
        char buf[BUFSIZ], *files[MAXARGS];
        char **argp, **arguments;
-       char *progsw = NULL;
 
 #ifdef LOCALE
        setlocale(LC_ALL, "");
@@ -286,11 +272,6 @@ main(int argc, char **argv)
                                print_version(invo_name);
                                done(1);
 
-                       case FOLDSW:
-                               if (!(folder = *argp++) || *folder == '-')
-                                       adios(NULL, "missing argument to %s",
-                                                       argp[-2]);
-                               continue;
                        case FORMSW:
                                if (!(form = *argp++) || *form == '-')
                                        adios(NULL, "missing argument to %s",
@@ -304,12 +285,6 @@ main(int argc, char **argv)
                                sleepsw = atoi(cp);  /* ZERO ok! */
                                continue;
 
-                       case PROGSW:
-                               if (!(progsw = *argp++) || *progsw == '-')
-                                       adios(NULL, "missing argument to %s",
-                                                       argp[-2]);
-                               continue;
-
                        case WIDTHSW:
                                if (!(cp = *argp++) || *cp == '-')
                                        adios(NULL, "missing argument to %s",
@@ -356,28 +331,13 @@ main(int argc, char **argv)
                files[vecp++] = cp;
        }
 
-       if (!folder)
-               folder = getenv("mhfolder");
-
-       if (isatty(fileno(stdout))) {
-               if (mhl_action) {
-                       SIGNAL(SIGINT, SIG_IGN);
-                       SIGNAL2(SIGQUIT, quitser);
-               }
-               SIGNAL2(SIGPIPE, pipeser);
-               m_popen(progsw ? progsw : moreproc);
-               ontty = PITTY;
-       } else {
-               ontty = NOTTY;
-       }
-
        mhl_format(form ? form : mhlformat, width);
 
        if (vecp == 0) {
-               process(folder, NULL, 1, vecp = 1);
+               process(NULL, 1, vecp = 1);
        } else {
                for (i = 0; i < vecp; i++)
-                       process(folder, files[i], i + 1, vecp);
+                       process(files[i], i + 1, vecp);
        }
 
        if (forwall) {
@@ -405,9 +365,6 @@ main(int argc, char **argv)
                adios("output", "error writing");
        }
 
-       if (ontty == PITTY)
-               m_pclose();
-
        return exitstat;
 }
 
@@ -416,7 +373,7 @@ static void
 mhl_format(char *file, int width)
 {
        int i;
-       char *bp, *cp, **ip;
+       char *bp, *cp;
        char *ap, buffer[BUFSIZ], name[NAMESZ];
        struct mcomp *c1;
        struct stat st;
@@ -444,7 +401,7 @@ mhl_format(char *file, int width)
                ino = st.st_ino;
        }
 
-       global.c_ovtxt = global.c_nfs = NULL;
+       global.c_ovtxt = global.c_fstr = NULL;
        global.c_fmt = NULL;
        global.c_offset = 0;
        global.c_ovoff = -1;
@@ -452,7 +409,7 @@ mhl_format(char *file, int width)
                global.c_width = i;
        global.c_cwidth = -1;
        global.c_flags = 0;
-       *(ip = ignores) = NULL;
+       *ignores = NULL;
 
        while (vfgets(fp, &ap) == OK) {
                bp = ap;
@@ -479,25 +436,20 @@ mhl_format(char *file, int width)
                        ** it to the end of the current "ignores" list.
                        */
                        if (!mh_strcasecmp(name, "ignores")) {
-                               char **tmparray, **p;
+                               char **tmparray;
                                int n = 0;
 
                                /* split the fields */
                                tmparray = brkstring(getcpy(++parptr), ",",
                                                NULL);
-
-                               /* count number of fields split */
-                               p = tmparray;
-                               while (*p++)
-                                       n++;
-
                                /*
                                ** copy pointers to split fields
                                ** to ignores array
                                */
-                               ip = copyip(tmparray, ip,
-                                               MAXARGS - num_ignores);
-                               num_ignores += n;
+                               while (tmparray[n] && num_ignores<MAXARGS-1) {
+                                       ignores[num_ignores++] = tmparray[n++];
+                               }
+                               ignores[num_ignores] = NULL;
                                continue;
                        }
                        parptr = bp;
@@ -516,13 +468,13 @@ mhl_format(char *file, int width)
                                if (evalvar(c1))
                                        adios(NULL, "format file syntax error: %s", bp);
                        }
-                       if (!c1->c_nfs && global.c_nfs) {
+                       if (!c1->c_fstr && global.c_fstr) {
                                if ((c1->c_flags & DATEFMT) &&
                                                (global.c_flags & DATEFMT)) {
-                                       c1->c_nfs = getcpy(global.c_nfs);
+                                       c1->c_fstr = getcpy(global.c_fstr);
                                } else if ((c1->c_flags & ADDRFMT) &&
                                                (global.c_flags & ADDRFMT)) {
-                                       c1->c_nfs = getcpy(global.c_nfs);
+                                       c1->c_fstr = getcpy(global.c_fstr);
                                }
                        }
                        continue;
@@ -536,7 +488,7 @@ mhl_format(char *file, int width)
        if (mhldebug) {
                for (c1 = fmthd; c1; c1 = c1->c_next) {
                        fprintf(stderr, "c1: name=\"%s\" text=\"%s\" ovtxt=\"%s\"\n", c1->c_name, c1->c_text, c1->c_ovtxt);
-                       fprintf(stderr, "\tnfs=0x%x fmt=0x%x\n", (unsigned int)(unsigned long) c1->c_nfs, (unsigned int)(unsigned long) c1->c_fmt);
+                       fprintf(stderr, "\tfstr=0x%x fmt=0x%x\n", (unsigned int)(unsigned long) c1->c_fstr, (unsigned int)(unsigned long) c1->c_fmt);
                        fprintf(stderr, "\toffset=%d ovoff=%d width=%d cwidth=%d\n", c1->c_offset, c1->c_ovoff, c1->c_width, c1->c_cwidth);
                        fprintf (stderr, "\tflags=%s\n", snprintb(buffer, sizeof(buffer), (unsigned) c1->c_flags, LBITS));
                }
@@ -571,23 +523,23 @@ evalvar(struct mcomp *c1)
                return ptos(name, &c1->c_ovtxt);
 
        if (!mh_strcasecmp(name, "formatfield")) {
-               char *nfs;
+               char *fmtstr;
 
                if (ptos(name, &cp))
                        return 1;
                cp = concat("=", cp, NULL);
-               nfs = new_fs(cp, NULL);
+               fmtstr = new_fs(cp, NULL);
                free(cp);
-               c1->c_nfs = getcpy(nfs);
+               c1->c_fstr = getcpy(fmtstr);
                c1->c_flags |= FORMAT;
                return 0;
        }
 
        if (!mh_strcasecmp(name, "decode")) {
-               char *nfs;
+               char *fmtstr;
 
-               nfs = new_fs("=%(decode{text})", NULL);
-               c1->c_nfs = getcpy(nfs);
+               fmtstr = new_fs("=%(decode{text})", NULL);
+               c1->c_fstr = getcpy(fmtstr);
                c1->c_flags |= FORMAT;
                return 0;
        }
@@ -682,9 +634,8 @@ parse(void)
 
 
 static void
-process(char *folder, char *fname, int ofilen, int ofilec)
+process(char *fname, int ofilen, int ofilec)
 {
-       char *cp = NULL;
        FILE *fp = NULL;
        struct mcomp *c1;
 
@@ -702,17 +653,13 @@ process(char *folder, char *fname, int ofilen, int ofilec)
                        fname = "(stdin)";
                        fp = stdin;
                }
-               cp = folder ? concat(folder, ":", fname, NULL) : getcpy(fname);
-               if (ontty != PITTY)
-                       SIGNAL(SIGINT, intrser);
-               mhlfile(fp, cp, ofilen, ofilec);
+               SIGNAL(SIGINT, intrser);
+               mhlfile(fp, fname, ofilen, ofilec);
                /* FALL THROUGH! */
        default:
-               if (ontty != PITTY)
-                       SIGNAL(SIGINT, SIG_IGN);
+               SIGNAL(SIGINT, SIG_IGN);
                if (mhl_action == NULL && fp != stdin)
                        fclose(fp);
-               free(cp);
                if (holder.c_text) {
                        free(holder.c_text);
                        holder.c_text = NULL;
@@ -895,7 +842,7 @@ mcomp_format(struct mcomp *c1, struct mcomp *c2)
        dat[2] = 0;
        dat[3] = sizeof(buffer) - 1;
        dat[4] = 0;
-       fmt_compile(c1->c_nfs, &c1->c_fmt);
+       fmt_compile(c1->c_fstr, &c1->c_fmt);
 
        if (!(c1->c_flags & ADDRFMT)) {
                FINDCOMP(cptr, "text");
@@ -1003,8 +950,8 @@ free_queue(struct mcomp **head, struct mcomp **tail)
                        free(c1->c_text);
                if (c1->c_ovtxt)
                        free(c1->c_ovtxt);
-               if (c1->c_nfs)
-                       free(c1->c_nfs);
+               if (c1->c_fstr)
+                       free(c1->c_fstr);
                if (c1->c_fmt)
                        free((char *) c1->c_fmt);
                free((char *) c1);
@@ -1037,7 +984,7 @@ putcomp(struct mcomp *c1, struct mcomp *c2, int flag)
                return;
        }
 
-       if (c1->c_nfs && (c1->c_flags & (ADDRFMT | DATEFMT | FORMAT)))
+       if (c1->c_fstr && (c1->c_flags & (ADDRFMT | DATEFMT | FORMAT)))
                mcomp_format(c1, c2);
 
        if (c1->c_flags & CENTER) {
@@ -1096,11 +1043,11 @@ putcomp(struct mcomp *c1, struct mcomp *c2, int flag)
        count = 0;
        if (cchdr) {
                if (flag == TWOCOMP)
-                       count = (c1->c_cwidth >= 0) ?
-                                       c1->c_cwidth : strlen(c2->c_name) + 2;
+                       count = (c1->c_cwidth >= 0) ? c1->c_cwidth :
+                                       (int)strlen(c2->c_name) + 2;
                else
-                       count = (c1->c_cwidth >= 0) ?
-                                       c1->c_cwidth : strlen(c1->c_text ?
+                       count = (c1->c_cwidth >= 0) ? (size_t)c1->c_cwidth :
+                                       strlen(c1->c_text ?
                                        c1->c_text : c1->c_name) + 2;
        }
        count += c1->c_offset;
@@ -1237,43 +1184,15 @@ putch(char ch)
 }
 
 
-static RETSIGTYPE
+static void
 intrser(int i)
 {
-#ifndef RELIABLE_SIGNALS
-       SIGNAL(SIGINT, intrser);
-#endif
-
        discard(stdout);
        putchar('\n');
        longjmp(env, DONE);
 }
 
 
-static RETSIGTYPE
-pipeser(int i)
-{
-#ifndef RELIABLE_SIGNALS
-       SIGNAL(SIGPIPE, pipeser);
-#endif
-
-       done(NOTOK);
-}
-
-
-static RETSIGTYPE
-quitser(int i)
-{
-#ifndef RELIABLE_SIGNALS
-       SIGNAL(SIGQUIT, quitser);
-#endif
-
-       putchar('\n');
-       fflush(stdout);
-       done(NOTOK);
-}
-
-
 #undef adios
 #undef done
 
@@ -1298,68 +1217,3 @@ mhldone(int status)
        else
                done(exitstat);
 }
-
-
-static int m_pid = NOTOK;
-static int sd = NOTOK;
-
-static void
-m_popen(char *name)
-{
-       int pd[2];
-
-       if (mhl_action && (sd = dup(fileno(stdout))) == NOTOK)
-               adios("standard output", "unable to dup()");
-
-       if (pipe(pd) == NOTOK)
-               adios("pipe", "unable to");
-
-       switch (m_pid = fork()) {
-       case NOTOK:
-               adios("fork", "unable to");
-
-       case OK:
-               SIGNAL(SIGINT, SIG_DFL);
-               SIGNAL(SIGQUIT, SIG_DFL);
-
-               close(pd[1]);
-               if (pd[0] != fileno(stdin)) {
-                       dup2(pd[0], fileno(stdin));
-                       close(pd[0]);
-               }
-               execlp(name, mhbasename(name), NULL);
-               fprintf(stderr, "unable to exec ");
-               perror(name);
-               _exit(-1);
-
-       default:
-               close(pd[0]);
-               if (pd[1] != fileno(stdout)) {
-                       dup2(pd[1], fileno(stdout));
-                       close(pd[1]);
-               }
-       }
-}
-
-
-void
-m_pclose(void)
-{
-       if (m_pid == NOTOK)
-               return;
-
-       if (sd != NOTOK) {
-               fflush(stdout);
-               if (dup2(sd, fileno(stdout)) == NOTOK)
-                       adios("standard output", "unable to dup2()");
-
-               clearerr(stdout);
-               close(sd);
-               sd = NOTOK;
-       }
-       else
-               fclose(stdout);
-
-       pidwait(m_pid, OK);
-       m_pid = NOTOK;
-}