next round of fixes, and more constification
authormirabilos <t.glaser@tarent.de>
Tue, 30 Jul 2019 13:42:39 +0000 (15:42 +0200)
committermirabilos <mirabilos@evolvis.org>
Tue, 30 Jul 2019 13:42:39 +0000 (15:42 +0200)
19 files changed:
Build.sh
contrib/xmindpath/main.c
contrib/xwintoppm/dsimple.c
contrib/xwintoppm/dsimple.h
contrib/xwintoppm/multiVis.c
contrib/xwintoppm/multiVis.h
contrib/xwintoppm/xwintoppm.c
draw.c
image/image.h
image/imagetypes.c
image/imlib_loader.c
image/path.c
image/send.c
image/xloadimage.h
mgp.c
mgp.h
plist.c
print.c
x11dummy.c

index 8039a44..945874e 100644 (file)
--- a/Build.sh
+++ b/Build.sh
@@ -1144,7 +1144,6 @@ if test 1 = $i; then
        ac_flags 1 wmissingproto -Wmissing-prototypes
        # for now (-Wextra)
        ac_flags 1 wnosigncompare -Wno-sign-compare
-       ac_flags 1 wnounusedparameter -Wno-unused-parameter
        # see comment in draw.c:epstoimage()
        ac_flags 1 wnoclobbered -Wno-clobbered
 fi
index d803d54..3377ac1 100644 (file)
@@ -57,6 +57,13 @@ extern int uu_lock(const char *);
 extern int uu_unlock(const char *);
 #endif
 
+#undef unused_parameter
+#if HAVE_ATTRIBUTE_UNUSED
+#define unused_parameter __attribute__((__unused__))
+#else
+#define unused_parameter /* nothing */
+#endif
+
 #ifdef X_DISPLAY_MISSING
 # error this program cannot be compiled without X11.
 #endif
@@ -305,7 +312,7 @@ uucpunlock(const char *name)
 }
 
 static void
-sigtrap(int no)
+sigtrap(int no unused_parameter)
 {
        uucpunlock(remote);
        exit(1);
index 210648d..636ce1c 100644 (file)
@@ -187,7 +187,7 @@ _bitmap_error(int status, const char *filename)
 }
 
 Pixmap
-ReadBitmapFile(Drawable d, char *filename, int *width, int *height,
+ReadBitmapFile(char *filename, int *width, int *height,
     int *x_hot, int *y_hot)
 {
   Pixmap bitmap;
index 2bb78d8..8c048a2 100644 (file)
@@ -63,7 +63,7 @@ char *Get_Display_Name(int *, char **);
 Display *Open_Display(char *);
 void Setup_Display_And_Screen(int *, char **);
 XFontStruct *Open_Font(char *);
-Pixmap ReadBitmapFile(Drawable, char *, int *, int *, int *, int *);
+Pixmap ReadBitmapFile(char *, int *, int *, int *, int *);
 void WriteBitmapFile(char *, Pixmap, int, int, int, int);
 Window Select_Window_Args(int *, char **);
 
index b81d7bd..8913114 100644 (file)
@@ -453,12 +453,11 @@ ReadRegionsInList(Display *disp, Visual *fakeVis,
 
 XImage *
 ReadAreaToImage(Display *disp,
-    /* root win on which grab was done */ Window srcRootWinid,
     /* root rel UL corner of bounding box of grab */ int x, int y,
     /* size of bounding box of grab */ unsigned int width, unsigned int height,
-    int numVisuals, XVisualInfo *pVisuals,
+    XVisualInfo *pVisuals,
     int numOverlayVisuals, OverlayInfo *pOverlayVisuals,
-    int numImageVisuals, XVisualInfo **pImageVisuals,
+    XVisualInfo **pImageVisuals,
     /* list of regions to read from */ list_ptr vis_regions,
     /* list of regions to read from */ list_ptr vis_image_regions,
     int format, int allImage)
index 9f89ddd..3b5a99c 100644 (file)
@@ -48,10 +48,9 @@ extern int GetMultiVisualRegions(
 
 extern XImage *ReadAreaToImage(
 #if NeedFunctionPrototypes
-    Display *, Window, int, int, unsigned int,
-    unsigned int, int, XVisualInfo *, int,
-    OverlayInfo        *, int, XVisualInfo **, list_ptr,
-    list_ptr, int, int
+    Display *, int, int, unsigned int, unsigned int,
+    XVisualInfo *, int, OverlayInfo *, XVisualInfo **,
+    list_ptr, list_ptr, int, int
 #endif
 );
 
index 6656e93..b689a75 100644 (file)
@@ -375,11 +375,11 @@ Window_Dump(Window window, FILE *outf)
            image = XGetImage (dpy, RootWindow(dpy, screen), absx, absy,
                     width, height, AllPlanes, format);
        else
-           image = ReadAreaToImage(dpy, RootWindow(dpy, screen), absx, absy,
+           image = ReadAreaToImage(dpy, absx, absy,
                 width, height,
-               numVisuals,pVisuals,numOverlayVisuals,pOverlayVisuals,
-                numImageVisuals, pImageVisuals,vis_regions,
-                vis_image_regions,format,allImage);
+               pVisuals, numOverlayVisuals, pOverlayVisuals,
+                pImageVisuals, vis_regions,
+                vis_image_regions, format, allImage);
     }
     else
        image = XGetImage (dpy, window, x, y, width, height, AllPlanes, format);
diff --git a/draw.c b/draw.c
index dc16f5e..6ec55e4 100644 (file)
--- a/draw.c
+++ b/draw.c
@@ -86,8 +86,7 @@ static Window search_child_window(void);
 static Window tsearch_child_window(const char *name);
 static Window getNamedWindow(const char *name, Window top);
 static void reparent_child_window(Window, int, int);
-static char *epstoimage(struct render_state *, char *, int, int, int,
-       int, int, int);
+static char *epstoimage(const char *, int, int, int, int, int, int);
 static void image_setcolor(struct render_state *);
 static void x_registerseed(struct render_state *, char *, const char *);
 static const char *x_findseed(struct render_state *, const char *);
@@ -1640,7 +1639,7 @@ obj_draw_image(Drawable target, u_int x, u_int y,
        if (obj->data.image.zoomonclk) {
                regist_zimage_position(obj, x, y, xim->width, xim->height, page);
        }
-       freeXImage(image, ximageinfo);
+       freeXImage(ximageinfo);
 }
 
 static void
@@ -2450,7 +2449,7 @@ image_load(struct render_state *state, char *filename,
                XFlush(display);
        }
 
-       if ((myimage = loadImage(filename, verbose)) == NULL) {
+       if ((myimage = loadImage(filename)) == NULL) {
                fprintf(stderr, "failed to load image file\n");
                cleanup(-1);
        }
@@ -2615,7 +2614,7 @@ image_load_ps(struct render_state *state, char *filename,
                zratio = (float) zoomonclk / 100.0 * window_width / width;
        else
                zratio = 1.0;
-       imagefile = epstoimage(state, fullname, x1, y1v,
+       imagefile = epstoimage(fullname, x1, y1v,
                width * zratio, height * zratio, xzoom * zratio, yzoom * zratio);
        if (imagefile == NULL) {
                fprintf(stderr, "WARN: cannot generate %s file from %s\n",
@@ -3424,7 +3423,7 @@ draw_reinit(struct render_state *state)
  * perhaps move relevant code out into another function?
  */
 static char *
-epstoimage(struct render_state *state, char *epsfile, int x, int y,
+epstoimage(const char *epsfile, int x, int y,
     int width, int height, int xzoom, int yzoom)
 {
        int fd, pfd[3][2];
@@ -3912,7 +3911,7 @@ reset_background_pixmap(void)
        for (i = 0; i < MAXBGPIXMAP; i ++) {
                if (bgpixmap[i].image){
                        XFreePixmap(display, bgpixmap[i].pixmap);
-                       freeXImage(bgpixmap[i].image, bgpixmap[i].ximageinfo);
+                       freeXImage(bgpixmap[i].ximageinfo);
                        freeImage(bgpixmap[i].image);
                }
                bgpixmap[i].ctl = NULL;
@@ -4054,8 +4053,7 @@ regist_background_pixmap(XImageInfo *ximageinfo, Image *image)
        if (i == MAXBGPIXMAP){
                /* no empty slot, we need to make one  */
                XFreePixmap(display, bgpixmap[MAXBGPIXMAP -1].pixmap);
-               freeXImage(bgpixmap[MAXBGPIXMAP -1].image,
-                                       bgpixmap[MAXBGPIXMAP -1].ximageinfo);
+               freeXImage(bgpixmap[MAXBGPIXMAP -1].ximageinfo);
                freeImage(bgpixmap[MAXBGPIXMAP -1].image);
                for (j = MAXBGPIXMAP -2; j >= 0; j --){
                        bgpixmap[j +1].ctl = bgpixmap[j].ctl;
index a4a524e..142a43e 100644 (file)
@@ -77,8 +77,8 @@ typedef struct {
 Image *dither(Image *, unsigned int);
 
 /* imagetypes.c */
-Image *loadImage(char *, unsigned int);
-void   identifyImage(char *);
+Image *loadImage(const char *);
+void   identifyImage(const char *);
 void goodImage(Image *, const char *);
 
 /* new.c */
@@ -133,8 +133,7 @@ extern void cleanup(int)
 #endif
     ;
 
-extern Image *imLoad(char *, char *, unsigned int);
-extern int imIdent(char *, char *); /*XXX TODO: remove, const 1 */
+extern Image *imLoad(const char *, const char *);
 
 void memoryExhausted(void)
 #if HAVE_ATTRIBUTE_NORETURN
index 6d9b356..401af5a 100644 (file)
 #include <errno.h>
 #include "xloadimage.h"
 
-/*XXX TODO: inline this somehow */
-
-static struct imagetypes {
-  int    (*identifier)(char *, char *); /* print out image info if this kind of image */
-  Image *(*loader)(char *, char *, unsigned int);     /* load image if this kind of image */
-  const char *name;       /* name of this image format */
-} ImageTypes[] = {
-  { imIdent,        imLoad,        "Imlib" },
-  { NULL,           NULL,          NULL }
-};
-/* SUPPRESS 560 */
-
 /* load a named image
  */
 
 Image *
-loadImage(char *name, unsigned int verbose)
+loadImage(const char *name)
 { char   fullname[MAXPATHLEN];
-  Image *image;
-  int    a;
 
   if (findImage(name, fullname) < 0) {
     if (errno == ENOENT)
@@ -39,21 +25,15 @@ loadImage(char *name, unsigned int verbose)
       perror(fullname);
     return(NULL);
   }
-  for (a= 0; ImageTypes[a].loader; a++)
-    if ((image = ImageTypes[a].loader(fullname, name, verbose))) {
-      return(image);
-    }
-  fprintf(stderr, "%s: unknown or unsupported image type\n", fullname);
-  return(NULL);
+  return (imLoad(fullname, name));
 }
 
 /* identify what kind of image a named image is
  */
 
 void
-identifyImage(char *name)
+identifyImage(const char *name)
 { char fullname[MAXPATHLEN];
-  int  a;
 
   if (findImage(name, fullname) < 0) {
     if (errno == ENOENT)
@@ -62,12 +42,6 @@ identifyImage(char *name)
       perror(fullname);
     return;
   }
-  for (a= 0; ImageTypes[a].identifier; a++) {
-    if (ImageTypes[a].identifier(fullname, name)) {
-      return;
-    }
-}
-  fprintf(stderr, "%s: unknown or unsupported image type\n", fullname);
 }
 
 void
index 320c698..e0eb9ef 100644 (file)
@@ -8,16 +8,10 @@ static Image *imagedata[IMFILENUM];
 static int imnum;
 static Imlib_Context *id;
 
-static void regist_imdata(char *, Imlib_Image *, Image *);
-
-int
-imIdent(char *fullname, char *name)
-{
-       return 1;
-}
+static void regist_imdata(const char *, Imlib_Image *, Image *);
 
 Image *
-imLoad(char *fullname, char *name, unsigned int verbose)
+imLoad(const char *fullname, const char *name)
 {
        static Display *disp = NULL;
        Image *image = NULL;
@@ -88,7 +82,7 @@ imLoad(char *fullname, char *name, unsigned int verbose)
 }
 
 Imlib_Image *
-search_imdata(char *fullname)
+search_imdata(const char *fullname)
 {
        int i;
        for (i = 0; i < imnum; i ++){
@@ -100,7 +94,7 @@ search_imdata(char *fullname)
 }
 
 static void
-regist_imdata(char *fullname, Imlib_Image *im, Image *image)
+regist_imdata(const char *fullname, Imlib_Image *im, Image *image)
 {
        strlcpy(imfile[imnum], fullname, 1024);
        imdata[imnum] = im;
index c01f7e2..9fdda69 100644 (file)
@@ -133,7 +133,7 @@ loadPathsAndExts(void)
 }
 
 static int
-fileIsOk(char *fullname, struct stat *sbuf)
+fileIsOk(const char *fullname, struct stat *sbuf)
 {
   if ((sbuf->st_mode & S_IFMT) == S_IFDIR) /* is a directory */
     return(0);
@@ -145,7 +145,7 @@ fileIsOk(char *fullname, struct stat *sbuf)
  */
 
 int
-findImage(char *name, char *fullname)
+findImage(const char *name, char *fullname)
 { unsigned int   p, e;
   struct stat    sbuf;
 
index 9ce3518..a3ff9c5 100644 (file)
@@ -56,7 +56,7 @@ ximageToPixmap(Display *disp, Window parent, XImageInfo *ximageinfo)
  */
 
 static unsigned int
-bitsPerPixelAtDepth(Display *disp, int scrn, unsigned int depth2)
+bitsPerPixelAtDepth(Display *disp, unsigned int depth2)
 {
 #if 1 /* the way things are */
   unsigned int a;
@@ -456,7 +456,7 @@ again:
     /* modify image data to match visual and colormap
      */
 
-    dbits= bitsPerPixelAtDepth(disp, scrn, ddepth);
+    dbits= bitsPerPixelAtDepth(disp, ddepth);
     ximageinfo->depth= ddepth;
     dpixlen= (dbits + 7) / 8;
 
@@ -625,7 +625,7 @@ sendXImage(XImageInfo *ximageinfo,
  */
 
 void
-freeXImage(Image *image, XImageInfo *ximageinfo)
+freeXImage(XImageInfo *ximageinfo)
 {
   if (ximageinfo->gc)
     XFreeGC(ximageinfo->disp, ximageinfo->gc);
index 1cc7a1e..517bfaa 100644 (file)
@@ -40,7 +40,7 @@ typedef struct {
 
 /* path.c */
 char *expandPath(char *);
-int findImage(char *, char *)
+int findImage(const char *, char *)
 #if HAVE_ATTRIBUTE_BOUNDED
     __attribute__((__bounded__(__minbytes__, 2, MAXPATHLEN)))
 #endif
@@ -55,9 +55,9 @@ void        sendXImage(XImageInfo *, int, int, int, int,
 XImageInfo *imageToXImage(Display *, int, Visual *, unsigned int,
     Image *, unsigned int, unsigned int, unsigned int, unsigned int);
 Pixmap      ximageToPixmap(Display *, Window, XImageInfo *);
-void        freeXImage(Image *, XImageInfo *);
+void        freeXImage(XImageInfo *);
 
 /* imlib_loader.c */
 Pixmap pixmap_fromimimage(Imlib_Image *, int, int, Window);
 void manage_pixmap(Pixmap, int, int);
-Imlib_Image *search_imdata(char *);
+Imlib_Image *search_imdata(const char *);
diff --git a/mgp.c b/mgp.c
index 33bf90a..43c555a 100644 (file)
--- a/mgp.c
+++ b/mgp.c
@@ -798,7 +798,7 @@ main_loop(u_int start_page)
                        for (i = 1; i <= maxpage; i++) {
                                if (e.xany.window == plwin[i]) {
                                        state_goto(&state, i, 0);
-                                       pl_off(&state);
+                                       pl_off();
                                        break;
                                }
                        }
@@ -969,7 +969,7 @@ rakugaki_toggle:
                        case XK_r:
                                if (control) {
 reload:
-                                       pl_off(&state);
+                                       pl_off();
                                        cached_page = 0;
                                        reset_background_pixmap();
                                        draw_reinit(&state);
@@ -1062,7 +1062,7 @@ repaint:;
                        switch (key) {
                        case XK_Control_L:
                        case XK_Control_R:
-                               pl_off(&state);
+                               pl_off();
                                control = 0;
                                break;
                        case XK_Shift_L:
@@ -1119,7 +1119,7 @@ repaint:;
 
                                if (pg_mode)
                                        pg_off();
-                               pl_off(&state);
+                               pl_off();
                                window_width = e.xconfigure.width;
                                window_height = e.xconfigure.height;
                                state.width = e.xconfigure.width;
diff --git a/mgp.h b/mgp.h
index be63330..9b2c9c7 100644 (file)
--- a/mgp.h
+++ b/mgp.h
@@ -94,6 +94,13 @@ typedef unsigned long u_long;
 #define __dead /* nothing */
 #endif
 
+#undef unused_parameter
+#if HAVE_ATTRIBUTE_UNUSED
+#define unused_parameter __attribute__((__unused__))
+#else
+#define unused_parameter /* nothing */
+#endif
+
 #define DEFAULT_FORE   "yellow"
 #define DEFAULT_BACK   "black"
 #define PAGELIST_FONT  "a14"
@@ -644,7 +651,7 @@ extern void debug1(struct ctrl *);
 
 /*plist.c*/
 extern void pl_on(struct render_state *);
-extern void pl_off(struct render_state *);
+extern void pl_off(void);
 extern void pl_pdraw(struct render_state *, int, GC);
 extern void pl_title(u_int);
 extern const char *page_title(u_int);
diff --git a/plist.c b/plist.c
index 71648dd..866da9c 100644 (file)
--- a/plist.c
+++ b/plist.c
@@ -89,7 +89,7 @@ pl_on(struct render_state *state)
  * Turn off the page list
  */
 void
-pl_off(struct render_state *state)
+pl_off(void)
 {
        int i;
 
diff --git a/print.c b/print.c
index 9eb2f0b..d813645 100644 (file)
--- a/print.c
+++ b/print.c
@@ -283,13 +283,13 @@ static void print_fragment(u_char *, u_int, int, int);
 static void text_remember(char *, int, int, int, int);
 static void icon_remember(int, int, int, u_long);
 static void image_remember(struct ctrl *, struct imagepool *);
-static void image_load_print(char *, int, int, int, int, int, int);
+static void image_load_print(const char *, int, int, int, int, int);
 static void print_usage(char *) __dead;
 static void setpencolor(u_long);
 static void print_full_image(XImage *, Visual *);
 static void print_8_image(XImage *);
 static void print_32_image(XImage *, Visual *);
-static void print_eps(FILE *, char *, int, int, int, int, int);
+static void print_eps(FILE *, char *, int, int, int);
 static struct fontmap *findfont(int, int, char *);
 static char *fontstring(struct fontmap *);
 static void loadfont(struct fontmap *, const char *);
@@ -1118,7 +1118,7 @@ line_end(void)
                cp1 = imagepool[i].image;
                if (cp1) {
                        image_load_print(cp1->ctm_fname, cp1->ctm_numcolor,
-                               cp1->ctm_ximagesize, cp1->ctm_yimagesize, 0,
+                               cp1->ctm_ximagesize, cp1->ctm_yimagesize,
                                cp1->ctm_zoomflag, cp1->ctm_rotate);
                }
                fprintf(fp, "/xpos xpos%d def xpos ypos moveto\n",
@@ -1398,7 +1398,7 @@ print_fragment(u_char *data, u_int len, int kanjimode, int pfx)
 }
 
 static char *
-checkeps(char *fname)
+checkeps(const char *fname)
 {
        static char epsfile[MAXPATHLEN];
        static char fullname[MAXPATHLEN];
@@ -1567,7 +1567,7 @@ image_remember(struct ctrl *cp, struct imagepool *pool)
                width = x2 - x1;
                height = y2 - y1v;
        } else {
-               myimage = loadImage(cp->ctm_fname, mgp_flag & FL_VERBOSE);
+               myimage = loadImage(cp->ctm_fname);
                if (!myimage) {
                        fprintf(stderr, "failed to open %s\n", cp->ctm_fname);
                        exit(1);
@@ -1616,7 +1616,7 @@ image_remember(struct ctrl *cp, struct imagepool *pool)
                return;
 
 noneps:
-       myimage = loadImage(cp->ctm_fname, mgp_flag & FL_VERBOSE);
+       myimage = loadImage(cp->ctm_fname);
        if (!myimage) {
                fprintf(stderr, "failed to open %s\n", cp->ctm_fname);
                exit(1);
@@ -1652,8 +1652,8 @@ noneps:
 }
 
 static void
-image_load_print(char *filename, int numcolor,
-    int ximagesize, int yimagesize, int backflag, int zoomflag, int rotate)
+image_load_print(const char *filename, int numcolor,
+    int ximagesize, int yimagesize, int zoomflag, int rotate)
 {
        Image *myimage, *image;
        Pixmap mypixmap;
@@ -1676,8 +1676,7 @@ image_load_print(char *filename, int numcolor,
        p = checkeps(filename);
        if (p) {
                epsfp = fopen(p, "r");
-               print_eps(epsfp, p, numcolor, ximagesize, yimagesize,
-                       backflag, zoomflag);
+               print_eps(epsfp, p, ximagesize, yimagesize, zoomflag);
                fclose(epsfp);
                return;
        }
@@ -1689,7 +1688,7 @@ image_load_print(char *filename, int numcolor,
 
        screen = DefaultScreen(display);
 
-       if ((myimage = loadImage(filename, verbose)) == NULL)
+       if ((myimage = loadImage(filename)) == NULL)
                exit(-1);       /* fail to load image data */
        switch (rotate) {
                case 0:
@@ -1796,7 +1795,7 @@ image_load_print(char *filename, int numcolor,
        fprintf(fp, "grestore\n");
 
        XFreePixmap(display, mypixmap);
-       freeXImage(myimage, ximageinfo);
+       freeXImage(ximageinfo);
        freeImage(myimage);
        XUndefineCursor(display, window); XFlush(display);
 }
@@ -1932,8 +1931,8 @@ print_8_image(XImage *print_image)
 }
 
 static void
-print_eps(FILE *epsfp, char *filename, int numcolor,
-    int ximagesize, int yimagesize, int backflag, int zoomflag)
+print_eps(FILE *epsfp, char *filename,
+    int ximagesize, int yimagesize, int zoomflag)
 {
        char line1[BUFSIZ];
        char line2[BUFSIZ];
index 95b1e83..4b0a5da 100644 (file)
@@ -32,7 +32,7 @@ int window_x;
 int window_y;
 
 void
-init_win1(char *geometry)
+init_win1(char *geometry unused_parameter)
 {
        if ((display = XOpenDisplay(NULL)) == NULL) {
                fprintf(stderr, "Can't open display\n");