accepted vi'is exit approach ^[ (suggested by Arkaduisz)

[dmenu.git] / draw.c

diff --git a/draw.c b/draw.c
index 2028fd94d2d72d239929018afe32d36d9c86774b..76f0c54eeff3086399a10d1698e0c47ea7ff056b 100644 (file)
--- a/draw.c
+++ b/draw.c
@@ -96,7 +96,7 @@ initdc(void) {
        DC *dc;
 
        if(!setlocale(LC_CTYPE, "") || !XSupportsLocale())
-               fprintf(stderr, "no locale support\n");
+               fputs("no locale support\n", stderr);
        if(!(dc = calloc(1, sizeof *dc)))
                eprintf("cannot malloc %u bytes:", sizeof *dc);
        if(!(dc->dpy = XOpenDisplay(NULL)))
@@ -121,26 +121,27 @@ initfont(DC *dc, const char *fontstr) {
 Bool
 loadfont(DC *dc, const char *fontstr) {
        char *def, **missing, **names;
-       int i, n = 1;
+       int i, n;
        XFontStruct **xfonts;
 
        if(!*fontstr)
                return False;
-       if((dc->font.set = XCreateFontSet(dc->dpy, fontstr, &missing, &n, &def)))
+       if((dc->font.set = XCreateFontSet(dc->dpy, fontstr, &missing, &n, &def))) {
                n = XFontsOfFontSet(dc->font.set, &xfonts, &names);
-       else if((dc->font.xfont = XLoadQueryFont(dc->dpy, fontstr)))
-               xfonts = &dc->font.xfont;
-       else
-               n = 0;
-
-       for(i = 0; i < n; i++) {
-               dc->font.ascent  = MAX(dc->font.ascent,  xfonts[i]->ascent);
-               dc->font.descent = MAX(dc->font.descent, xfonts[i]->descent);
-               dc->font.width   = MAX(dc->font.width,   xfonts[i]->max_bounds.width);
+               for(i = 0; i < n; i++) {
+                       dc->font.ascent  = MAX(dc->font.ascent,  xfonts[i]->ascent);
+                       dc->font.descent = MAX(dc->font.descent, xfonts[i]->descent);
+                       dc->font.width   = MAX(dc->font.width,   xfonts[i]->max_bounds.width);
+               }
+       }
+       else if((dc->font.xfont = XLoadQueryFont(dc->dpy, fontstr))) {
+               dc->font.ascent  = dc->font.xfont->ascent;
+               dc->font.descent = dc->font.xfont->descent;
+               dc->font.width   = dc->font.xfont->max_bounds.width;
        }
        if(missing)
                XFreeStringList(missing);
-       return (dc->font.set || dc->font.xfont);
+       return dc->font.set || dc->font.xfont;
 }
 
 void
@@ -153,10 +154,10 @@ resizedc(DC *dc, unsigned int w, unsigned int h) {
        if(dc->canvas)
                XFreePixmap(dc->dpy, dc->canvas);
 
-       dc->canvas = XCreatePixmap(dc->dpy, DefaultRootWindow(dc->dpy), w, h,
-                                  DefaultDepth(dc->dpy, DefaultScreen(dc->dpy)));
        dc->w = w;
        dc->h = h;
+       dc->canvas = XCreatePixmap(dc->dpy, DefaultRootWindow(dc->dpy), w, h,
+                                  DefaultDepth(dc->dpy, DefaultScreen(dc->dpy)));
 }
 
 int