Changed types and added casts so that build is clean with gcc -Wsign-compare.

Pulled in from nmh. Thanks to David Levine.

diff --git a/sbr/fmt_compile.c b/sbr/fmt_compile.c
index 435740eca429d224694c5721b1326c8ecaff791c..dd0b52af2374295afd2ecc57c466a266259e6ab8 100644 (file)
--- a/sbr/fmt_compile.c
+++ b/sbr/fmt_compile.c
@@ -302,7 +302,7 @@ int
 fmt_compile(char *fstring, struct format **fmt)
 {
        register char *cp;
-       int i;
+       size_t i;
 
        if (format_string)
                free(format_string);

diff --git a/sbr/fmt_scan.c b/sbr/fmt_scan.c
index 94285a6fbc02f2125cb854a947236bd7f2cbdfbd..5103dbd772697372477cda28ae083248f9611872 100644 (file)
--- a/sbr/fmt_scan.c
+++ b/sbr/fmt_scan.c
@@ -475,13 +475,13 @@ fmt_scan(struct format *format, char *scanl, int width, int *dat)
                                                ljust++;
                                }
 
-                               if (!ljust && i > 0 && strlen(str) > i)
+                               if (!ljust && i > 0 && (int)strlen(str) > i)
                                                str[i] = '\0';
                                xp = str;
                                xp += strlen(str) - 1;
                                while (xp > str && isspace(*xp))
                                                *xp-- = '\0';
-                               if (ljust && i > 0 && strlen(str) > i)
+                               if (ljust && i > 0 && (int)strlen(str) > i)
                                        str += strlen(str) - i;
                        }
                        break;

diff --git a/sbr/m_convert.c b/sbr/m_convert.c
index 734bdb58f6cdd53e631b2cb8bab0f1650d5dc851..85d4e587770d0ef4f462e9b33740b48f673c3d9a 100644 (file)
--- a/sbr/m_convert.c
+++ b/sbr/m_convert.c
@@ -272,7 +272,7 @@ m_conv(struct msgs *mp, char *str, int call)
                        return BADNUM;
        }
 
-       for (bp = buf; isalpha(*cp) && (bp - buf < sizeof(buf) - 1); ) {
+       for (bp = buf; isalpha(*cp) && (bp - buf < (int)sizeof(buf) - 1); ) {
                *bp++ = *cp++;
        }
        *bp++ = '\0';

diff --git a/sbr/pidstatus.c b/sbr/pidstatus.c
index 5886ffc0a39485fe446267642dcf30c702c30ff0..c4e1e75847b0577291ee713e0c04e5c17ab04a02 100644 (file)
--- a/sbr/pidstatus.c
+++ b/sbr/pidstatus.c
@@ -49,7 +49,8 @@ pidstatus(int status, FILE *fp, char *cp)
                        if (cp)
                                fprintf(fp, "%s: ", cp);
                        fprintf(fp, "signal %d", signum);
-                       if (signum >= 0 && signum < sizeof(sigmsg) && sigmsg[signum] != NULL)
+                       if (signum >= 0 && signum < (int)sizeof(sigmsg) &&
+                                       sigmsg[signum] != NULL)
                                fprintf(fp, " (%s%s)\n", sigmsg[signum],
                                         WCOREDUMP(status) ? ", core dumped" : "");
                        else

diff --git a/sbr/seq_add.c b/sbr/seq_add.c
index b1228098dcaf14a48d3863e7680a5b14c036c679..7fa7558f2d17c21a550438fea435b3c4e1ddbf98 100644 (file)
--- a/sbr/seq_add.c
+++ b/sbr/seq_add.c
@@ -24,7 +24,8 @@
 int
 seq_addsel(struct msgs *mp, char *cp, int public, int zero)
 {
-       int i, msgnum, new_seq = 1;
+       unsigned int i;
+       int msgnum, new_seq = 1;
 
        if (!seq_nameok(cp))
                return 0;
@@ -116,7 +117,8 @@ seq_addsel(struct msgs *mp, char *cp, int public, int zero)
 int
 seq_addmsg(struct msgs *mp, char *cp, int msgnum, int public, int zero)
 {
-       int i, j, new_seq = 1;
+       unsigned int i;
+       int j, new_seq = 1;
 
        if (!seq_nameok(cp))
                return 0;

diff --git a/sbr/seq_del.c b/sbr/seq_del.c
index e342d1cb0cac4dd0412dc3f911c09c51c3f81f6a..5555f1155fdbee3051f344998c4494a29564cd1a 100644 (file)
--- a/sbr/seq_del.c
+++ b/sbr/seq_del.c
@@ -24,7 +24,8 @@
 int
 seq_delsel(struct msgs *mp, char *cp, int public, int zero)
 {
-       int i, msgnum, new_seq = 1;
+       unsigned int i;
+       int msgnum, new_seq = 1;
 
        if (!seq_nameok(cp))
                return 0;

diff --git a/sbr/seq_read.c b/sbr/seq_read.c
index 5707dae9643634f452dbeb4832814c933789e20b..bfecaf0184dd5bd557d07b57142bb2c2ecf03d54 100644 (file)
--- a/sbr/seq_read.c
+++ b/sbr/seq_read.c
@@ -158,7 +158,8 @@ seq_private(struct msgs *mp)
 static int
 seq_init(struct msgs *mp, char *name, char *field)
 {
-       int i, j, k, is_current;
+       unsigned int i;
+       int j, k, is_current;
        char *cp, **ap;
 
        /*

diff --git a/uip/aliasbr.c b/uip/aliasbr.c
index 6a68c51ae746c7c86109d3d107062d946e2946e7..8b32826b5f9122611ae92bf0aea0c26b24d6f4e2 100644 (file)
--- a/uip/aliasbr.c
+++ b/uip/aliasbr.c
@@ -525,7 +525,7 @@ seek_home(char *name)
        ** The only place where there might be problems.
        ** This assumes that ALL usernames are kept in lowercase.
        */
-       for (c = name, c1 = lname; *c && (c1 - lname < sizeof(lname) - 1);
+       for (c = name, c1 = lname; *c && (c1 - lname < (int)sizeof(lname) - 1);
                        c++, c1++) {
                if (isalpha(*c) && isupper(*c))
                        *c1 = tolower(*c);

diff --git a/uip/anno.c b/uip/anno.c
index d03c99cc06d512701dcc5c7b917881d4b94d07b9..b1d866785536936a6a913baedcde5e1d6642c905 100644 (file)
--- a/uip/anno.c
+++ b/uip/anno.c
@@ -429,7 +429,7 @@ annosbr(int fd, char *file, char *comp, char *text, int datesw, int delete,
        int length;
        int n;  /* number of bytes written */
 
-       mode = fstat(fd, &st) != NOTOK ? (st.st_mode & 0777) : m_gmprot();
+       mode = fstat(fd, &st) != NOTOK ? (int)(st.st_mode & 0777) : m_gmprot();
 
        strncpy(tmpfil, m_mktemp2(file, "annotate", NULL, &tmp),
                        sizeof(tmpfil));

diff --git a/uip/burst.c b/uip/burst.c
index fdc78f43ffa69ac629c11791ae51edfc0674a079..d0d06b6db0216a15e6e571f57d3990348f0680fb 100644 (file)
--- a/uip/burst.c
+++ b/uip/burst.c
@@ -260,7 +260,7 @@ burst(struct msgs **mpp, int msgnum, struct smsg *smsgs, int numburst,
                adios(msgnam, "unable to read message");
 
        mode = fstat(fileno(in), &st) != NOTOK ?
-                       (st.st_mode & 0777) : m_gmprot();
+                       (int)(st.st_mode & 0777) : m_gmprot();
        mp = *mpp;
 
        /*

diff --git a/uip/comp.c b/uip/comp.c
index a9d2543765b3179045163fb9c788ce823ac69f8c..87953ad278baf3fd8883eced99ce5fc7cbaad5ac 100644 (file)
--- a/uip/comp.c
+++ b/uip/comp.c
@@ -169,7 +169,7 @@ main(int argc, char **argv)
                if ((out = creat(drft, m_gmprot())) == NOTOK) {
                        adios(drft, "unable to create");
                }
-               if (write(out, fmtstr, strlen(fmtstr)) != strlen(fmtstr)) {
+               if (write(out, fmtstr, strlen(fmtstr)) != (int)strlen(fmtstr)) {
                        adios(drft, "error writing");
                }
                close(out);

diff --git a/uip/dist.c b/uip/dist.c
index 4eba2eb8ce64f442cf17ce457f894f4e9cc7765b..dabfec75a7dd15bd51f6f1d8f032f2e35eeed052 100644 (file)
--- a/uip/dist.c
+++ b/uip/dist.c
@@ -127,7 +127,7 @@ main(int argc, char **argv)
                adios(drft, "unable to create");
 
        fmtstr = new_fs(form, distcomps);
-       if (write(out, fmtstr, strlen(fmtstr)) != strlen(fmtstr)) {
+       if (write(out, fmtstr, strlen(fmtstr)) != (int)strlen(fmtstr)) {
                adios(drft, "error writing");
        }
        close(out);

diff --git a/uip/dropsbr.c b/uip/dropsbr.c
index 124e5ab55432347e03e640201677f9bcf62fc8b8..3c3e105264f2bdf50c7a72fa9b1a3d5e897395f6 100644 (file)
--- a/uip/dropsbr.c
+++ b/uip/dropsbr.c
@@ -169,7 +169,7 @@ mbox_copy(int to, int from)
                if (i != 0 && strncmp(buffer, "From ", 5) == 0) {
                        write(to, ">", 1);
                }
-               if (write(to, buffer, strlen(buffer)) != strlen(buffer)) {
+               if (write(to, buffer, strlen(buffer)) != (int)strlen(buffer)) {
                        fclose(fp);
                        return NOTOK;
                }

diff --git a/uip/flist.c b/uip/flist.c
index c69055ab77beb9c54a6edaa0bb5b084390241ea5..9302a8a395aef9376cc7a40a99954eea38b9a2e1 100644 (file)
--- a/uip/flist.c
+++ b/uip/flist.c
@@ -71,7 +71,7 @@ static struct Folder *orders = NULL;
 static int nOrders = 0;
 static int nOrdersAlloced = 0;
 static struct Folder *folders = NULL;
-static int nFolders = 0;
+static unsigned int nFolders = 0;
 static int nFoldersAlloced = 0;
 
 /* info on folders to search */
@@ -81,7 +81,7 @@ static int maxfolders;
 
 /* info on sequences to search for */
 static char *sequencesToDo[NUMATTRS];
-static int numsequences;
+static unsigned int numsequences;
 
 static int all = FALSE;  /* scan all folders in top level? */
 static int alphaOrder = FALSE;  /* want alphabetical order only */
@@ -457,7 +457,8 @@ BuildFolderListRecurse(char *dirName, struct stat *s, int searchdepth)
 int
 AddFolder(char *name, int force)
 {
-       int i, msgnum, nonzero;
+       unsigned int i;
+       int msgnum, nonzero;
        int seqnum[NUMATTRS], nSeq[NUMATTRS];
        struct Folder *f;
        struct msgs *mp;
@@ -529,8 +530,8 @@ void
 PrintFolders(void)
 {
        char tmpname[BUFSIZ];
-       int i, j, len, has_private = 0;
-       int maxfolderlen = 0, maxseqlen = 0;
+       unsigned int i, j, len, has_private = 0;
+       unsigned int maxfolderlen = 0, maxseqlen = 0;
        int maxnum = 0, maxseq = 0;
 
        if (!Total) {

diff --git a/uip/forw.c b/uip/forw.c
index e79e39584ccf637831c76702ec3da8b2a3e8a2aa..53b09683b083848178bfbe01dc52c11c8fe61c47 100644 (file)
--- a/uip/forw.c
+++ b/uip/forw.c
@@ -238,7 +238,7 @@ main(int argc, char **argv)
                close(in);
        } else {
                fmtstr = new_fs(form, forwcomps);
-               if (write(out, fmtstr, strlen(fmtstr)) != strlen(fmtstr)) {
+               if (write(out, fmtstr, strlen(fmtstr)) != (int)strlen(fmtstr)) {
                        adios(drft, "error writing");
                }
        }

diff --git a/uip/mark.c b/uip/mark.c
index 79763ff232235ac3da672790fa43a0f19a621823..1917e84f4f41cc6407a4b6fcae0d34513d1247b2 100644 (file)
--- a/uip/mark.c
+++ b/uip/mark.c
@@ -48,8 +48,8 @@ int
 main(int argc, char **argv)
 {
        int addsw = 0, deletesw = 0, debugsw = 0;
-       int listsw = 0, publicsw = -1, zerosw = 0;
-       int seqp = 0, msgnum;
+       int listsw = 0, publicsw = -1, zerosw = 0, msgnum;
+       unsigned int seqp = 0;
        char *cp, *maildir, *folder = NULL, buf[BUFSIZ];
        char **argp, **arguments;
        char *seqs[NUMATTRS + 1];

diff --git a/uip/mhl.c b/uip/mhl.c
index 321c5f989cde23edceaf994eeb9cd4a3b1d9d254..c78619ee6ca8615d058c87677dc5283e1d12a289 100644 (file)
--- a/uip/mhl.c
+++ b/uip/mhl.c
@@ -178,12 +178,12 @@ static int mhldebug = 0;
 #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;
 
@@ -1082,11 +1082,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;

diff --git a/uip/mhmail.c b/uip/mhmail.c
index 6a77d5001ff8c4471925ceae67df17f8095a97a8..2ab9cfd96b0c14fae712f8d495730c474e0ebee8 100644 (file)
--- a/uip/mhmail.c
+++ b/uip/mhmail.c
@@ -45,7 +45,8 @@ int
 main(int argc, char **argv)
 {
        pid_t child_id;
-       int status, i, iscc = 0, nvec;
+       int status, iscc = 0, nvec;
+       unsigned int i;
        int queued = 0, resent = 0, somebody;
        char *cp, *tolist = NULL, *cclist = NULL, *subject = NULL;
        char *from = NULL, *body = NULL, **argp, **arguments;

diff --git a/uip/mhoutsbr.c b/uip/mhoutsbr.c
index f82cc6c35094fe806cbc5f5dfff054a210631c4e..130b4e55f93cf0d8eb37336e531cf2ef4c0236a9 100644 (file)
--- a/uip/mhoutsbr.c
+++ b/uip/mhoutsbr.c
@@ -429,7 +429,7 @@ writeBase64(CT ct, FILE *out)
 int
 writeBase64aux(FILE *in, FILE *out)
 {
-       int cc, n;
+       unsigned int cc, n;
        char inbuf[3];
 
        n = BPERLIN;

diff --git a/uip/mhshowsbr.c b/uip/mhshowsbr.c
index ea9f72e09ff0118af8c178c6c2e2b60470bcdefc..d3ddb948c1acb4047b8db5ad847c4d49e275fc7f 100644 (file)
--- a/uip/mhshowsbr.c
+++ b/uip/mhshowsbr.c
@@ -449,7 +449,8 @@ raw:
                }
        }
 
-       if (buflen <= 0 || (ct->c_termproc && buflen <= strlen(ct->c_termproc))) {
+       if (buflen <= 0 || (ct->c_termproc &&
+                       (size_t)buflen <= strlen(ct->c_termproc))) {
                /*
                ** content_error would provide a more useful error message
                ** here, except that if we got overrun, it probably would
@@ -948,8 +949,8 @@ raw:
                }
        }
 
-       if (buflen <= 0 ||
-                       (ct->c_termproc && buflen <= strlen(ct->c_termproc))) {
+       if (buflen <= 0 || (ct->c_termproc &&
+                       (size_t)buflen <= strlen(ct->c_termproc))) {
                /*
                ** content_error would provide a more useful error message
                ** here, except that if we got overrun, it probably would

diff --git a/uip/pick.c b/uip/pick.c
index 896c37b3e73cec27a793218fadefa501687b36d3..3fb853cdc40e512dfaf1fbfcc38f525953f2efc8 100644 (file)
--- a/uip/pick.c
+++ b/uip/pick.c
@@ -81,7 +81,8 @@ static void putzero_done(int) NORETURN;
 int
 main(int argc, char **argv)
 {
-       int publicsw = -1, zerosw = 1, seqp = 0, vecp = 0;
+       int publicsw = -1, zerosw = 1, vecp = 0;
+       unsigned int seqp = 0;
        int lo, hi, msgnum;
        char *maildir, *folder = NULL, buf[100];
        char *cp, **argp, **arguments;

diff --git a/uip/rcvstore.c b/uip/rcvstore.c
index 04b62d85ca4b6ff41a8c9d79ef24c06c9d74e829..fc98b4ac11b9016fca32672154fad56243adacf1 100644 (file)
--- a/uip/rcvstore.c
+++ b/uip/rcvstore.c
@@ -51,7 +51,8 @@ main(int argc, char **argv)
 {
        int publicsw = -1, zerosw = 0;
        int create = 1, unseensw = 1;
-       int fd, msgnum, seqp = 0;
+       int fd, msgnum;
+       size_t seqp = 0;
        char *cp, *maildir, *folder = NULL, buf[BUFSIZ];
        char **argp, **arguments, *seqs[NUMATTRS+1];
        struct msgs *mp;

diff --git a/uip/sortm.c b/uip/sortm.c
index 125991cc112b1ce7dfd3c4ae2e81bdfedfaeba18..6ba8a9c2adb54a180530fb4fa60377ea1a42029a 100644 (file)
--- a/uip/sortm.c
+++ b/uip/sortm.c
@@ -44,7 +44,7 @@ static struct smsg *smsgs;
 int nmsgs;
 
 char *subjsort = NULL;  /* sort on subject if != 0 */
-unsigned long datelimit = 0;
+time_t datelimit = 0;
 int submajor = 0;  /* if true, sort on subject-major */
 int verbose;