scansbr: Further refactoring.
[mmh] / uip / scansbr.c
index 4b71243..9f4143e 100644 (file)
@@ -41,25 +41,22 @@ static struct format *fmt;
 static struct comp *datecomp;  /* pntr to "date" comp */
 static struct comp *bodycomp;  /* pntr to "body" pseudo-comp (if referenced) */
 static int ncomps = 0;  /* # of interesting components */
-static char **compbuffers = 0;  /* buffers for component text */
-static struct comp **used_buf = 0;  /* stack for comp that use buffers */
+static char **compbuffers = NULL;  /* buffers for component text */
+static struct comp **used_buf = NULL;  /* stack for comp that use buffers */
 
 static int dat[5];  /* aux. data for format routine */
 
-char *scanl = 0;  /* text of most recent scanline */
-
-#define DIEWRERR() adios(scnmsg, "write error on")
+char *scanl = NULL;  /* text of most recent scanline */
 
 #define FPUTS(buf) {\
-               if (mh_fputs(buf,scnout) == EOF)\
-                       DIEWRERR();\
+               if (fputs(buf, scnout) == EOF)\
+                       adios(scnmsg, "write error on");\
        }
 
 /*
 ** prototypes
 */
 int sc_width(void);  /* from termsbr.c */
-static int mh_fputs(char *, FILE *);
 
 #ifdef MULTIBYTE_SUPPORT
 #define SCAN_CHARWIDTH MB_CUR_MAX
@@ -68,8 +65,8 @@ static int mh_fputs(char *, FILE *);
 #endif
 
 int
-scan(FILE *inb, int innum, int outnum, char *nfs, int width, int curflg,
-       int unseen, char *folder, long size, int noisy)
+scan(FILE *inb, int innum, int outnum, char *fmtstr, int width, int curflg,
+       int unseen)
 {
        int i, compnum, state;
        unsigned char *cp, *tmpbuf;
@@ -81,11 +78,10 @@ scan(FILE *inb, int innum, int outnum, char *nfs, int width, int curflg,
        FILE *scnout = NULL;
        char name[NAMESZ];
        static int rlwidth, slwidth;
-
-#ifdef RPATHS
        char returnpath[BUFSIZ];
        char deliverydate[BUFSIZ];
-#endif
+       int incing = (outnum > 0);
+       int scanfolder = (outnum == 0);
 
        /* first-time only initialization */
        if (!scanl) {
@@ -98,20 +94,14 @@ scan(FILE *inb, int innum, int outnum, char *nfs, int width, int curflg,
                dat[3] = slwidth = width;
                scanl = (char *) mh_xmalloc((size_t) SCAN_CHARWIDTH *
                                (slwidth + 2) );
-               if (outnum)
+               if (incing)
                        umask(~m_gmprot());
 
                /* Compile format string */
-               ncomps = fmt_compile(nfs, &fmt) + 1;
+               ncomps = fmt_compile(fmtstr, &fmt) + 1;
 
                FINDCOMP(bodycomp, "body");
                FINDCOMP(datecomp, "date");
-               FINDCOMP(cptr, "folder");
-               if (cptr && folder)
-                       cptr->c_text = folder;
-               FINDCOMP(cptr, "dtimenow");
-               if (cptr)
-                       cptr->c_text = getcpy(dtimenow(0));
                nxtbuf = compbuffers = (char **) calloc((size_t) ncomps,
                                sizeof(char *));
                if (nxtbuf == NULL)
@@ -149,17 +139,12 @@ scan(FILE *inb, int innum, int outnum, char *nfs, int width, int curflg,
                }
        }
 
-       if (outnum) {
-               if (outnum > 0) {
-                       scnmsg = m_name(outnum);
-                       if (*scnmsg == '?')  /* msg num out of range */
-                               return SCNNUM;
-               } else {
-                       scnmsg = "/dev/null";
-               }
+       if (incing) {
+               scnmsg = m_name(outnum);
+               if (*scnmsg == '?')  /* msg num out of range */
+                       return SCNNUM;
                if ((scnout = fopen(scnmsg, "w")) == NULL)
                        adios(scnmsg, "unable to write");
-#ifdef RPATHS
                /*
                ** Add the Return-Path and Delivery-Date
                ** header fields to message.
@@ -171,7 +156,6 @@ scan(FILE *inb, int innum, int outnum, char *nfs, int width, int curflg,
                        FPUTS("Delivery-Date: ");
                        FPUTS(deliverydate);
                }
-#endif /* RPATHS */
        }
 
        /* scan - main loop */
@@ -181,10 +165,9 @@ scan(FILE *inb, int innum, int outnum, char *nfs, int width, int curflg,
                case FLD:
                case FLDPLUS:
                        compnum++;
-                       if (outnum) {
+                       if (incing) {
                                FPUTS(name);
-                               if (putc(':', scnout) == EOF)
-                                       DIEWRERR();
+                               FPUTS(":");
                                FPUTS(tmpbuf);
                        }
                        /*
@@ -215,18 +198,18 @@ scan(FILE *inb, int innum, int outnum, char *nfs, int width, int curflg,
                        while (state == FLDPLUS) {
                                state = m_getfld(state, name, tmpbuf, rlwidth,
                                                inb);
-                               if (outnum)
+                               if (incing)
                                        FPUTS(tmpbuf);
                        }
                        break;
 
                case BODY:
                        compnum = -1;
-                       if (! outnum) {
-                               state = FILEEOF; /* stop now if scan cmd */
+                       if (!incing) {
+                               state = FILEEOF; /* stop here if scan cmd */
                                goto finished;
                        }
-                       if (putc('\n', scnout) == EOF) DIEWRERR();
+                       FPUTS("\n");
                        FPUTS(tmpbuf);
                        /*
                        ** performance hack: some people like to run "inc"
@@ -249,7 +232,7 @@ body:;
                                if (scnout->_cnt <= 0) {
 #endif
                                        if (fflush(scnout) == EOF)
-                                               DIEWRERR();
+                                               adios(scnmsg, "write error on");
                                }
 #ifdef LINUX_STDIO
                                state = m_getfld(state, name, scnout->_IO_write_ptr, (long)scnout->_IO_write_ptr-(long)scnout->_IO_write_end , inb);
@@ -259,7 +242,7 @@ body:;
                                ((struct __FILE_public *)scnout)->_w -= msg_count;
                                ((struct __FILE_public *)scnout)->_p += msg_count;
 #else
-                               state = m_getfld( state, name, scnout->_ptr, -(scnout->_cnt), inb);
+                               state = m_getfld(state, name, scnout->_ptr, -(scnout->_cnt), inb);
                                scnout->_cnt -= msg_count;
                                scnout->_ptr += msg_count;
 #endif
@@ -271,15 +254,14 @@ body:;
                        fprintf(stderr, innum ? "??Format error (message %d) in " : "??Format error in ", outnum ? outnum : innum);
                        fprintf(stderr, "component %d\n", compnum);
 
-                       if (outnum) {
+                       if (incing) {
                                FPUTS("\n\nBAD MSG:\n");
                                FPUTS(name);
-                               if (putc('\n', scnout) == EOF)
-                                       DIEWRERR();
+                               FPUTS("\n");
                                state = BODY;
                                goto body;
                        }
-                       /* fall through */
+                       /* fall through if we scan only */
 
                case FILEEOF:
                        goto finished;
@@ -289,12 +271,11 @@ body:;
                }
        }
 
-       /*
-       ** format and output the scan line.
-       */
 finished:
+
+       /* Format and output the scan line. */
        if (ferror(inb)) {
-               advise("read", "unable to"); /* "read error" */
+               advise("read", "unable to");
                return SCNFAT;
        }
 
@@ -304,22 +285,19 @@ finished:
                bodycomp->c_text = tmpbuf;
        }
 
-       if (size)
-               dat[2] = size;
-       else if (outnum > 0) {
-               dat[2] = ftell(scnout);
-               if (dat[2] == EOF)
-                       DIEWRERR();
+       if (incing) {
+               if ((dat[2] = ftell(scnout)) == EOF)
+                       adios(scnmsg, "write error on");
        }
 
-       if ((datecomp && !datecomp->c_text) || (!size && !outnum)) {
+       if ((datecomp && !datecomp->c_text) || scanfolder) {
                struct stat st;
 
                fstat(fileno(inb), &st);
-               if (!size && !outnum)
+               if (scanfolder)
                        dat[2] = st.st_size;
                if (datecomp) {
-                       if (! datecomp->c_text) {
+                       if (!datecomp->c_text) {
                                if (datecomp->c_tws == NULL)
                                        datecomp->c_tws = (struct tws *) calloc((size_t) 1, sizeof(*datecomp->c_tws));
                                if (datecomp->c_tws == NULL)
@@ -332,14 +310,14 @@ finished:
                }
        }
 
+       /* print the scan line */
        fmt_scan(fmt, scanl, slwidth, dat);
+       if (fmtstr)
+               fputs(scanl, stdout);
 
        if (bodycomp)
                bodycomp->c_text = saved_c_text;
 
-       if (noisy)
-               fputs(scanl, stdout);
-
        /* return dynamically allocated buffers to pool */
        while ((cptr = *savecomp++)) {
                *--nxtbuf = cptr->c_text;
@@ -347,19 +325,8 @@ finished:
        }
        *--nxtbuf = tmpbuf;
 
-       if (outnum && (ferror(scnout) || fclose(scnout) == EOF))
-               DIEWRERR();
+       if (incing && (ferror(scnout) || fclose(scnout) == EOF))
+               adios(scnmsg, "write error on");
 
        return (state != FILEEOF ? SCNERR : SCNMSG);
 }
-
-static int
-mh_fputs(char *s, FILE *stream)
-{
-       char c;
-
-       while ((c = *s++))
-               if (putc(c,stream) == EOF )
-                       return(EOF);
-       return (0);
-}