Got questions, comments, patches, etc.?
Contact Jason Woofenden
gitweb
/
dwm.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
|
inline
| side by side (from parent 1:
703c4dd
)
applied anonymous patch, I don't think the reversed array access has semantic side...
author
Anselm R Garbe
<anselm@garbe.us>
Fri, 7 Jan 2011 16:05:22 +0000
(16:05 +0000)
committer
Anselm R Garbe
<anselm@garbe.us>
Fri, 7 Jan 2011 16:05:22 +0000
(16:05 +0000)
dwm.c
patch
|
blob
|
history
diff --git
a/dwm.c
b/dwm.c
index
d171fe2
..
be11381
100644
(file)
--- a/
dwm.c
+++ b/
dwm.c
@@
-980,7
+980,7
@@
grabkeys(void) {
void
initfont(const char *fontstr) {
char *def, **missing;
void
initfont(const char *fontstr) {
char *def, **missing;
- int i, n;
+ int n;
missing = NULL;
dc.font.set = XCreateFontSet(dpy, fontstr, &missing, &n, &def);
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);
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++;
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) {
Bool
isprotodel(Client *c) {
- int i, n;
+ int n;
Atom *protocols;
Bool ret = False;
if(XGetWMProtocols(dpy, c->win, &protocols, &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;
XFree(protocols);
}
return ret;
@@
-1029,12
+1028,10
@@
isprotodel(Client *c) {
#ifdef XINERAMA
static Bool
#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;
}
return False;
return True;
}