[hackers] [dwm] separate program-specific c99 bool and X11 || Hiltjo Posthuma
commit 43e82adf0dc37411fdcc6406b3cdf5d572387b9f
Author: Hiltjo Posthuma <hiltjo_AT_codemadness.org>
AuthorDate: Sat Nov 7 14:04:49 2015 +0100
Commit: Hiltjo Posthuma <hiltjo_AT_codemadness.org>
CommitDate: Sun Nov 8 20:35:25 2015 +0100
separate program-specific c99 bool and X11
True, False are X11-specific (int), make sure to use c99 stdbool for
program-specific things.
diff --git a/config.def.h b/config.def.h
index 3fde3cf..76073a5 100644
--- a/config.def.h
+++ b/config.def.h
_AT_@ -13,8 +13,8 @@ static const char selbgcolor[] = "#005577";
static const char selfgcolor[] = "#eeeeee";
static const unsigned int borderpx = 1; /* border pixel of windows */
static const unsigned int snap = 32; /* snap pixel */
-static const Bool showbar = True; /* False means no bar */
-static const Bool topbar = True; /* False means bottom bar */
+static const bool showbar = true; /* false means no bar */
+static const bool topbar = true; /* false means bottom bar */
/* tagging */
static const char *tags[] = { "1", "2", "3", "4", "5", "6", "7", "8", "9" };
_AT_@ -25,14 +25,14 @@ static const Rule rules[] = {
* WM_NAME(STRING) = title
*/
/* class instance title tags mask isfloating monitor */
- { "Gimp", NULL, NULL, 0, True, -1 },
- { "Firefox", NULL, NULL, 1 << 8, False, -1 },
+ { "Gimp", NULL, NULL, 0, true, -1 },
+ { "Firefox", NULL, NULL, 1 << 8, false, -1 },
};
/* layout(s) */
static const float mfact = 0.55; /* factor of master area size [0.05..0.95] */
static const int nmaster = 1; /* number of clients in master area */
-static const Bool resizehints = True; /* True means respect size hints in tiled resizals */
+static const bool resizehints = true; /* true means respect size hints in tiled resizals */
static const Layout layouts[] = {
/* symbol arrange function */
diff --git a/dwm.c b/dwm.c
index 3639165..5d64141 100644
--- a/dwm.c
+++ b/dwm.c
_AT_@ -24,6 +24,7 @@
#include <locale.h>
#include <stdarg.h>
#include <signal.h>
+#include <stdbool.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
_AT_@ -92,7 +93,7 @@ struct Client {
int basew, baseh, incw, inch, maxw, maxh, minw, minh;
int bw, oldbw;
unsigned int tags;
- Bool isfixed, isfloating, isurgent, neverfocus, oldstate, isfullscreen;
+ bool isfixed, isfloating, isurgent, neverfocus, oldstate, isfullscreen;
Client *next;
Client *snext;
Monitor *mon;
_AT_@ -122,8 +123,8 @@ struct Monitor {
unsigned int seltags;
unsigned int sellt;
unsigned int tagset[2];
- Bool showbar;
- Bool topbar;
+ bool showbar;
+ bool topbar;
Client *clients;
Client *sel;
Client *stack;
_AT_@ -137,13 +138,13 @@ typedef struct {
const char *instance;
const char *title;
unsigned int tags;
- Bool isfloating;
+ bool isfloating;
int monitor;
} Rule;
/* function declarations */
static void applyrules(Client *c);
-static Bool applysizehints(Client *c, int *x, int *y, int *w, int *h, Bool interact);
+static bool applysizehints(Client *c, int *x, int *y, int *w, int *h, bool interact);
static void arrange(Monitor *m);
static void arrangemon(Monitor *m);
static void attach(Client *c);
_AT_@ -170,10 +171,10 @@ static void focus(Client *c);
static void focusin(XEvent *e);
static void focusmon(const Arg *arg);
static void focusstack(const Arg *arg);
-static Bool getrootptr(int *x, int *y);
+static bool getrootptr(int *x, int *y);
static long getstate(Window w);
-static Bool gettextprop(Window w, Atom atom, char *text, unsigned int size);
-static void grabbuttons(Client *c, Bool focused);
+static bool gettextprop(Window w, Atom atom, char *text, unsigned int size);
+static void grabbuttons(Client *c, bool focused);
static void grabkeys(void);
static void incnmaster(const Arg *arg);
static void keypress(XEvent *e);
_AT_@ -189,17 +190,17 @@ static void pop(Client *);
static void propertynotify(XEvent *e);
static void quit(const Arg *arg);
static Monitor *recttomon(int x, int y, int w, int h);
-static void resize(Client *c, int x, int y, int w, int h, Bool interact);
+static void resize(Client *c, int x, int y, int w, int h, bool interact);
static void resizeclient(Client *c, int x, int y, int w, int h);
static void resizemouse(const Arg *arg);
static void restack(Monitor *m);
static void run(void);
static void scan(void);
-static Bool sendevent(Client *c, Atom proto);
+static bool sendevent(Client *c, Atom proto);
static void sendmon(Client *c, Monitor *m);
static void setclientstate(Client *c, long state);
static void setfocus(Client *c);
-static void setfullscreen(Client *c, Bool fullscreen);
+static void setfullscreen(Client *c, bool fullscreen);
static void setlayout(const Arg *arg);
static void setmfact(const Arg *arg);
static void setup(void);
_AT_@ -213,10 +214,10 @@ static void togglebar(const Arg *arg);
static void togglefloating(const Arg *arg);
static void toggletag(const Arg *arg);
static void toggleview(const Arg *arg);
-static void unfocus(Client *c, Bool setfocus);
-static void unmanage(Client *c, Bool destroyed);
+static void unfocus(Client *c, bool setfocus);
+static void unmanage(Client *c, bool destroyed);
static void unmapnotify(XEvent *e);
-static Bool updategeom(void);
+static bool updategeom(void);
static void updatebarpos(Monitor *m);
static void updatebars(void);
static void updateclientlist(void);
_AT_@ -259,7 +260,7 @@ static void (*handler[LASTEvent]) (XEvent *) = {
[UnmapNotify] = unmapnotify
};
static Atom wmatom[WMLast], netatom[NetLast];
-static Bool running = True;
+static bool running = true;
static Cur *cursor[CurLast];
static ClrScheme scheme[SchemeLast];
static Display *dpy;
_AT_@ -283,7 +284,8 @@ applyrules(Client *c) {
XClassHint ch = { NULL, NULL };
/* rule matching */
- c->isfloating = c->tags = 0;
+ c->isfloating = false;
+ c->tags = 0;
XGetClassHint(dpy, c->win, &ch);
class = ch.res_class ? ch.res_class : broken;
instance = ch.res_name ? ch.res_name : broken;
_AT_@ -308,9 +310,9 @@ applyrules(Client *c) {
c->tags = c->tags & TAGMASK ? c->tags & TAGMASK : c->mon->tagset[c->mon->seltags];
}
-Bool
-applysizehints(Client *c, int *x, int *y, int *w, int *h, Bool interact) {
- Bool baseismin;
+bool
+applysizehints(Client *c, int *x, int *y, int *w, int *h, bool interact) {
+ bool baseismin;
Monitor *m = c->mon;
/* set minimum possible */
_AT_@ -417,7 +419,7 @@ buttonpress(XEvent *e) {
click = ClkRootWin;
/* focus monitor if necessary */
if((m = wintomon(ev->window)) && m != selmon) {
- unfocus(selmon->sel, True);
+ unfocus(selmon->sel, true);
selmon = m;
focus(NULL);
}
_AT_@ -468,7 +470,7 @@ cleanup(void) {
selmon->lt[selmon->sellt] = &foo;
for(m = mons; m; m = m->next)
while(m->stack)
- unmanage(m->stack, False);
+ unmanage(m->stack, false);
XUngrabKey(dpy, AnyKey, AnyModifier, root);
while(mons)
cleanupmon(mons);
_AT_@ -504,7 +506,7 @@ void
clearurgent(Client *c) {
XWMHints *wmh;
- c->isurgent = False;
+ c->isurgent = false;
if(!(wmh = XGetWMHints(dpy, c->win)))
return;
wmh->flags &= ~XUrgencyHint;
_AT_@ -555,7 +557,7 @@ void
configurenotify(XEvent *e) {
Monitor *m;
XConfigureEvent *ev = &e->xconfigure;
- Bool dirty;
+ bool dirty;
/* TODO: updategeom handling sucks, needs to be simplified */
if(ev->window == root) {
_AT_@ -648,7 +650,7 @@ destroynotify(XEvent *e) {
XDestroyWindowEvent *ev = &e->xdestroywindow;
if((c = wintoclient(ev->window)))
- unmanage(c, True);
+ unmanage(c, true);
}
void
_AT_@ -759,7 +761,7 @@ enternotify(XEvent *e) {
c = wintoclient(ev->window);
m = c ? c->mon : wintomon(ev->window);
if(m != selmon) {
- unfocus(selmon->sel, True);
+ unfocus(selmon->sel, true);
selmon = m;
}
else if(!c || c == selmon->sel)
_AT_@ -782,7 +784,7 @@ focus(Client *c) {
for(c = selmon->stack; c && !ISVISIBLE(c); c = c->snext);
/* was if(selmon->sel) */
if(selmon->sel && selmon->sel != c)
- unfocus(selmon->sel, False);
+ unfocus(selmon->sel, false);
if(c) {
if(c->mon != selmon)
selmon = c->mon;
_AT_@ -790,7 +792,7 @@ focus(Client *c) {
clearurgent(c);
detachstack(c);
attachstack(c);
- grabbuttons(c, True);
+ grabbuttons(c, true);
XSetWindowBorder(dpy, c->win, scheme[SchemeSel].border->pix);
setfocus(c);
}
_AT_@ -818,7 +820,7 @@ focusmon(const Arg *arg) {
return;
if((m = dirtomon(arg->i)) == selmon)
return;
- unfocus(selmon->sel, False); /* s/True/False/ fixes input focus issues
+ unfocus(selmon->sel, false); /* s/true/false/ fixes input focus issues
in gedit and anjuta */
selmon = m;
focus(NULL);
_AT_@ -865,13 +867,13 @@ getatomprop(Client *c, Atom prop) {
return atom;
}
-Bool
+bool
getrootptr(int *x, int *y) {
int di;
unsigned int dui;
Window dummy;
- return XQueryPointer(dpy, root, &dummy, &dummy, x, y, &di, &di, &dui);
+ return XQueryPointer(dpy, root, &dummy, &dummy, x, y, &di, &di, &dui) == True;
}
long
_AT_@ -891,18 +893,18 @@ getstate(Window w) {
return result;
}
-Bool
+bool
gettextprop(Window w, Atom atom, char *text, unsigned int size) {
char **list = NULL;
int n;
XTextProperty name;
if(!text || size == 0)
- return False;
+ return false;
text[0] = '\0';
XGetTextProperty(dpy, w, &name, atom);
if(!name.nitems)
- return False;
+ return false;
if(name.encoding == XA_STRING)
strncpy(text, (char *)name.value, size - 1);
else {
_AT_@ -913,11 +915,11 @@ gettextprop(Window w, Atom atom, char *text, unsigned int size) {
}
text[size - 1] = '\0';
XFree(name.value);
- return True;
+ return true;
}
void
-grabbuttons(Client *c, Bool focused) {
+grabbuttons(Client *c, bool focused) {
updatenumlockmask();
{
unsigned int i, j;
_AT_@ -962,13 +964,13 @@ incnmaster(const Arg *arg) {
}
#ifdef XINERAMA
-static Bool
+static bool
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;
}
#endif /* XINERAMA */
_AT_@ -1044,7 +1046,7 @@ manage(Window w, XWindowAttributes *wa) {
updatesizehints(c);
updatewmhints(c);
XSelectInput(dpy, w, EnterWindowMask|FocusChangeMask|PropertyChangeMask|StructureNotifyMask);
- grabbuttons(c, False);
+ grabbuttons(c, false);
if(!c->isfloating)
c->isfloating = c->oldstate = trans != None || c->isfixed;
if(c->isfloating)
_AT_@ -1056,7 +1058,7 @@ manage(Window w, XWindowAttributes *wa) {
XMoveResizeWindow(dpy, c->win, c->x + 2 * sw, c->y, c->w, c->h); /* some windows require this */
setclientstate(c, NormalState);
if (c->mon == selmon)
- unfocus(selmon->sel, False);
+ unfocus(selmon->sel, false);
c->mon->sel = c;
arrange(c->mon);
XMapWindow(dpy, c->win);
_AT_@ -1096,7 +1098,7 @@ monocle(Monitor *m) {
if(n > 0) /* override layout symbol */
snprintf(m->ltsymbol, sizeof m->ltsymbol, "[%d]", n);
for(c = nexttiled(m->clients); c; c = nexttiled(c->next))
- resize(c, m->wx, m->wy, m->ww - 2 * c->bw, m->wh - 2 * c->bw, False);
+ resize(c, m->wx, m->wy, m->ww - 2 * c->bw, m->wh - 2 * c->bw, false);
}
void
_AT_@ -1108,7 +1110,7 @@ motionnotify(XEvent *e) {
if(ev->window != root)
return;
if((m = recttomon(ev->x_root, ev->y_root, 1, 1)) != mon && mon) {
- unfocus(selmon->sel, True);
+ unfocus(selmon->sel, true);
selmon = m;
focus(NULL);
}
_AT_@ -1165,7 +1167,7 @@ movemouse(const Arg *arg) {
togglefloating(NULL);
}
if(!selmon->lt[selmon->sellt]->arrange || c->isfloating)
- resize(c, nx, ny, c->w, c->h, True);
+ resize(c, nx, ny, c->w, c->h, true);
break;
}
} while(ev.type != ButtonRelease);
_AT_@ -1229,7 +1231,7 @@ propertynotify(XEvent *e) {
void
quit(const Arg *arg) {
- running = False;
+ running = false;
}
Monitor *
_AT_@ -1246,7 +1248,7 @@ recttomon(int x, int y, int w, int h) {
}
void
-resize(Client *c, int x, int y, int w, int h, Bool interact) {
+resize(Client *c, int x, int y, int w, int h, bool interact) {
if(applysizehints(c, &x, &y, &w, &h, interact))
resizeclient(c, x, y, w, h);
}
_AT_@ -1307,7 +1309,7 @@ resizemouse(const Arg *arg) {
togglefloating(NULL);
}
if(!selmon->lt[selmon->sellt]->arrange || c->isfloating)
- resize(c, c->x, c->y, nw, nh, True);
+ resize(c, c->x, c->y, nw, nh, true);
break;
}
} while(ev.type != ButtonRelease);
_AT_@ -1385,7 +1387,7 @@ void
sendmon(Client *c, Monitor *m) {
if(c->mon == m)
return;
- unfocus(c, True);
+ unfocus(c, true);
detach(c);
detachstack(c);
c->mon = m;
_AT_@ -1404,11 +1406,11 @@ setclientstate(Client *c, long state) {
PropModeReplace, (unsigned char *)data, 2);
}
-Bool
+bool
sendevent(Client *c, Atom proto) {
int n;
Atom *protocols;
- Bool exists = False;
+ bool exists = false;
XEvent ev;
if(XGetWMProtocols(dpy, c->win, &protocols, &n)) {
_AT_@ -1440,22 +1442,22 @@ setfocus(Client *c) {
}
void
-setfullscreen(Client *c, Bool fullscreen) {
+setfullscreen(Client *c, bool fullscreen) {
if(fullscreen && !c->isfullscreen) {
XChangeProperty(dpy, c->win, netatom[NetWMState], XA_ATOM, 32,
PropModeReplace, (unsigned char*)&netatom[NetWMFullscreen], 1);
- c->isfullscreen = True;
+ c->isfullscreen = true;
c->oldstate = c->isfloating;
c->oldbw = c->bw;
c->bw = 0;
- c->isfloating = True;
+ c->isfloating = true;
resizeclient(c, c->mon->mx, c->mon->my, c->mon->mw, c->mon->mh);
XRaiseWindow(dpy, c->win);
}
else if(!fullscreen && c->isfullscreen){
XChangeProperty(dpy, c->win, netatom[NetWMState], XA_ATOM, 32,
PropModeReplace, (unsigned char*)0, 0);
- c->isfullscreen = False;
+ c->isfullscreen = false;
c->isfloating = c->oldstate;
c->bw = c->oldbw;
c->x = c->oldx;
_AT_@ -1560,7 +1562,7 @@ showhide(Client *c) {
if(ISVISIBLE(c)) { /* show clients top down */
XMoveWindow(dpy, c->win, c->x, c->y);
if((!c->mon->lt[c->mon->sellt]->arrange || c->isfloating) && !c->isfullscreen)
- resize(c, c->x, c->y, c->w, c->h, False);
+ resize(c, c->x, c->y, c->w, c->h, false);
showhide(c->snext);
}
else { /* hide clients bottom up */
_AT_@ -1623,12 +1625,12 @@ tile(Monitor *m) {
for(i = my = ty = 0, c = nexttiled(m->clients); c; c = nexttiled(c->next), i++)
if(i < m->nmaster) {
h = (m->wh - my) / (MIN(n, m->nmaster) - i);
- resize(c, m->wx, m->wy + my, mw - (2*c->bw), h - (2*c->bw), False);
+ resize(c, m->wx, m->wy + my, mw - (2*c->bw), h - (2*c->bw), false);
my += HEIGHT(c);
}
else {
h = (m->wh - ty) / (n - i);
- resize(c, m->wx + mw, m->wy + ty, m->ww - mw - (2*c->bw), h - (2*c->bw), False);
+ resize(c, m->wx + mw, m->wy + ty, m->ww - mw - (2*c->bw), h - (2*c->bw), false);
ty += HEIGHT(c);
}
}
_AT_@ -1650,7 +1652,7 @@ togglefloating(const Arg *arg) {
selmon->sel->isfloating = !selmon->sel->isfloating || selmon->sel->isfixed;
if(selmon->sel->isfloating)
resize(selmon->sel, selmon->sel->x, selmon->sel->y,
- selmon->sel->w, selmon->sel->h, False);
+ selmon->sel->w, selmon->sel->h, false);
arrange(selmon);
}
_AT_@ -1680,10 +1682,10 @@ toggleview(const Arg *arg) {
}
void
-unfocus(Client *c, Bool setfocus) {
+unfocus(Client *c, bool setfocus) {
if(!c)
return;
- grabbuttons(c, False);
+ grabbuttons(c, false);
XSetWindowBorder(dpy, c->win, scheme[SchemeNorm].border->pix);
if(setfocus) {
XSetInputFocus(dpy, root, RevertToPointerRoot, CurrentTime);
_AT_@ -1692,7 +1694,7 @@ unfocus(Client *c, Bool setfocus) {
}
void
-unmanage(Client *c, Bool destroyed) {
+unmanage(Client *c, bool destroyed) {
Monitor *m = c->mon;
XWindowChanges wc;
_AT_@ -1725,7 +1727,7 @@ unmapnotify(XEvent *e) {
if(ev->send_event)
setclientstate(c, WithdrawnState);
else
- unmanage(c, False);
+ unmanage(c, false);
}
}
_AT_@ -1774,9 +1776,9 @@ updateclientlist() {
(unsigned char *) &(c->win), 1);
}
-Bool
+bool
updategeom(void) {
- Bool dirty = False;
+ bool dirty = false;
#ifdef XINERAMA
if(XineramaIsActive(dpy)) {
_AT_@ -1807,7 +1809,7 @@ updategeom(void) {
|| (unique[i].x_org != m->mx || unique[i].y_org != m->my
|| unique[i].width != m->mw || unique[i].height != m->mh))
{
- dirty = True;
+ dirty = true;
m->num = i;
m->mx = m->wx = unique[i].x_org;
m->my = m->wy = unique[i].y_org;
_AT_@ -1820,7 +1822,7 @@ updategeom(void) {
for(i = nn; i < n; i++) {
for(m = mons; m && m->next; m = m->next);
while(m->clients) {
- dirty = True;
+ dirty = true;
c = m->clients;
m->clients = c->next;
detachstack(c);
_AT_@ -1842,7 +1844,7 @@ updategeom(void) {
if(!mons)
mons = createmon();
if(mons->mw != sw || mons->mh != sh) {
- dirty = True;
+ dirty = true;
mons->mw = mons->ww = sw;
mons->mh = mons->wh = sh;
updatebarpos(mons);
_AT_@ -1941,9 +1943,9 @@ updatewindowtype(Client *c) {
Atom wtype = getatomprop(c, netatom[NetWMWindowType]);
if(state == netatom[NetWMFullscreen])
- setfullscreen(c, True);
+ setfullscreen(c, true);
if(wtype == netatom[NetWMWindowTypeDialog])
- c->isfloating = True;
+ c->isfloating = true;
}
void
_AT_@ -1956,11 +1958,11 @@ updatewmhints(Client *c) {
XSetWMHints(dpy, c->win, wmh);
}
else
- c->isurgent = (wmh->flags & XUrgencyHint) ? True : False;
+ c->isurgent = (wmh->flags & XUrgencyHint) ? true : false;
if(wmh->flags & InputHint)
c->neverfocus = !wmh->input;
else
- c->neverfocus = False;
+ c->neverfocus = false;
XFree(wmh);
}
}
Received on Sun Nov 08 2015 - 21:11:45 CET
This archive was generated by hypermail 2.3.0
: Sun Nov 08 2015 - 21:12:21 CET