Eliminated compilation warnings.
authorDan Harkless <dan@harkless.org>
Fri, 16 Jul 1999 01:11:44 +0000 (01:11 +0000)
committerDan Harkless <dan@harkless.org>
Fri, 16 Jul 1999 01:11:44 +0000 (01:11 +0000)
37 files changed:
mts/sendmail/hosts.c
mts/sendmail/sendmail.c
sbr/addrsbr.c
sbr/fmt_addr.c
sbr/fmt_scan.c
uip/aliasbr.c
uip/mhbuild.c
uip/mhlist.c
uip/mhn.c
uip/mhparam.c
uip/mhpath.c
uip/mhshow.c
uip/mhstore.c
uip/mhtest.c
uip/packf.c
uip/pick.c
uip/popi.c
uip/prompter.c
uip/rcvpack.c
uip/rcvstore.c
uip/refile.c
uip/repl.c
uip/rmf.c
uip/rmm.c
uip/scan.c
uip/send.c
uip/sendsbr.c
uip/show.c
uip/termsbr.c
uip/vmh.c
uip/vmhtest.c
uip/whatnow.c
uip/whatnowsbr.c
uip/whom.c
uip/wmh.c
zotnet/bboards/getbbent.c
zotnet/mf/mf.c

index 55f339387af6ea2c986fe5e83add4dcad9c07527..7d1d808593881a10f09f03a1405f0a6e8daf2749 100644 (file)
@@ -55,7 +55,7 @@ OfficialName (char *name)
        strncpy (buffer, hp->h_name, sizeof(buffer));
        return buffer;
     }
-    if (hosts.h_name || init_hs ())
+    if (hosts.h_name || init_hs ()) {
        for (h = hosts.h_next; h; h = h->h_next)
            if (!strcasecmp (h->h_name, q))
                return h->h_name;
@@ -63,6 +63,7 @@ OfficialName (char *name)
                for (r = h->h_aliases; *r; r++)
                    if (!strcasecmp (*r, q))
                        return h->h_name;
+    }
 
     strncpy (buffer, site, sizeof(buffer));
     return buffer;
index 1ff41387ad26b309f710bd65904012cd45998718..05f8c33f0b2a9ad18d0aa63a4847c4760af23edd 100644 (file)
@@ -112,11 +112,12 @@ sm_init (char *client, char *server, int watch, int verbose,
     if (sm_rfp != NULL && sm_wfp != NULL)
        return RP_OK;
 
-    if (client == NULL || *client == '\0')
+    if (client == NULL || *client == '\0') {
        if (clientname)
            client = clientname;
        else
            client = LocalName();       /* no clientname -> LocalName */
+    }
 
 #ifdef ZMAILER
     if (client == NULL || *client == '\0')
index 0e2f0a424171e88481a8f8bb5a92a50eb30a0575..d1fd730dbb7bd3425c1f46693474c4f65d9d0f06 100644 (file)
@@ -285,7 +285,7 @@ auxformat (struct mailname *mp, int extras)
     if (!extras)
        return addr;
 
-    if (mp->m_pers || mp->m_path)
+    if (mp->m_pers || mp->m_path) {
        if (mp->m_note)
            snprintf (buffer, sizeof(buffer), "%s %s <%s>",
                    legal_person (mp->m_pers ? mp->m_pers : mp->m_mbox),
@@ -294,6 +294,7 @@ auxformat (struct mailname *mp, int extras)
            snprintf (buffer, sizeof(buffer), "%s <%s>",
                    legal_person (mp->m_pers ? mp->m_pers : mp->m_mbox),
                    addr);
+    }
     else
        if (mp->m_note)
            snprintf (buffer, sizeof(buffer), "%s %s", addr, mp->m_note);
index 782eb0a91b2ef9dbb45016b5340e5813ab93d1a9..150aa73d2bd9027abe0d091d8232344164826883 100644 (file)
@@ -82,7 +82,7 @@ formataddr (char *orig, char *str)
     }
 
     /* concatenate all the new addresses onto 'buf' */
-    for (isgroup = 0; cp = getname (str); ) {
+    for (isgroup = 0; (cp = getname (str)); ) {
        if ((mp = getm (cp, NULL, 0, fmt_norm, NULL)) == NULL)
            continue;
 
index 630cfc5c2f4b4db49a75a1abab89043dcf349d7c..0af69aba2533b64127c4f529be202202e2e40b75 100644 (file)
@@ -234,7 +234,7 @@ get_x400_friendly (char *mbox, char *buffer, int buffer_len)
        return NULL;
 
     if (get_x400_comp (mbox, "/PN=", buffer, buffer_len)) {
-       for (mbox = buffer; mbox = strchr(mbox, '.'); )
+       for (mbox = buffer; (mbox = strchr(mbox, '.')); )
            *mbox++ = ' ';
 
        return buffer;
@@ -636,7 +636,7 @@ fmt_scan (struct format *format, char *scanl, int width, int *dat)
            }
            if (fmt->f_type == FT_LS_ADDR)
                goto unfriendly;
-           if ((str = mn->m_pers) == NULL)
+           if ((str = mn->m_pers) == NULL) {
                if ((str = mn->m_note)) {
                    strncpy (buffer, str, sizeof(buffer));
                    str = buffer;
@@ -674,6 +674,7 @@ fmt_scan (struct format *format, char *scanl, int width, int *dat)
                        break;
                  }
                }
+           }
            break;
 
        case FT_LOCALDATE:
index 8ea8c13e12f41aee84ccb4eaedb7745774e9fe8d..b15865e20c602025f131f8cd95f7e668eefae289 100644 (file)
@@ -450,11 +450,12 @@ getalias (char *addrs)
     if (*qp == ',')
        *qp++ = 0;
     for (cp = qp, qp--; qp > pp; qp--)
-       if (*qp != 0)
+       if (*qp != 0) {
            if (isspace (*qp))
                *qp = 0;
            else
                break;
+       }
 
     return pp;
 }
index bacec1003d26f62ecb697b4d678dfdf9f9b91da6..dc0059c1b51f0e81b6a6828d6476b949447fa715 100644 (file)
@@ -363,12 +363,11 @@ main (int argc, char **argv)
     unlink_outfile = 0;
 
     free_content (ct);
-    done (0);
-    /* NOT REACHED */
+    return done (0);
 }
 
 
-void
+int
 done (int status)
 {
     /*
@@ -381,4 +380,5 @@ done (int status)
        unlink (outfile);
 
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index d5f290823540ebd5116f8f252a3b0c5df5b07a4c..ec2f9459f98735719338ec4011f830269d5797a8 100644 (file)
@@ -395,8 +395,7 @@ do_cache:
        context_save ();                  /* save the context file  */
     }
 
-    done (0);
-    /* NOTREACHED */
+    return done (0);
 }
 
 
@@ -415,7 +414,7 @@ pipeser (int i)
 }
 
 
-void
+int
 done (int status)
 {
     CT *ctp;
@@ -425,4 +424,5 @@ done (int status)
            free_content (*ctp);
 
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index 70f730b57aa40390c1b3d2185402e7d8a8cec175..f73853e465d2304cadbb851d6bf11cbdffffd73c 100644 (file)
--- a/uip/mhn.c
+++ b/uip/mhn.c
@@ -708,8 +708,7 @@ do_cache:
        context_save ();                  /* save the context file  */
     }
 
-    done (0);
-    /* NOTREACHED */
+    return done (0);
 }
 
 
@@ -728,7 +727,7 @@ pipeser (int i)
 }
 
 
-void
+int
 done (int status)
 {
     CT *ctp;
@@ -738,4 +737,5 @@ done (int status)
            free_content (*ctp);
 
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index 1da25d312a7e505ef57bad4974f03d56d15abd88..f6ebb651c4b6bcce8a64a1ccce88c478215154d0 100644 (file)
@@ -177,7 +177,7 @@ main(int argc, char **argv)
        }
     }
     
-    done (missed);
+    return done (missed);
 }
 
 
index ac1854e4e803fcfde0e35e92277a02d4534a4759..918371a954f4cd3bb792f4d5da5038ae5f45ed14 100644 (file)
@@ -144,5 +144,5 @@ main(int argc, char **argv)
     seq_save (mp);     /* synchronize message sequences */
     context_save ();   /* save the context file         */
     folder_free (mp);  /* free folder/message structure */
-    done (0);
+    return done (0);
 }
index ff2bee855fbbf4858b1e8f7e32f65fc26326ba00..0791c4837e991127c7b451f4a23c7b630df8d33a 100644 (file)
@@ -475,8 +475,7 @@ do_cache:
        context_save ();                  /* save the context file  */
     }
 
-    done (0);
-    /* NOTREACHED */
+    return done (0);
 }
 
 
@@ -495,7 +494,7 @@ pipeser (int i)
 }
 
 
-void
+int
 done (int status)
 {
     CT *ctp;
@@ -505,4 +504,5 @@ done (int status)
            free_content (*ctp);
 
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index a52e00bf52c2b4085cbaa882f0cd86f4588cd015..c9b6a26b138fa09b091f774ba597eefe9af4c0ff 100644 (file)
@@ -407,8 +407,7 @@ do_cache:
        context_save ();                  /* save the context file  */
     }
 
-    done (0);
-    /* NOTREACHED */
+    return done (0);
 }
 
 
@@ -427,7 +426,7 @@ pipeser (int i)
 }
 
 
-void
+int
 done (int status)
 {
     CT *ctp;
@@ -437,4 +436,5 @@ done (int status)
            free_content (*ctp);
 
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index 80a0c7b3fdbce4c51c6c4212e2d50140071c880f..5c0310916aa6e940a4443f92eb2f8b151e4ea8b7 100644 (file)
@@ -386,8 +386,7 @@ do_cache:
        context_save ();                  /* save the context file  */
     }
 
-    done (0);
-    /* NOTREACHED */
+    return done (0);
 }
 
 
@@ -421,7 +420,7 @@ pipeser (int i)
 }
 
 
-void
+int
 done (int status)
 {
     CT *ctp;
@@ -431,4 +430,5 @@ done (int status)
            free_content (*ctp);
 
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index ec2f975a62eb20531dc45831e12cd086197dcc71..a607da81453c125954d48780cf4cf5d425b498ef 100644 (file)
@@ -197,12 +197,13 @@ main (int argc, char **argv)
     seq_save (mp);
     context_save ();                   /* save the context file         */
     folder_free (mp);                  /* free folder/message structure */
-    done (0);
+    return done (0);
 }
 
-void
+int
 done (int status)
 {
     mbx_close (file, md);
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index d2f72efb768874137525e17cbd170bfabeb71bb9..fb34285627e761ab6c2a56ae12f5bb52704b7fd6 100644 (file)
@@ -303,14 +303,15 @@ main (int argc, char **argv)
     seq_save (mp);                     /* synchronize message sequences */
     context_save ();                   /* save the context file         */
     folder_free (mp);                  /* free folder/message structure */
-    done (0);
+    return done (0);
 }
 
 
-void
+int
 done (int status)
 {
     if (listsw && status && !isatty (fileno (stdout)))
        printf ("0\n");
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index 59ab61aead8d66bacfef2167d3e8ec3ce05e0d54..a48c2234e8c18f4dc80f7db29905777411061ab2 100644 (file)
@@ -255,9 +255,7 @@ main (int argc, char **argv)
     seq_setunseen (mp, 0);             /* set the Unseen-Sequence */
     seq_save (mp);
     context_save ();                   /* save the context file   */
-    done (0);
-
-    /* NOTREACHED */
+    return done (0);
 }
 
 
index fa1bfae8ec5f807c4ef90517dd61d65b0df710b2..e1f0e8d5de6828f9420e5f8ec4ca9ed8a0e7db61 100644 (file)
@@ -397,7 +397,7 @@ no_body:
     unlink (tmpfil);
 
     context_save ();   /* save the context file */
-    done (0);
+    return done (0);
 }
 
 
index 1520cbe147879e6612efc4ae98493495d1f09039..f9834be44811a5ed289bd5dd7f2cf17a1ded8535 100644 (file)
@@ -99,5 +99,5 @@ main (int argc, char **argv)
     if (mbx_close (file, md) == NOTOK)
        done (RCV_MBX);
 
-    done (RCV_MOK);
+    return done (RCV_MOK);
 }
index 81a03e8f98f0feac456fd84db802924645037352..5a3dcbe02447e88823f102053c163e0b489b9f49 100644 (file)
@@ -218,16 +218,17 @@ main (int argc, char **argv)
     unlink (tmpfilenam);       /* remove temporary file                  */
     tmpfilenam = NULL;
 
-    done (0);
+    return done (0);
 }
 
 /*
  * Clean up and exit
  */
-void
+int
 done(int status)
 {
     if (tmpfilenam && *tmpfilenam)
        unlink (tmpfilenam);
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index 83e56395b6be6b3cdf0d3428366c8a6ba887c820..906ea4aa7ba22b857b1df1a27be30d5514a219fe 100644 (file)
@@ -280,7 +280,7 @@ main (int argc, char **argv)
     context_replace (pfolder, folder); /* update current folder   */
     context_save ();                   /* save the context file   */
     folder_free (mp);                  /* free folder structure   */
-    done (0);
+    return done (0);
 }
 
 
index af58c59c5c7d561692ece9880845a7e0f43fc937..67155fa5db55f8058ef4074a06c16f202b010e28 100644 (file)
@@ -430,7 +430,7 @@ try_it_again:
        done (0);
     what_now (ed, nedit, NOUSE, drft, msg, 0, mp,
            anot ? "Replied" : NULL, inplace, cwd);
-    done (1);
+    return done (1);
 }
 
 void
index 059e607503f5ee88c222af6c66943592f8573853..9b12304b56b685b7eea74a2a5737ff31774c76ac 100644 (file)
--- a/uip/rmf.c
+++ b/uip/rmf.c
@@ -115,7 +115,7 @@ main (int argc, char **argv)
        context_replace (pfolder, newfolder);   /* update current folder */
     }
     context_save ();   /* save the context file */
-    done (0);
+    return done (0);
 }
 
 static int
index 1b09907a5b8932e5a7d48511d14506de645588fb..21a0f901f8418593eb2bca7ece22dcbf507cb12a 100644 (file)
--- a/uip/rmm.c
+++ b/uip/rmm.c
@@ -147,5 +147,5 @@ main (int argc, char **argv)
     context_replace (pfolder, folder); /* update current folder   */
     context_save ();                   /* save the context file   */
     folder_free (mp);                  /* free folder structure   */
-    done (0);
+    return done (0);
 }
index 1f9b8c0d6891c1ad350e81fc06be4dc66c63eafb..cbd92773fe2d6f6452c69e6aa53574698669fe62 100644 (file)
@@ -347,5 +347,5 @@ main (int argc, char **argv)
     if (clearflag)
        clear_screen ();
 
-    done (0);
+    return done (0);
 }
index 2370ea4cffceef5323d4bd46550c7da5153a1779..b98d89a8406b1b0055a6b0041f8aaf8c72a2b56e 100644 (file)
@@ -428,5 +428,5 @@ go_to_it:
     }
 
     context_save ();   /* save the context file */
-    done (status);
+    return done (status);
 }
index 245a538c9b6dbe0ae844dfa52c01cec9cb5805bc..b7c4cf24b4447e03687d2a2bb9e1a33015f06ec4 100644 (file)
@@ -31,7 +31,7 @@ static jmp_buf env;
  * external prototypes
  */
 int sendsbr (char **, int, char *, struct stat *, int);
-void done (int);
+int done (int);
 char *getusername (void);
 
 /*
@@ -640,11 +640,12 @@ oops:
 }
 
 
-void
+int
 done (int status)
 {
     if (armed)
        longjmp (env, status ? status : NOTOK);
 
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
index 75e4827fc3e30924c0897944b61e1532fd640bef..7927ac564c0275f1b913f8ad304bb0df52ce6748 100644 (file)
@@ -353,6 +353,7 @@ go_to_it: ;
     vec[0] = r1bindex (proc, '/');
     execvp (proc, vec);
     adios (proc, "unable to exec");
+    return 0;  /* dead code to satisfy the compiler */
 }
 
 /*
index 77af4680fadf1c85143e1c615419780ff82a644a..28aaa8f5088920caacf8202c162c743f2ecf382b 100644 (file)
@@ -170,7 +170,7 @@ sc_length (void)
 static int
 outc (int c)
 {
-    putchar(c);
+    return putchar(c);
 }
 
 
index 8b398e6053ba0f5232aada9f29292c7ba33b78d3..b64bdab534ee60ab14be7ce4d831acf60d8890b3 100644 (file)
--- a/uip/vmh.c
+++ b/uip/vmh.c
@@ -278,7 +278,7 @@ main (int argc, char **argv)
 
     vmh ();
 
-    done (0);
+    return done (0);
 }
 
 
@@ -1432,13 +1432,14 @@ TSTPser (int sig)
 
 /* MISCELLANY */
 
-void
+int
 done (int status)
 {
     TTYoff ();
     pFIN ();
 
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
 
 
index 813e6b74d402d9bd0603876a1d18a053df91498d..46ecf8cc98c484a9ccad1d67cd44873e3ae2c46b 100644 (file)
@@ -88,7 +88,7 @@ main (int argc, char **argv)
     pINI ();
     pLOOP ();
 
-    done (0);
+    return done (0);
 }
 
 
index 083650ed46896f1e52775d022d93207d4f9f1f40..34c560ee8d29a90f534363d01f2daddba58255b1 100644 (file)
@@ -14,5 +14,5 @@ main (int argc, char **argv)
 #ifdef LOCALE
     setlocale(LC_ALL, "");
 #endif
-    WhatNow (argc, argv);
+    return WhatNow (argc, argv);
 }
index b12307dc365cf9681a6541151088aaa323e578c2..854a97258db4eedac03f85e0f948789275add19d 100644 (file)
@@ -363,13 +363,14 @@ editfile (char **ed, char **arg, char *file, int use, struct msgs *mp,
                else {
 #endif
                if (((status & 0xff00) != 0xff00)
-                       && (!reedit || (status & 0x00ff)))
+                   && (!reedit || (status & 0x00ff))) {
                    if (!use && (status & 0xff00) &&
                            (rename (file, cp = m_backup (file)) != NOTOK)) {
                        advise (NULL, "problems with edit--draft left in %s", cp);
                    } else {
                        advise (NULL, "problems with edit--%s preserved", file);
                    }
+               }
                status = -2;    /* maybe "reedit ? -2 : -1"? */
                break;
 #ifdef ATTVIBUG
index aeb8d3b660ea3b167e6c2e72c202e73422a1e1ac..5a8253446b9a6eb56fde2ab2eed20af1aae397f3 100644 (file)
@@ -185,5 +185,5 @@ main (int argc, char **argv)
            done (status);
     }
 
-    exit (-1); /* NOT REACHED */
+    return 0;  /* dead code to satisfy the compiler */
 }
index fce1f59b805ea72e10c9526f4d9f528c692fb41f..2e46dbb1c1555756bcb857321a81bccfdc61fec4 100644 (file)
--- a/uip/wmh.c
+++ b/uip/wmh.c
@@ -200,7 +200,7 @@ main (int argc, char **argv)
 
     vmh ();
 
-    done (0);
+    return done (0);
 }
 
 
@@ -1285,7 +1285,7 @@ SIGser (int sig)
 }
 
 
-void
+int
 done (int status)
 {
     if (dfd != NOTOK)
@@ -1294,6 +1294,7 @@ done (int status)
     pFIN ();
 
     exit (status);
+    return 1;  /* dead code to satisfy the compiler */
 }
 
 
index b9904be25bff5d776bb624ea7d95d30dcb0f6535..5f8e88bbce715e46d67f11513497ec18f27ff388 100644 (file)
@@ -393,11 +393,12 @@ BBread (void)
     r = strchr(bb->bb_addr, '@');
     BBRequest[0] = 0;
 
-    if (*bb->bb_request == '-')
+    if (*bb->bb_request == '-') {
        if (p == NULL && r && *r == '@')
            snprintf (BBRequest, sizeof(BBRequest), "%s%s%s", bb->bb_name, bb->bb_request, r);
        else
            snprintf (BBRequest, sizeof(BBRequest), "%s%s", bb->bb_name, bb->bb_request);
+    }
     else
        if (p == NULL && r && *r == '@' && *bb->bb_request)
            snprintf (BBRequest, sizeof(BBRequest), "%s%s", bb->bb_request, r);
index b129785e88d8f36c759ccc9222adac6cf9f350f9..d1db19fc11a93024156c114d7cd21320fbfb19c5 100644 (file)
@@ -163,7 +163,7 @@ uucpadrx (char *addrs)
        return NULL;
     }
 
-    if ((wp = strchr(cp, ',')) == NULL)
+    if ((wp = strchr(cp, ',')) == NULL) {
        if ((wp = strchr(cp, ' ')) != NULL) {
            xp = wp;
            while (isspace (*xp))
@@ -172,11 +172,12 @@ uucpadrx (char *addrs)
                yp = xp + 4;
                while (isspace (*yp))
                    yp++;
-               if (*yp != 0)
+               if (*yp != 0) {
                    if ((zp = strchr(yp, ' ')) != NULL)
                        *zp = 0, tp = ++zp;
                    else
                        tp = NULL;
+               }
                else
                    *wp = 0, tp = ++wp;
            }
@@ -185,6 +186,7 @@ uucpadrx (char *addrs)
        }
        else
            tp = NULL;
+    }
     else
        *wp = 0, tp = ++wp;
 
@@ -951,7 +953,7 @@ mfgets (FILE *in, char **bp)
                *cp++ = i;
                break;
        }
-       if (cp >= ep)
+       if (cp >= ep) {
            if (!(dp = realloc (pp, (size_t) (len += BUFSIZ)))) {
                free (pp);
                pp = NULL;
@@ -959,5 +961,6 @@ mfgets (FILE *in, char **bp)
            }
            else
                cp += dp - pp, ep = (pp = cp) + len - 2;
+       }
     }
 }