next round of warning cleanup, almost done
authormirabilos <t.glaser@tarent.de>
Tue, 30 Jul 2019 14:19:29 +0000 (16:19 +0200)
committermirabilos <mirabilos@evolvis.org>
Tue, 30 Jul 2019 14:19:29 +0000 (16:19 +0200)
Build.sh
draw.c
globals.c
image/reduce.c
image/send.c
mgp.h

index 945874e..af97b34 100644 (file)
--- a/Build.sh
+++ b/Build.sh
@@ -1142,8 +1142,6 @@ if test 1 = $i; then
        ac_flags 1 woldstyledefn -Wold-style-definition
        ac_flags 1 wmissingdecl -Wmissing-declarations
        ac_flags 1 wmissingproto -Wmissing-prototypes
-       # for now (-Wextra)
-       ac_flags 1 wnosigncompare -Wno-sign-compare
        # see comment in draw.c:epstoimage()
        ac_flags 1 wnoclobbered -Wno-clobbered
 fi
diff --git a/draw.c b/draw.c
index 6ec55e4..aa5bc46 100644 (file)
--- a/draw.c
+++ b/draw.c
@@ -57,7 +57,8 @@ static void draw_line_output(struct render_state *, char *);
 static void cutin(struct render_state *, int, int, int);
 
 static void draw_string(struct render_state *, char *);
-static u_char *draw_fragment(struct render_state *, u_char *, u_int, const char *, int);
+static u_char *draw_fragment(struct render_state *, u_char *, size_t,
+    const char *, int);
 static struct render_object *obj_alloc(struct render_state *state);
 static void obj_free(struct render_state *, struct render_object *);
 static int obj_new_xfont(struct render_state *, int, int, int,
@@ -93,11 +94,11 @@ static const char *x_findseed(struct render_state *, const char *);
 static int get_regid(const char *);
 
 static void XClearPixmap(Display *, Drawable);
-static void cache_page(struct render_state *, int);
+static void cache_page(struct render_state *, unsigned int);
 static void cache_effect1(void);
 static void cache_effect2(void);
 static void set_from_cache(struct render_state *);
-static void pcache_process(int);
+static void pcache_process(unsigned int);
 static void predraw(struct render_state *);
 static void set_background_pixmap(struct ctrl *);
 static void get_background_pixmap(struct ctrl *, struct render_state *);
@@ -106,12 +107,12 @@ static int valign = VL_BOTTOM;
 
 #define CHECK_CACHE    do {if (caching) {caching = -1; return; }} while (0)
 
-static void set_xrender_color(long, int);
+static void set_xrender_color(unsigned long, int);
 static XftDraw * xft_getdraw(Drawable);
 static u_char *xft_draw_fragment(struct render_state *,
-    u_char *, u_int, const char *, int);
+    u_char *, size_t, const char *, int);
 static int obj_new_xftfont(struct render_state *, int, int, u_char *,
-    int, const char *, const char *, int, int, XftFont *);
+    size_t, const char *, const char *, int, int, XftFont *);
 static XftFont * xft_setfont(const char *, int, const char *);
 XftFont *xft_font;
 XftDraw *xft_draw[2];
@@ -396,7 +397,7 @@ draw_one(struct render_state *state, XEvent *e)
                if ((mgp_flag & FL_FRDCACHE) && cache_mode) {
                        if (XCheckMaskEvent(display, emask, e) == True)
                                goto got_event;
-                       cache_page(&cache_state, state->page +1);
+                       cache_page(&cache_state, state->page + 1);
                        /* check if we got some events during caching */
                        if (XCheckMaskEvent(display, emask, e) == True)
                                goto got_event;
@@ -414,7 +415,7 @@ draw_one(struct render_state *state, XEvent *e)
                /* we have no event in 2sec, so..*/
                if (!FD_ISSET(xfd, &fds)) {
                        if ((mgp_flag & FL_FRDCACHE) && !cache_mode)
-                               cache_page(&cache_state, state->page +1);
+                               cache_page(&cache_state, state->page + 1);
                        timebar(state);
                        e->type = 0;
                        return True;
@@ -843,7 +844,7 @@ draw_line_start(struct render_state *state)
 
 void
 draw_line_itemsize(struct render_state *state,
-    int ascent, int descent, int flheight)
+    unsigned int ascent, unsigned int descent, int flheight)
 {
        ascent -= state->charoff;
        descent += state->charoff;
@@ -1131,7 +1132,7 @@ draw_string(struct render_state *state, char *data)
 }
 
 static u_char *
-draw_fragment(struct render_state *state, u_char *p, u_int len,
+draw_fragment(struct render_state *state, u_char *p, size_t len,
     const char *registry, /* 2-octet charset? */ int charset16)
 {
        u_int char_len, i;
@@ -1389,7 +1390,7 @@ obj_new_icon(struct render_state *state, int x, int y,
     u_int itype, u_int isize, u_long color, u_int npoint, XPoint *xpoint)
 {
        struct render_object *obj;
-       int i;
+       unsigned int i;
 
        obj = obj_alloc(state);
        if (obj == NULL)
@@ -1420,7 +1421,7 @@ obj_new_icon(struct render_state *state, int x, int y,
 static Pixel
 obj_image_color(Image *image, Image *bimage, Pixel d, int *inithist)
 {
-       int i, j;
+       unsigned int i, j;
        RGBMap rgb;
        int r, g, b;
        static char hist[256];
@@ -1497,7 +1498,7 @@ static Image *
 obj_image_trans(Image *image, u_int x, u_int y)
 {
        Image *timage;
-       int i, j;
+       unsigned int i, j;
        byte *p, *b;
        Pixel d, n, pd;
        static XColor xcol;
@@ -1565,14 +1566,14 @@ obj_image_trans(Image *image, u_int x, u_int y)
                                b = bgpixmap[bgindex].image->data +
                                        bgpixmap[bgindex].image->width * by * bpl;
                        }
-                       if (memToVal(p, pl) != trans)
+                       if ((int)memToVal(p, pl) != trans)
                                continue;
                        d = memToVal(b, bpl);
                        if (d != pd) {
                                pd = d;
                                n = obj_image_color(image,
                                                bgpixmap[bgindex].image, d, &inithist);
-                               if (n == -1)
+                               if (n == (Pixel)-1)
                                        goto expand;
                        }
                        valToMem(n, p, pl);
@@ -1652,7 +1653,7 @@ obj_draw(struct render_state *state, Drawable target, u_int xpos, u_int ypos)
        const char *registry;
        XChar2b kch[2];
        u_int isize;
-       int i;
+       unsigned int i;
        int lineoff;   /* ypos correction for lines with superscripts */
        XftDraw *dummy;
 
@@ -2032,12 +2033,12 @@ x_setfont(const char *xfont, u_int csize, const char *registry, int *truescalabl
                } else if (fsize > maxsize || fsize < minsize) {
                        continue;
                }
-               if (fsize > csize) {
+               if ((unsigned int)fsize > csize) {
                        fsize = fsize - csize + 100;
                                        /* penalty for larger font */
                } else
                        fsize = csize - fsize;
-               if (error > fsize) {
+               if (error > (unsigned int)fsize) {
                        error = fsize;
                        best = i;
                        if (verbose) {
@@ -2690,7 +2691,7 @@ timebar(struct render_state *state)
 
 static const struct icon_point {
        XPoint xpoint[4];
-       int     point_num;
+       unsigned int point_num;
 } icon_point[] = {{ {{1, 0}, {0, 2}, {2, 2}, {0, 0}}, 3 },
                  { {{0, 0}, {2, 0}, {1, 2}, {0, 0}}, 3 },
                  { {{0, 0}, {0, 2}, {2, 1}, {0, 0}}, 3 },
@@ -2791,7 +2792,7 @@ draw_bar(struct render_state *state, struct ctrl *cp)
                XSetFunction(display, gcsbar, GXcopy);
        }
        col.pixel = cp->ctb_color;
-       if (col.pixel == -1)
+       if (col.pixel == (unsigned int)-1)
                col.pixel = fore_color[caching];
        if (col.pixel != prevcolor) {
                prevcolor = col.pixel;
@@ -2823,7 +2824,7 @@ static void
 process_system(struct render_state *state, struct ctrl *cp)
 {
        pid_t pid;
-       int i;
+       unsigned int i;
        char **argv;
        char buf[BUFSIZ];
 
@@ -2947,7 +2948,7 @@ static void
 process_xsystem(struct render_state *state, struct ctrl *cp)
 {
        pid_t pid;
-       int i;
+       unsigned int i;
        u_int dumint;
        int xloc, yloc;
        u_int xsiz, ysiz;
@@ -3067,7 +3068,7 @@ fail:
        else
                regchild(pid, cp, window_id, cp->cta_flag);
 
-       if (window_id != -1)
+       if (window_id != (Window)-1)
                reparent_child_window(window_id, window_width, window_height);
        else {
                if (mgp_flag & FL_VERBOSE) {
@@ -3111,8 +3112,7 @@ static void
 process_tsystem(struct render_state *state, struct ctrl *cp)
 {
        pid_t pid;
-       int i, argc;
-       u_int dumint;
+       unsigned int i, argc, dumint;
        int xloc, yloc;
        u_int xsiz, ysiz;
        char **argv;
@@ -3635,7 +3635,7 @@ image_setcolor(struct render_state *state)
 {
        struct render_object *obj;
        Image *image;
-       int i;
+       unsigned int i;
        Intensity *red, *green, *blue;
        XColor fore, back;
 
@@ -3824,7 +3824,7 @@ x_findseed(struct render_state *state, const char *registry)
 
 /* cache specified page */
 static void
-cache_page(struct render_state *state, int page)
+cache_page(struct render_state *state, unsigned int page)
 {
        struct ctrl *tmp_bg_ctl;
        int tmp_bgindex;
@@ -3968,7 +3968,7 @@ cache_effect2(void)
        pcache directive process
 */
 static void
-pcache_process(int page)
+pcache_process(unsigned int page)
 {
        if (!pcache.flag)
                return;
@@ -4159,7 +4159,7 @@ get_regid(const char *registry)
 }
 
 void
-set_xrender_color(long value, int opaque)
+set_xrender_color(unsigned long value, int opaque)
 {
        XColor xc;
 
@@ -4177,13 +4177,13 @@ set_xrender_color(long value, int opaque)
 }
 
 static u_char *
-xft_draw_fragment(struct render_state *state, u_char *p, u_int len,
+xft_draw_fragment(struct render_state *state, u_char *p, size_t len,
     const char *registry, /* 2-octet charset? */ int charset16)
 {
        XGlyphInfo extents;
        struct ctrl *cp;
        char *fontname = NULL;
-       int i;
+       unsigned int i;
        static char etab[3][20] = { "iso-2022-jp", "gb2312", "ksc5601"};
        static char rtab[3][20] = { "jisx208", "gb2312", "ksc5601"};
        static char prefix[3][20] = { "\033$B", "\033$A", "\033$(C"};
@@ -4233,7 +4233,7 @@ xft_draw_fragment(struct render_state *state, u_char *p, u_int len,
                    sizeof(out16) - oleft, &extents);
 
                /* line folding check */
-               if (state->width - state->leftfillpos / 2 - state->linewidth <
+               if ((int)(state->width - state->leftfillpos / 2 - state->linewidth) <
                    extents.xOff) {
                        draw_line_end(state);
                        draw_line_start(state);
@@ -4254,10 +4254,11 @@ xft_draw_fragment(struct render_state *state, u_char *p, u_int len,
        XftTextExtents8(display, xft_font, (XftChar8 *)p, len, &extents);
 
        /* line folding check */
-       if (state->width - state->leftfillpos / 2 - state->linewidth < extents.xOff) {
+       if ((int)(state->width - state->leftfillpos / 2 - state->linewidth) < extents.xOff) {
                if (isspace(*(p + len -1))) {
                    XftTextExtents8(display, xft_font, (XftChar8 *)p, len -1, &extents);
-                       if (state->width - state->leftfillpos / 2 - state->linewidth >= extents.xOff) goto nofolding;
+                       if ((int)(state->width - state->leftfillpos / 2 - state->linewidth) >= extents.xOff)
+                               goto nofolding;
                        draw_line_end(state);
                        draw_line_start(state);
                        state->linewidth = state->leftfillpos;
@@ -4284,7 +4285,7 @@ nofolding:
 
 static int
 obj_new_xftfont(struct render_state *state, int x, int y, u_char *p,
-    int len, const char *fontname, const char *registry, int size,
+    size_t len, const char *fontname, const char *registry, int size,
     int charset16, XftFont *xft_font2)
 {
        struct render_object *obj;
index 9c0298b..777c470 100644 (file)
--- a/globals.c
+++ b/globals.c
@@ -58,7 +58,7 @@ int screen;
 int window_width;
 int window_height;
 int caching = 0;
-int cached_page = 0;
+unsigned int cached_page = 0;
 int cache_hit = 0;
 int cache_mode = 0;
 int cache_effect = 0;
index c22b472..7f9d512 100644 (file)
@@ -360,7 +360,7 @@ reduce(Image *image, unsigned int n, unsigned int verbose)
 { unsigned long pixel_counts[32768]; /* pixel occurrance histogram */
   unsigned short pixel_array[32768];
   unsigned long count, midpoint;
-  int x, y, num_pixels, allocated, depth;
+  unsigned int x, y, num_pixels, allocated, depth;
   byte *pixel, *dpixel;
   struct color_area *areas, *largest_area, *smallest_area;
   struct color_area *new_area, *old_area;
@@ -570,7 +570,7 @@ Image *
 expand(Image *image)
 {
   Image *new_image;
-  int x, y;
+  unsigned int x, y;
   Pixel spixval, dpixval;
   byte *spixel, *dpixel, *line;
   unsigned int linelen;
index a3ff9c5..d7ee34d 100644 (file)
@@ -61,8 +61,8 @@ bitsPerPixelAtDepth(Display *disp, unsigned int depth2)
 #if 1 /* the way things are */
   unsigned int a;
 
-  for (a= 0; a < disp->nformats; a++)
-    if (disp->pixmap_format[a].depth == depth2)
+  for (a= 0; a < (unsigned int)disp->nformats; a++)
+    if ((unsigned int)disp->pixmap_format[a].depth == depth2)
       return(disp->pixmap_format[a].bits_per_pixel);
 
 #else /* the way things should be */
@@ -185,9 +185,9 @@ imageToXImage(Display *disp, int scrn,
       /* sanity check
        */
 
-      if ((redcolors > visual2->map_entries) ||
-         (greencolors > visual2->map_entries) ||
-         (bluecolors > visual2->map_entries)) {
+      if ((redcolors > (unsigned int)visual2->map_entries) ||
+         (greencolors > (unsigned int)visual2->map_entries) ||
+         (bluecolors > (unsigned int)visual2->map_entries)) {
        fprintf(stderr, "\
 Warning: inconsistency in color information (this may be ugly)\n");
       }
@@ -196,7 +196,7 @@ Warning: inconsistency in color information (this may be ugly)\n");
       greenstep= 256 / greencolors;
       bluestep= 256 / bluecolors;
       redbottom= greenbottom= bluebottom= 0;
-      for (a= 0; a < visual2->map_entries; a++) {
+      for (a= 0; a < (unsigned int)visual2->map_entries; a++) {
        if (redbottom < 256)
          redtop= redbottom + redstep;
        if (greenbottom < 256)
diff --git a/mgp.h b/mgp.h
index 2f4467a..abce71b 100644 (file)
--- a/mgp.h
+++ b/mgp.h
@@ -381,7 +381,7 @@ struct page_attribute {
 extern struct page_attribute page_attribute[MAXPAGE];
 
 extern int     caching;
-extern int     cached_page;
+extern unsigned int cached_page;
 extern int     cache_hit;
 extern int     cache_mode;
 extern int     cache_effect;
@@ -424,11 +424,13 @@ struct render_state {
        int have_mark;
        u_int mark_ypos;
        u_int repaint;
-       int maxascent;
-       int maxdescent;
+       unsigned int maxascent;
+       unsigned int maxdescent;
        int maxflheight;
-       int max_lineascent;   /* max size above baseline ignoring supscript */
-       int max_linedescent;  /* max size below baseline ignoring subscript */
+       /* max size above baseline ignoring supscript */
+       unsigned int max_lineascent;
+       /* max size below baseline ignoring subscript */
+       unsigned int max_linedescent;
        u_int linewidth;
        u_int brankline;
        u_int opaque;
@@ -467,8 +469,8 @@ struct render_object {
        } data;
        int x;  /* relative from left position of line */
        int y;  /* relative from center position of line (usually negative) */
-       int ascent;     /* size above the baseline */
-       int descent;    /* size below the baseline */
+       unsigned int ascent;    /* size above the baseline */
+       unsigned int descent;   /* size below the baseline */
        int vertloc;    /* vertical placement control */
 #define VL_BASE                0
 #define VL_CENTER      1
@@ -621,7 +623,8 @@ extern void draw_page(struct render_state *, struct ctrl *);
 extern Bool draw_one(struct render_state *, XEvent *);
 extern void timebar(struct render_state *);
 extern void draw_reinit(struct render_state *);
-extern void draw_line_itemsize(struct render_state *, int, int, int);
+extern void draw_line_itemsize(struct render_state *,
+    unsigned int, unsigned int, int);
 extern void draw_line_start(struct render_state *);
 extern void draw_line_end(struct render_state *);