Replace some ints that are only ever 0 or 1 with bool.

[nmh] / uip / distsbr.c

diff --git a/uip/distsbr.c b/uip/distsbr.c
index 6fc773723d095e683d86fee45c730abe35aad459..64b8ae1952ddf8c71367f08853b5e9cca3295321 100644 (file)
--- a/uip/distsbr.c
+++ b/uip/distsbr.c
@@ -8,6 +8,7 @@
 #include <h/mh.h>
 #include <fcntl.h>
 #include <h/utils.h>
 #include <h/mh.h>
 #include <fcntl.h>
 #include <h/utils.h>

+#include "sbr/m_mktemp.h"

 
 static int  hdrfd = NOTOK;
 static int  txtfd = NOTOK;
 
 static int  hdrfd = NOTOK;
 static int  txtfd = NOTOK;


@@ -27,9 +28,9 @@ distout (char *drft, char *msgnam, char *backup)
 {
     int state;
     char *dp, *resent;
 {
     int state;
     char *dp, *resent;

-    char name[NAMESZ], buffer[BUFSIZ];
+    char name[NAMESZ], buffer[NMH_BUFSIZ];

     FILE *ifp, *ofp;
     FILE *ifp, *ofp;

-    m_getfld_state_t gstate = 0;
+    m_getfld_state_t gstate;

 
     if (rename (drft, strcpy (backup, m_backup (drft))) == NOTOK)
        adios (backup, "unable to rename %s to",drft);
 
     if (rename (drft, strcpy (backup, m_backup (drft))) == NOTOK)
        adios (backup, "unable to rename %s to",drft);


@@ -41,12 +42,13 @@ distout (char *drft, char *msgnam, char *backup)
     chmod (drft, m_gmprot ());
 
     ready_msg (msgnam);
     chmod (drft, m_gmprot ());
 
     ready_msg (msgnam);

-    lseek (hdrfd, (off_t) 0, SEEK_SET); /* msgnam not accurate */
+    lseek(hdrfd, 0, SEEK_SET); /* msgnam not accurate */

     cpydata (hdrfd, fileno (ofp), msgnam, drft);
 
     cpydata (hdrfd, fileno (ofp), msgnam, drft);
 

+    gstate = m_getfld_state_init(ifp);

     for (resent = NULL;;) {
        int buffersz = sizeof buffer;
     for (resent = NULL;;) {
        int buffersz = sizeof buffer;

-       switch (state = m_getfld (&gstate, name, buffer, &buffersz, ifp)) {
+       switch (state = m_getfld2(&gstate, name, buffer, &buffersz)) {

            case FLD: 
            case FLDPLUS: 
                if (uprf (name, "distribute-"))
            case FLD: 
            case FLDPLUS: 
                if (uprf (name, "distribute-"))


@@ -63,7 +65,7 @@ distout (char *drft, char *msgnam, char *backup)
                fprintf (ofp, "%s: %s", name, buffer);
                while (state == FLDPLUS) {
                    buffersz = sizeof buffer;
                fprintf (ofp, "%s: %s", name, buffer);
                while (state == FLDPLUS) {
                    buffersz = sizeof buffer;

-                   state = m_getfld (&gstate, name, buffer, &buffersz, ifp);
+                   state = m_getfld2(&gstate, name, buffer, &buffersz);

                    resent = add (buffer, resent);
                    fputs (buffer, ofp);
                }
                    resent = add (buffer, resent);
                    fputs (buffer, ofp);
                }


@@ -91,7 +93,7 @@ distout (char *drft, char *msgnam, char *backup)
                return NOTOK;
 
            default: 
                return NOTOK;
 
            default: 

-               adios (NULL, "getfld() returned %d", state);
+               die("getfld() returned %d", state);

        }
     }
 process: ;
        }
     }
 process: ;


@@ -110,7 +112,7 @@ process: ;
     free (resent);
 
     if (txtfd != NOTOK) {
     free (resent);
 
     if (txtfd != NOTOK) {

-       lseek (txtfd, (off_t) 0, SEEK_SET); /* msgnam not accurate */
+       lseek(txtfd, 0, SEEK_SET); /* msgnam not accurate */

        cpydata (txtfd, fileno (ofp), msgnam, drft);
     }
 
        cpydata (txtfd, fileno (ofp), msgnam, drft);
     }
 


@@ -124,32 +126,37 @@ static void
 ready_msg (char *msgnam)
 {
     int state, out;
 ready_msg (char *msgnam)
 {
     int state, out;

-    char name[NAMESZ], buffer[BUFSIZ], tmpfil[BUFSIZ];
+    char name[NAMESZ], buffer[NMH_BUFSIZ], tmpfil[BUFSIZ];

     FILE *ifp, *ofp;
     char *cp = NULL;
     FILE *ifp, *ofp;
     char *cp = NULL;

-    m_getfld_state_t gstate = 0;
+    m_getfld_state_t gstate;

 
 

-    if (hdrfd != NOTOK)
-       close (hdrfd), hdrfd = NOTOK;
-    if (txtfd != NOTOK)
-       close (txtfd), txtfd = NOTOK;
+    if (hdrfd != NOTOK) {
+        close (hdrfd);
+        hdrfd = NOTOK;
+    }
+    if (txtfd != NOTOK) {
+        close (txtfd);
+        txtfd = NOTOK;
+    }

 
     if ((ifp = fopen (msgnam, "r")) == NULL)
        adios (msgnam, "unable to open message");
 
     cp = m_mktemp2(NULL, "dist", &hdrfd, NULL);
     if (cp == NULL) {
 
     if ((ifp = fopen (msgnam, "r")) == NULL)
        adios (msgnam, "unable to open message");
 
     cp = m_mktemp2(NULL, "dist", &hdrfd, NULL);
     if (cp == NULL) {

-       adios(NULL, "unable to create temporary file in %s", get_temp_dir());
+       die("unable to create temporary file in %s", get_temp_dir());

     }
     strncpy(tmpfil, cp, sizeof(tmpfil));
     if ((out = dup (hdrfd)) == NOTOK
            || (ofp = fdopen (out, "w")) == NULL)
     }
     strncpy(tmpfil, cp, sizeof(tmpfil));
     if ((out = dup (hdrfd)) == NOTOK
            || (ofp = fdopen (out, "w")) == NULL)

-       adios (NULL, "no file descriptors -- you lose big");
+       die("no file descriptors -- you lose big");

     (void) m_unlink (tmpfil);
 
     (void) m_unlink (tmpfil);
 

+    gstate = m_getfld_state_init(ifp);

     for (;;) {
        int buffersz = sizeof buffer;
     for (;;) {
        int buffersz = sizeof buffer;

-       switch (state = m_getfld (&gstate, name, buffer, &buffersz, ifp)) {
+       switch (state = m_getfld2(&gstate, name, buffer, &buffersz)) {

            case FLD: 
            case FLDPLUS: 
                if (uprf (name, "resent"))
            case FLD: 
            case FLDPLUS: 
                if (uprf (name, "resent"))


@@ -157,7 +164,7 @@ ready_msg (char *msgnam)
                fprintf (ofp, "%s: %s", name, buffer);
                while (state == FLDPLUS) {
                    buffersz = sizeof buffer;
                fprintf (ofp, "%s: %s", name, buffer);
                while (state == FLDPLUS) {
                    buffersz = sizeof buffer;

-                   state = m_getfld (&gstate, name, buffer, &buffersz, ifp);
+                   state = m_getfld2(&gstate, name, buffer, &buffersz);

                    fputs (buffer, ofp);
                }
                break;
                    fputs (buffer, ofp);
                }
                break;


@@ -167,19 +174,19 @@ ready_msg (char *msgnam)
 
                 cp = m_mktemp2(NULL, "dist", &txtfd, NULL);
                 if (cp == NULL) {
 
                 cp = m_mktemp2(NULL, "dist", &txtfd, NULL);
                 if (cp == NULL) {

-                   adios(NULL, "unable to create temporary file in %s",
+                   die("unable to create temporary file in %s",

                          get_temp_dir());
                 }
                 fchmod(txtfd, 0600);
                strncpy (tmpfil, cp, sizeof(tmpfil));
                if ((out = dup (txtfd)) == NOTOK
                        || (ofp = fdopen (out, "w")) == NULL)
                          get_temp_dir());
                 }
                 fchmod(txtfd, 0600);
                strncpy (tmpfil, cp, sizeof(tmpfil));
                if ((out = dup (txtfd)) == NOTOK
                        || (ofp = fdopen (out, "w")) == NULL)

-                   adios (NULL, "no file descriptors -- you lose big");
+                   die("no file descriptors -- you lose big");

                (void) m_unlink (tmpfil);
                fprintf (ofp, "\n%s", buffer);
                while (state == BODY) {
                    buffersz = sizeof buffer;
                (void) m_unlink (tmpfil);
                fprintf (ofp, "\n%s", buffer);
                while (state == BODY) {
                    buffersz = sizeof buffer;

-                   state = m_getfld (&gstate, name, buffer, &buffersz, ifp);
+                   state = m_getfld2(&gstate, name, buffer, &buffersz);

                    fputs (buffer, ofp);
                }
            case FILEEOF: 
                    fputs (buffer, ofp);
                }
            case FILEEOF: 


@@ -187,10 +194,10 @@ ready_msg (char *msgnam)
 
            case LENERR: 
            case FMTERR: 
 
            case LENERR: 
            case FMTERR: 

-               adios (NULL, "format error in message %s", msgnam);
+               die("format error in message %s", msgnam);

 
            default: 
 
            default: 

-               adios (NULL, "getfld() returned %d", state);
+               die("getfld() returned %d", state);

        }
     }
 process: ;
        }
     }
 process: ;