part of -Wsign-compare
authormirabilos <t.glaser@tarent.de>
Tue, 30 Jul 2019 13:55:25 +0000 (15:55 +0200)
committermirabilos <mirabilos@evolvis.org>
Tue, 30 Jul 2019 13:55:25 +0000 (15:55 +0200)
contrib/xmindpath/main.c
contrib/xwintoppm/dsimple.c
contrib/xwintoppm/multiVis.c
contrib/xwintoppm/xwintoppm.c
grammar.y
mgp.c
mgp.h
parse.c
plist.c
print.c

index 3377ac1..f62d06c 100644 (file)
@@ -287,7 +287,8 @@ buttonpress(int button, int state)
        static int buttonstate[5];      /* X11 defines 5 buttons */
 
        offset = button - Button1;
-       if (offset < 0 || sizeof(buttonstate)/sizeof(buttonstate[0]) <= offset)
+       if (offset < 0 ||
+           sizeof(buttonstate)/sizeof(buttonstate[0]) <= (size_t)offset)
                return;
 
        /* edge trigger */
index 636ce1c..9b6ff94 100644 (file)
@@ -417,8 +417,7 @@ Window
 Window_With_Name(Display *dpy2, Window top, char *name)
 {
        Window *children, dummy;
-       unsigned int nchildren;
-       int i;
+       unsigned int nchildren, i;
        Window w=0;
        char *window_name;
 
index 8913114..b708497 100644 (file)
@@ -229,9 +229,12 @@ QueryColorMap(Display *disp, Colormap src_cmap, Visual *src_vis,
        *gShift = greenShift ;
        *bShift = blueShift ;
         for (i=0; i<ncolors; i++) {
-               if( i <= redMask)colors[i].pixel = (i<<redShift) ;
-               if( i <= greenMask)colors[i].pixel |= (i<<greenShift) ;
-               if( i <= blueMask)colors[i].pixel |= (i<<blueShift) ;
+               if((unsigned int)i <= redMask)
+                       colors[i].pixel = (i<<redShift);
+               if((unsigned int)i <= greenMask)
+                       colors[i].pixel |= (i<<greenShift);
+               if((unsigned int)i <= blueMask)
+                       colors[i].pixel |= (i<<blueShift);
                /***** example :for gecko's 3-3-2 map, blue index should be <= 3.
                 colors[i].pixel = (i<<redShift)|(i<<greenShift)|(i<<blueShift);
                *****/
index b689a75..797fae5 100644 (file)
@@ -341,8 +341,10 @@ Window_Dump(Window window, FILE *outf)
     /* clip to window */
     if (absx < 0) width += absx, absx = 0;
     if (absy < 0) height += absy, absy = 0;
-    if (absx + width > dwidth) width = dwidth - absx;
-    if (absy + height > dheight) height = dheight - absy;
+    if ((unsigned int)absx + width > (unsigned int)dwidth)
+       width = dwidth - absx;
+    if ((unsigned int)absy + height > (unsigned int)dheight)
+       height = dheight - absy;
 
     XFetchName(dpy, window, &win_name);
     if (!win_name || !win_name[0]) {
index 2a3b462..fa94724 100644 (file)
--- a/grammar.y
+++ b/grammar.y
@@ -537,7 +537,7 @@ static struct ctrl *
 gen_argsfromstr(int op, char *str, int flag)
 {
        struct ctrl *ct;
-       int siz;
+       unsigned int siz;
        char **h;
 
        if (!(ct = ctlalloc1(op))) {
diff --git a/mgp.c b/mgp.c
index 43c555a..5e10990 100644 (file)
--- a/mgp.c
+++ b/mgp.c
@@ -82,7 +82,7 @@ volatile int ttykey_enable;
 #endif
 static pid_t mypid;
 
-static void genhtml(int);
+static void genhtml(unsigned int);
 static void mgp_usage(char *) __dead;
 static void mgp_show_version(char *) __dead;
 static void beep(void);
@@ -178,7 +178,7 @@ main(int argc, char *argv[])
        char **tmp_argv;       /* manipulated 'argv' */
        int opt;
        char *progname;
-       int start_page = 1;
+       unsigned int start_page = 1;
        char buf[BUFSIZ], *p, *p2;
 
 #if HAVE_SETLOCALE_CTYPE
@@ -419,14 +419,14 @@ main(int argc, char *argv[])
 }
 
 static void
-genhtml(int start_page)
+genhtml(unsigned int start_page)
 {
        struct render_state state;
        char buf[BUFSIZ];
        int fd;
        FILE *html;
        FILE *txt;
-       int page;
+       unsigned int page;
        const char *childdebug;
        const char *convdb[][3] = {
                { "jpg", "cjpeg", "djpeg" },
diff --git a/mgp.h b/mgp.h
index 9b2c9c7..2f4467a 100644 (file)
--- a/mgp.h
+++ b/mgp.h
@@ -652,7 +652,7 @@ extern void debug1(struct ctrl *);
 /*plist.c*/
 extern void pl_on(struct render_state *);
 extern void pl_off(void);
-extern void pl_pdraw(struct render_state *, int, GC);
+extern void pl_pdraw(struct render_state *, unsigned int, GC);
 extern void pl_title(u_int);
 extern const char *page_title(u_int);
 extern void pg_on(void);
diff --git a/parse.c b/parse.c
index f35cdee..9c78ece 100644 (file)
--- a/parse.c
+++ b/parse.c
@@ -417,8 +417,8 @@ page:
                        p2--;
                        *p2 = '\0';
                }
-               while (buf < p2 && p2 - buf < sizeof(buf) && p2[-1] == '\\') {
-
+               while (buf < p2 && ((size_t)(p2 - buf)) < sizeof(buf) &&
+                   p2[-1] == '\\') {
                        /*
                         * check for quoted backslashes at the end of a line:
                         * if it is so, then do NOT ignore the line break and
@@ -516,7 +516,7 @@ page:
                                int pipefd[2];
 
                                if (mgp_flag & FL_NOFORK) {
-                                       int i;
+                                       unsigned int i;
                                        if (mgp_flag & FL_VERBOSE) {
                                            fprintf(stderr, "%s:%d: %%filter ",
                                                filename, lineno);
@@ -1309,7 +1309,7 @@ thirdpass(void)
 void
 debug0(struct ctrl *p)
 {
-       int i;
+       unsigned int i;
 
        fprintf(stderr, "%p: ", p);
        fprintf(stderr, " %s ", ctl_words[p->ct_op].ctl_string);
diff --git a/plist.c b/plist.c
index 866da9c..2f6fdb3 100644 (file)
--- a/plist.c
+++ b/plist.c
@@ -45,7 +45,8 @@ void
 pl_on(struct render_state *state)
 {
        XSetWindowAttributes wattr;
-       int i, ny, pl_x;
+       unsigned int i;
+       int ny, pl_x;
        u_int pl_nx, pl_ny, pl_y;
 
        if (pl_titlewin || !plfs)
@@ -91,7 +92,7 @@ pl_on(struct render_state *state)
 void
 pl_off(void)
 {
-       int i;
+       unsigned int i;
 
        pg_lastpage = -1;
        if (!pl_titlewin)
@@ -112,7 +113,7 @@ pl_off(void)
  * Enbold the current page
  */
 void
-pl_pdraw(struct render_state *state, int i, GC gc)
+pl_pdraw(struct render_state *state, unsigned int i, GC gc)
 {
        char buf[10];
 
@@ -157,7 +158,7 @@ pl_title(u_int page)
 const char *
 page_title(u_int page)
 {
-       int l;
+       unsigned int l;
        const char *p;
        struct ctrl *cp;
 
@@ -223,7 +224,7 @@ pg_draw(struct render_state *state)
                return;
 
        page = state->page;
-       if (page == pg_lastpage)
+       if (page == (unsigned int)pg_lastpage)
                return;
 
        showlast = /* !state->cp || (state->cp->ct_op == CTL_PAUSE && state->cp->cti_value) */
diff --git a/print.c b/print.c
index d813645..b347a14 100644 (file)
--- a/print.c
+++ b/print.c
@@ -1250,7 +1250,7 @@ static void
 line_skip(int x, int y)
 {
        linewidth += x;
-       lineheight = (lineheight < y) ? y : lineheight;
+       lineheight = ((int)lineheight < y) ? (unsigned int)y : lineheight;
 }
 
 static void