]> diplodocus.org Git - nmh/blobdiff - uip/post.c
uip: Replace some ints that are only ever 0 or 1 with bool.
[nmh] / uip / post.c
index 56ae555c23ccd29e06fe24d7d5a000ae7bcb1a7b..2917945b9b61c2df81514db293739892134c7673 100644 (file)
@@ -12,6 +12,7 @@
 #include <h/aliasbr.h>
 #include <h/dropsbr.h>
 #include <h/mime.h>
+#include "h/done.h"
 #include <h/utils.h>
 #include <h/tws.h>
 #include <h/mts.h>
@@ -229,7 +230,7 @@ static int unkadr = 0;              /* how many of those were unknown        */
 static int badadr = 0;         /* number of bad addrs                   */
 static int badmsg = 0;         /* message has bad semantics             */
 static int verbose = 0;                /* spell it out                          */
-static int format = 1;         /* format addresses                      */
+static bool format = true;     /* format addresses                      */
 static int mime = 0;           /* use MIME-style encapsulations for Bcc */
 static int msgid = 0;          /* add msgid                             */
 static int debug = 0;          /* debugging post                        */
@@ -325,7 +326,7 @@ main (int argc, char **argv)
     FILE *in, *out;
     m_getfld_state_t gstate;
 
-    if (nmh_init(argv[0], 0 /* use context_foil() */)) { return 1; }
+    if (nmh_init(argv[0], false, false)) { return 1; }
 
     mts_init ();
     arguments = getarguments (invo_name, argc, argv, 0);
@@ -338,7 +339,7 @@ main (int argc, char **argv)
                    ambigsw (cp, switches);
                    done (1);
                case UNKWNSW: 
-                   adios (NULL, "-%s unknown", cp);
+                   die("-%s unknown", cp);
 
                case HELPSW: 
                    snprintf (buf, sizeof(buf), "%s [switches] file", invo_name);
@@ -350,7 +351,7 @@ main (int argc, char **argv)
 
                case LIBSW:
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    /* create a minimal context */
                    if (context_foil (cp) == -1)
                        done (1);
@@ -358,9 +359,9 @@ main (int argc, char **argv)
 
                case ALIASW: 
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    if ((state = alias (cp)) != AK_OK)
-                       adios (NULL, "aliasing error in %s - %s",
+                       die("aliasing error in %s - %s",
                                cp, akerror (state));
                    continue;
 
@@ -381,7 +382,7 @@ main (int argc, char **argv)
 
                case FILTSW:
                    if (!(filter = *argp++) || *filter == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    mime = 0;
                    continue;
                case NFILTSW:
@@ -389,10 +390,10 @@ main (int argc, char **argv)
                    continue;
                
                case FRMTSW: 
-                   format++;
+                   format = true;
                    continue;
                case NFRMTSW: 
-                   format = 0;
+                   format = false;
                    continue;
 
                case BITSTUFFSW:
@@ -437,25 +438,25 @@ main (int argc, char **argv)
 
                case WIDTHSW: 
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    if ((outputlinelen = atoi (cp)) < 10)
-                       adios (NULL, "impossible width %d", outputlinelen);
+                       die("impossible width %d", outputlinelen);
                    continue;
 
                case ANNOSW: 
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    if ((pfd = atoi (cp)) <= 2)
-                       adios (NULL, "bad argument %s %s", argp[-2], cp);
+                       die("bad argument %s %s", argp[-2], cp);
                    continue;
 
                case CLIESW:
                    if (!(clientsw = *argp++) || *clientsw == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    continue;
                case SERVSW:
                    if (!(serversw = *argp++) || *serversw == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    continue;
                case SNOOPSW:
                    snoop++;
@@ -463,7 +464,7 @@ main (int argc, char **argv)
 
                case PARTSW:
                    if (!(partno = *argp++) || *partno == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    continue;
 
                case SASLSW:
@@ -476,12 +477,12 @@ main (int argc, char **argv)
 
                case SASLMECHSW:
                    if (!(saslmech = *argp++) || *saslmech == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    continue;
 
                case AUTHSERVICESW:
                    if (!(auth_svc = *argp++) || *auth_svc == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    oauth_flag++;
                    continue;
 
@@ -496,7 +497,7 @@ main (int argc, char **argv)
                    int i;
 
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
 
                    for (i = 0; oauthswitches[i].profname != NULL; i++) {
                        if (oauthswitches[i].switchnum == swnum) {
@@ -506,7 +507,7 @@ main (int argc, char **argv)
                    }
 
                    if (oauthswitches[i].profname == NULL)
-                       adios (NULL, "internal error: cannot map switch %s "
+                       die("internal error: cannot map switch %s "
                               "to profile entry", argp[-2]);
 
                    oauth_flag++;
@@ -515,12 +516,12 @@ main (int argc, char **argv)
 
                case USERSW:
                    if (!(user = *argp++) || *user == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    continue;
 
                case PORTSW:
                    if (!(port = *argp++) || *port == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    continue;
 
                case TLSSW:
@@ -545,55 +546,55 @@ main (int argc, char **argv)
 
                case FILEPROCSW:
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    fileproc = cp;
                    continue;
 
                case MHLPROCSW:
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    mhlproc = cp;
                    continue;
 
                case MTSSM:
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                     sendmail = cp;
                    continue;
 
                case MTSSW:
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                     save_mts_method (cp);
                    continue;
 
                case CREDENTIALSSW: {
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                    add_profile_entry ("credentials", cp);
                    continue;
                }
 
                case MESSAGEIDSW:
                    if (!(cp = *argp++) || *cp == '-')
-                       adios (NULL, "missing argument to %s", argp[-2]);
+                       die("missing argument to %s", argp[-2]);
                     if (save_message_id_style (cp) != 0)
-                       adios (NULL, "unsupported messageid \"%s\"", cp);
+                       die("unsupported messageid \"%s\"", cp);
                    continue;
            }
        }
        if (msg)
-           adios (NULL, "only one message at a time!");
+           die("only one message at a time!");
         msg = cp;
     }
 
     alias (AliasFile);
 
     if (!msg)
-       adios (NULL, "usage: %s [switches] file", invo_name);
+       die("usage: %s [switches] file", invo_name);
 
     if (outputlinelen < 10)
-       adios (NULL, "impossible width %d", outputlinelen);
+       die("impossible width %d", outputlinelen);
 
     if ((in = fopen (msg, "r")) == NULL)
        adios (msg, "unable to open");
@@ -609,7 +610,7 @@ main (int argc, char **argv)
        } else {
            char *cp = m_mktemp2(NULL, invo_name, NULL, &out);
            if (cp == NULL) {
-               adios(NULL, "unable to create temporary file in %s",
+               die("unable to create temporary file in %s",
                      get_temp_dir());
            }
             strncpy(tmpfil, cp, sizeof(tmpfil));
@@ -653,10 +654,10 @@ main (int argc, char **argv)
 
            case LENERR: 
            case FMTERR: 
-               adios (NULL, "message format error in component #%d", compnum);
+               die("message format error in component #%d", compnum);
 
            default: 
-               adios (NULL, "getfld() returned %d", state);
+               die("getfld() returned %d", state);
        }
        break;
     }
@@ -726,7 +727,7 @@ main (int argc, char **argv)
        char sbuf[128];
 
        if (auth_svc == NULL) {
-           adios(NULL, "No authentication service given with -authservice");
+           die("No authentication service given with -authservice");
        }
 
        for (i = 0; oauthswitches[i].profname != NULL; i++) {
@@ -999,8 +1000,7 @@ putfmt (char *name, char *str, int *eai, FILE *out)
                           aliases and put them in Bcc:, but then
                           they'd have the Blind-Carbon-Copy
                           indication. */
-                       adios (NULL,
-                              "blind lists not compatible with"
+                       die(                           "blind lists not compatible with"
                               " sendmail/pipe");
                    }
 
@@ -1210,9 +1210,9 @@ finish_headers (FILE *out)
     }
 
     if (badmsg)
-       adios (NULL, "re-format message and try again");
+       die("re-format message and try again");
     if (!recipients)
-       adios (NULL, "no addressees");
+       die("no addressees");
 }
 
 
@@ -1234,7 +1234,8 @@ static int
 putadr (char *name, char *aka, struct mailname *mp, FILE *out,
        unsigned int flags, char *savehdr, unsigned int savehdrsize)
 {
-    int len, saveappend = 0;
+    int len;
+    bool saveappend = false;
     unsigned int shlen;
     char *cp;
     char buffer[BUFSIZ];
@@ -1252,7 +1253,7 @@ putadr (char *name, char *aka, struct mailname *mp, FILE *out,
 
     if (savehdr) {
        shlen = strlen(savehdr);
-       saveappend = 1;
+       saveappend = true;
     }
 
     if (*aka && mp->m_type != UUCPHOST && !mp->m_pers)
@@ -1274,7 +1275,7 @@ putadr (char *name, char *aka, struct mailname *mp, FILE *out,
            fprintf (out, ",\n%*s", linepos = nameoutput, "");
            if (saveappend) {
                if (shlen + 2 + nameoutput + len >= savehdrsize) {
-                   saveappend = 0;
+                   saveappend = false;
                } else {
                    snprintf(savehdr + shlen, savehdrsize - shlen, ",\n%*s",
                             linepos, "");
@@ -1285,7 +1286,7 @@ putadr (char *name, char *aka, struct mailname *mp, FILE *out,
            linepos += 2;
            if (saveappend) {
                if (shlen + 2 + len >= savehdrsize) {
-                   saveappend = 0;
+                   saveappend = false;
                } else {
                    strncat(savehdr, ", ", savehdrsize - shlen);
                }
@@ -1369,26 +1370,26 @@ pl (void)
     int i;
     struct mailname *mp;
 
-    printf ("-------\n\t-- Addresses --\nlocal:\t");
+    fputs("-------\n\t-- Addresses --\nlocal:\t", stdout);
     for (mp = localaddrs.m_next; mp; mp = mp->m_next)
        printf ("%s%s%s", mp->m_mbox,
                mp->m_bcc ? "[BCC]" : "",
                mp->m_next ? ",\n\t" : "");
 
-    printf ("\nnet:\t");
+    fputs("\nnet:\t", stdout);
     for (mp = netaddrs.m_next; mp; mp = mp->m_next)
        printf ("%s%s@%s%s%s", FENDNULL(mp->m_path),
                mp->m_mbox, mp->m_host,
                mp->m_bcc ? "[BCC]" : "",
                mp->m_next ? ",\n\t" : "");
 
-    printf ("\nuucp:\t");
+    fputs("\nuucp:\t", stdout);
     for (mp = uuaddrs.m_next; mp; mp = mp->m_next)
        printf ("%s!%s%s%s", mp->m_host, mp->m_mbox,
                mp->m_bcc ? "[BCC]" : "",
                mp->m_next ? ",\n\t" : "");
 
-    printf ("\n\t-- Folder Copies --\nfcc:\t");
+    fputs("\n\t-- Folder Copies --\nfcc:\t", stdout);
     for (i = 0; i < fccind; i++)
        printf ("%s%s", fccfold[i], i + 1 < fccind ? ",\n\t" : "");
     putchar('\n');
@@ -1446,7 +1447,7 @@ insert_fcc (struct headers *hdr, char *pp)
        return;
 
     if (fccind >= FCCS)
-       adios (NULL, "too many %ss", hdr->value);
+       die("too many %ss", hdr->value);
     fccfold[fccind++] = mh_xstrdup(cp);
 }
 
@@ -1464,7 +1465,7 @@ make_bcc_file (int dashstuff)
     char *tfile = NULL, *program;
 
     if ((tfile = m_mktemp2(NULL, "bccs", NULL, &out)) == 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 (bccfil, tfile, sizeof(bccfil));
 
@@ -1489,12 +1490,12 @@ make_bcc_file (int dashstuff)
         * prefix and check again.
         */
        if ((cp = strchr(prefix, 'a')) == NULL)
-           adios (NULL, "lost prefix start");
+           die("lost prefix start");
        while (find_prefix () == NOTOK) {
            if (*cp < 'z')
                (*cp)++;
            else if (*++cp == 0)
-                adios (NULL, "can't find a unique delimiter string");
+                die("can't find a unique delimiter string");
             else
                 (*cp)++;
        }
@@ -1847,7 +1848,7 @@ do_an_address (struct mailname *lp, int talk)
        return;
     }
     if (talk)
-       printf (": ");
+       fputs(": ", stdout);
     fflush (stdout);
 
     switch (retval = sm_wadr (mbox, host,