diff options
| author | Anselm R Garbe <anselm@garbe.us> | 2011-01-07 16:05:22 +0000 |
|---|---|---|
| committer | Anselm R Garbe <anselm@garbe.us> | 2011-01-07 16:05:22 +0000 |
| commit | 0bc4e41ebdf4dfbdb4e865ab5297f5df4f0d9e16 (patch) | |
| tree | b67ef2fb9a2f4208859fa59477ed60800fd19027 /dwm.c | |
| parent | 703c4dd2530bf7d4876e388757e62d04f40dc643 (diff) | |
| download | dwm-0bc4e41ebdf4dfbdb4e865ab5297f5df4f0d9e16.tar.gz | |
applied anonymous patch, I don't think the reversed array access has semantic side-effects
Diffstat (limited to 'dwm.c')
| -rw-r--r-- | dwm.c | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/dwm.c b/dwm.c index d171fe2..be11381 100644 --- a/dwm.c +++ b/dwm.c @@ -980,7 +980,7 @@ grabkeys(void) { void initfont(const char *fontstr) { char *def, **missing; - int i, n; + int n; missing = NULL; dc.font.set = XCreateFontSet(dpy, fontstr, &missing, &n, &def); @@ -996,7 +996,7 @@ initfont(const char *fontstr) { dc.font.ascent = dc.font.descent = 0; XExtentsOfFontSet(dc.font.set); n = XFontsOfFontSet(dc.font.set, &xfonts, &font_names); - for(i = 0, dc.font.ascent = 0, dc.font.descent = 0; i < n; i++) { + while(n--) { dc.font.ascent = MAX(dc.font.ascent, (*xfonts)->ascent); dc.font.descent = MAX(dc.font.descent,(*xfonts)->descent); xfonts++; @@ -1014,14 +1014,13 @@ initfont(const char *fontstr) { Bool isprotodel(Client *c) { - int i, n; + int n; Atom *protocols; Bool ret = False; if(XGetWMProtocols(dpy, c->win, &protocols, &n)) { - for(i = 0; !ret && i < n; i++) - if(protocols[i] == wmatom[WMDelete]) - ret = True; + while(!ret && n--) + ret = protocols[n] == wmatom[WMDelete]; XFree(protocols); } return ret; @@ -1029,12 +1028,10 @@ isprotodel(Client *c) { #ifdef XINERAMA static Bool -isuniquegeom(XineramaScreenInfo *unique, size_t len, XineramaScreenInfo *info) { - unsigned int i; - - for(i = 0; i < len; i++) - if(unique[i].x_org == info->x_org && unique[i].y_org == info->y_org - && unique[i].width == info->width && unique[i].height == info->height) +isuniquegeom(XineramaScreenInfo *unique, size_t n, XineramaScreenInfo *info) { + while(n--) + if(unique[n].x_org == info->x_org && unique[n].y_org == info->y_org + && unique[n].width == info->width && unique[n].height == info->height) return False; return True; } |