[hackers] [tabbed] Style cleanup || Quentin Rameau

From: <git_AT_suckless.org>
Date: Sun, 3 Jan 2016 14:11:36 +0100 (CET)

commit 735bed582460a2150702a74f24b6822397e41fc0
Author: Quentin Rameau <quinq_AT_fifth.space>
AuthorDate: Sun Jan 3 13:00:57 2016 +0100
Commit: Christoph Lohmann <20h_AT_r-36.net>
CommitDate: Sun Jan 3 14:10:38 2016 +0100

    Style cleanup
    
    Signed-off-by: Christoph Lohmann <20h_AT_r-36.net>

diff --git a/config.def.h b/config.def.h
index 41f39c3..587ce73 100644
--- a/config.def.h
+++ b/config.def.h
_AT_@ -23,42 +23,43 @@ static int newposition = 0;
 static Bool npisrelative = False;
 
 #define SETPROP(p) { \
- .v = (char *[]){ "/bin/sh", "-c", \
- "prop=\"`xwininfo -children -id $1 | grep '^ 0x' | sed -e's_AT_^ *\\(0x[0-9a-f]*\\) \"\\([^\"]*\\)\".*@\\1 \\2@' | xargs -0 printf %b | dmenu -l 10`\" &&" \
- "xprop -id $1 -f $0 8s -set $0 \"$prop\"", \
- p, winid, NULL \
- } \
+ .v = (char *[]){ "/bin/sh", "-c", \
+ "prop=\"`xwininfo -children -id $1 | grep '^ 0x' |" \
+ "sed -e's_AT_^ *\\(0x[0-9a-f]*\\) \"\\([^\"]*\\)\".*@\\1 \\2@' |" \
+ "xargs -0 printf %b | dmenu -l 10`\" &&" \
+ "xprop -id $1 -f $0 8s -set $0 \"$prop\"", \
+ p, winid, NULL \
+ } \
 }
 
 #define MODKEY ControlMask
-static Key keys[] = { \
- /* modifier key function argument */
- { MODKEY|ShiftMask, XK_Return, focusonce, { 0 } },
- { MODKEY|ShiftMask, XK_Return, spawn, { 0 } },
- { MODKEY, XK_t, spawn, SETPROP("_TABBED_SELECT_TAB") },
+static Key keys[] = {
+ /* modifier key function argument */
+ { MODKEY|ShiftMask, XK_Return, focusonce, { 0 } },
+ { MODKEY|ShiftMask, XK_Return, spawn, { 0 } },
+ { MODKEY, XK_t, spawn, SETPROP("_TABBED_SELECT_TAB") },
 
- { MODKEY|ShiftMask, XK_l, rotate, { .i = +1 } },
- { MODKEY|ShiftMask, XK_h, rotate, { .i = -1 } },
- { MODKEY|ShiftMask, XK_j, movetab, { .i = -1 } },
- { MODKEY|ShiftMask, XK_k, movetab, { .i = +1 } },
- { MODKEY, XK_Tab, rotate, { .i = 0 } },
+ { MODKEY|ShiftMask, XK_l, rotate, { .i = +1 } },
+ { MODKEY|ShiftMask, XK_h, rotate, { .i = -1 } },
+ { MODKEY|ShiftMask, XK_j, movetab, { .i = -1 } },
+ { MODKEY|ShiftMask, XK_k, movetab, { .i = +1 } },
+ { MODKEY, XK_Tab, rotate, { .i = 0 } },
 
- { MODKEY, XK_1, move, { .i = 0 } },
- { MODKEY, XK_2, move, { .i = 1 } },
- { MODKEY, XK_3, move, { .i = 2 } },
- { MODKEY, XK_4, move, { .i = 3 } },
- { MODKEY, XK_5, move, { .i = 4 } },
- { MODKEY, XK_6, move, { .i = 5 } },
- { MODKEY, XK_7, move, { .i = 6 } },
- { MODKEY, XK_8, move, { .i = 7 } },
- { MODKEY, XK_9, move, { .i = 8 } },
- { MODKEY, XK_0, move, { .i = 9 } },
+ { MODKEY, XK_1, move, { .i = 0 } },
+ { MODKEY, XK_2, move, { .i = 1 } },
+ { MODKEY, XK_3, move, { .i = 2 } },
+ { MODKEY, XK_4, move, { .i = 3 } },
+ { MODKEY, XK_5, move, { .i = 4 } },
+ { MODKEY, XK_6, move, { .i = 5 } },
+ { MODKEY, XK_7, move, { .i = 6 } },
+ { MODKEY, XK_8, move, { .i = 7 } },
+ { MODKEY, XK_9, move, { .i = 8 } },
+ { MODKEY, XK_0, move, { .i = 9 } },
 
- { MODKEY, XK_q, killclient, { 0 } },
+ { MODKEY, XK_q, killclient, { 0 } },
 
- { MODKEY, XK_u, focusurgent, { .v = NULL } },
- { MODKEY|ShiftMask, XK_u, toggle, { .v = (void*) &urgentswitch } },
+ { MODKEY, XK_u, focusurgent, { 0 } },
+ { MODKEY|ShiftMask, XK_u, toggle, { .v = (void*) &urgentswitch } },
 
- { 0, XK_F11, fullscreen, { 0 } },
+ { 0, XK_F11, fullscreen, { 0 } },
 };
-
diff --git a/config.mk b/config.mk
index ad0dfb2..441b2de 100644
--- a/config.mk
+++ b/config.mk
_AT_@ -22,4 +22,3 @@ LDFLAGS = -s ${LIBS}
 
 # compiler and linker
 CC = cc
-
diff --git a/tabbed.c b/tabbed.c
index ea43d34..89c2f8a 100644
--- a/tabbed.c
+++ b/tabbed.c
_AT_@ -4,12 +4,12 @@
 
 #include <sys/wait.h>
 #include <locale.h>
-#include <stdarg.h>
-#include <unistd.h>
 #include <signal.h>
+#include <stdarg.h>
 #include <stdio.h>
-#include <string.h>
 #include <stdlib.h>
+#include <string.h>
+#include <unistd.h>
 #include <X11/Xatom.h>
 #include <X11/Xlib.h>
 #include <X11/Xproto.h>
_AT_@ -40,15 +40,15 @@
 #define XEMBED_FOCUS_LAST 2
 
 /* Macros */
-#define MAX(a, b) ((a) > (b) ? (a) : (b))
-#define MIN(a, b) ((a) < (b) ? (a) : (b))
-#define LENGTH(x) (sizeof((x)) / sizeof(*(x)))
-#define CLEANMASK(mask) (mask & ~(numlockmask|LockMask))
-#define TEXTW(x) (textnw(x, strlen(x)) + dc.font.height)
+#define MAX(a, b) ((a) > (b) ? (a) : (b))
+#define MIN(a, b) ((a) < (b) ? (a) : (b))
+#define LENGTH(x) (sizeof((x)) / sizeof(*(x)))
+#define CLEANMASK(mask) (mask & ~(numlockmask | LockMask))
+#define TEXTW(x) (textnw(x, strlen(x)) + dc.font.height)
 
-enum { ColFG, ColBG, ColLast }; /* color */
+enum { ColFG, ColBG, ColLast }; /* color */
 enum { WMProtocols, WMDelete, WMName, WMState, WMFullscreen,
- XEmbed, WMSelectTab, WMLast }; /* default atoms */
+ XEmbed, WMSelectTab, WMLast }; /* default atoms */
 
 typedef union {
         int i;
_AT_@ -78,7 +78,7 @@ typedef struct {
         } font;
 } DC; /* draw context */
 
-typedef struct Client {
+typedef struct {
         char name[256];
         Window win;
         int tabx;
_AT_@ -93,7 +93,6 @@ static void clientmessage(const XEvent *e);
 static void configurenotify(const XEvent *e);
 static void configurerequest(const XEvent *e);
 static void createnotify(const XEvent *e);
-static void unmapnotify(const XEvent *e);
 static void destroynotify(const XEvent *e);
 static void die(const char *errstr, ...);
 static void drawbar(void);
_AT_@ -104,9 +103,9 @@ static void expose(const XEvent *e);
 static void focus(int c);
 static void focusin(const XEvent *e);
 static void focusonce(const Arg *arg);
-static void focusurgent(const Arg *);
+static void focusurgent(const Arg *arg);
 static void fullscreen(const Arg *arg);
-static char* getatom(int a);
+static char *getatom(int a);
 static int getclient(Window w);
 static unsigned long getcolor(const char *colstr);
 static int getfirsttab(void);
_AT_@ -124,13 +123,14 @@ static void resize(int c, int w, int h);
 static void rotate(const Arg *arg);
 static void run(void);
 static void sendxembed(int c, long msg, long detail, long d1, long d2);
-static void setup(void);
 static void setcmd(int argc, char *argv[], int);
+static void setup(void);
 static void sigchld(int unused);
 static void spawn(const Arg *arg);
 static int textnw(const char *text, unsigned int len);
 static void toggle(const Arg *arg);
 static void unmanage(int c);
+static void unmapnotify(const XEvent *e);
 static void updatenumlockmask(void);
 static void updatetitle(int c);
 static int xerror(Display *dpy, XErrorEvent *ee);
_AT_@ -153,21 +153,21 @@ static void (*handler[LASTEvent]) (const XEvent *) = {
         [PropertyNotify] = propertynotify,
 };
 static int bh, wx, wy, ww, wh;
-static unsigned int numlockmask = 0;
+static unsigned int numlockmask;
 static Bool running = True, nextfocus, doinitspawn = True,
- fillagain = False, closelastclient = False;
+ fillagain = False, closelastclient = False;
 static Display *dpy;
 static DC dc;
 static Atom wmatom[WMLast];
 static Window root, win;
-static Client **clients = NULL;
-static int nclients = 0, sel = -1, lastsel = -1;
+static Client **clients;
+static int nclients, sel = -1, lastsel = -1;
 static int (*xerrorxlib)(Display *, XErrorEvent *);
-static int cmd_append_pos = 0;
+static int cmd_append_pos;
 static char winid[64];
-static char **cmd = NULL;
+static char **cmd;
 static char *wmname = "tabbed";
-static const char *geometry = NULL;
+static const char *geometry;
 
 char *argv0;
 
_AT_@ -175,21 +175,21 @@ char *argv0;
 #include "config.h"
 
 void
-buttonpress(const XEvent *e) {
+buttonpress(const XEvent *e)
+{
         const XButtonPressedEvent *ev = &e->xbutton;
- int i;
- int fc;
+ int i, fc;
         Arg arg;
 
- if(ev->y < 0 || ev->y > bh)
+ if (ev->y < 0 || ev->y > bh)
                 return;
 
- if(((fc = getfirsttab()) > 0 && ev->x < TEXTW(before)) || ev->x < 0)
+ if (((fc = getfirsttab()) > 0 && ev->x < TEXTW(before)) || ev->x < 0)
                 return;
 
- for(i = fc; i < nclients; i++) {
- if(clients[i]->tabx > ev->x) {
- switch(ev->button) {
+ for (i = fc; i < nclients; i++) {
+ if (clients[i]->tabx > ev->x) {
+ switch (ev->button) {
                         case Button1:
                                 focus(i);
                                 break;
_AT_@ -197,7 +197,7 @@ buttonpress(const XEvent *e) {
                                 focus(i);
                                 killclient(NULL);
                                 break;
- case Button4:
+ case Button4: /* FALLTHROUGH */
                         case Button5:
                                 arg.i = ev->button == Button4 ? -1 : 1;
                                 rotate(&arg);
_AT_@ -209,10 +209,11 @@ buttonpress(const XEvent *e) {
 }
 
 void
-cleanup(void) {
+cleanup(void)
+{
         int i;
 
- for(i = 0; i < nclients; i++) {
+ for (i = 0; i < nclients; i++) {
                 focus(i);
                 killclient(NULL);
                 killclient(NULL);
_AT_@ -222,11 +223,10 @@ cleanup(void) {
         free(clients);
         clients = NULL;
 
- if(dc.font.set) {
+ if (dc.font.set)
                 XFreeFontSet(dpy, dc.font.set);
- } else {
+ else
                 XFreeFont(dpy, dc.font.xfont);
- }
 
         XFreePixmap(dpy, dc.drawable);
         XFreeGC(dpy, dc.gc);
_AT_@ -236,38 +236,40 @@ cleanup(void) {
 }
 
 void
-clientmessage(const XEvent *e) {
+clientmessage(const XEvent *e)
+{
         const XClientMessageEvent *ev = &e->xclient;
 
- if(ev->message_type == wmatom[WMProtocols]
- && ev->data.l[0] == wmatom[WMDelete]) {
+ if (ev->message_type == wmatom[WMProtocols] &&
+ ev->data.l[0] == wmatom[WMDelete])
                 running = False;
- }
 }
 
 void
-configurenotify(const XEvent *e) {
+configurenotify(const XEvent *e)
+{
         const XConfigureEvent *ev = &e->xconfigure;
 
- if(ev->window == win && (ev->width != ww || ev->height != wh)) {
+ if (ev->window == win && (ev->width != ww || ev->height != wh)) {
                 ww = ev->width;
                 wh = ev->height;
                 XFreePixmap(dpy, dc.drawable);
                 dc.drawable = XCreatePixmap(dpy, root, ww, wh,
- DefaultDepth(dpy, screen));
- if(sel > -1)
+ DefaultDepth(dpy, screen));
+ if (sel > -1)
                         resize(sel, ww, wh - bh);
                 XSync(dpy, False);
         }
 }
 
 void
-configurerequest(const XEvent *e) {
+configurerequest(const XEvent *e)
+{
         const XConfigureRequestEvent *ev = &e->xconfigurerequest;
         XWindowChanges wc;
         int c;
 
- if((c = getclient(ev->window)) > -1) {
+ if ((c = getclient(ev->window)) > -1) {
                 wc.x = 0;
                 wc.y = bh;
                 wc.width = ww;
_AT_@ -280,33 +282,27 @@ configurerequest(const XEvent *e) {
 }
 
 void
-createnotify(const XEvent *e) {
+createnotify(const XEvent *e)
+{
         const XCreateWindowEvent *ev = &e->xcreatewindow;
 
- if(ev->window != win && getclient(ev->window) < 0)
+ if (ev->window != win && getclient(ev->window) < 0)
                 manage(ev->window);
 }
 
 void
-unmapnotify(const XEvent *e) {
- const XUnmapEvent *ev = &e->xunmap;
- int c;
-
- if((c = getclient(ev->window)) > -1)
- unmanage(c);
-}
-
-void
-destroynotify(const XEvent *e) {
+destroynotify(const XEvent *e)
+{
         const XDestroyWindowEvent *ev = &e->xdestroywindow;
         int c;
 
- if((c = getclient(ev->window)) > -1)
+ if ((c = getclient(ev->window)) > -1)
                 unmanage(c);
 }
 
 void
-die(const char *errstr, ...) {
+die(const char *errstr, ...)
+{
         va_list ap;
 
         va_start(ap, errstr);
_AT_@ -316,12 +312,13 @@ die(const char *errstr, ...) {
 }
 
 void
-drawbar(void) {
+drawbar(void)
+{
         unsigned long *col;
         int c, cc, fc, width;
         char *name = NULL;
 
- if(nclients == 0) {
+ if (nclients == 0) {
                 dc.x = 0;
                 dc.w = ww;
                 XFetchName(dpy, win, &name);
_AT_@ -334,10 +331,10 @@ drawbar(void) {
 
         width = ww;
         cc = ww / tabwidth;
- if(nclients > cc)
+ if (nclients > cc)
                 cc = (ww - TEXTW(before) - TEXTW(after)) / tabwidth;
 
- if((fc = getfirsttab()) + cc < nclients) {
+ if ((fc = getfirsttab()) + cc < nclients) {
                 dc.w = TEXTW(after);
                 dc.x = width - dc.w;
                 drawtext(after, dc.sel);
_AT_@ -345,7 +342,7 @@ drawbar(void) {
         }
         dc.x = 0;
 
- if(fc > 0) {
+ if (fc > 0) {
                 dc.w = TEXTW(before);
                 drawtext(before, dc.sel);
                 dc.x += dc.w;
_AT_@ -353,9 +350,9 @@ drawbar(void) {
         }
 
         cc = MIN(cc, nclients);
- for(c = fc; c < fc + cc; c++) {
+ for (c = fc; c < fc + cc; c++) {
                 dc.w = width / cc;
- if(c == sel) {
+ if (c == sel) {
                         col = dc.sel;
                         dc.w += width % cc;
                 } else {
_AT_@ -370,14 +367,15 @@ drawbar(void) {
 }
 
 void
-drawtext(const char *text, unsigned long col[ColLast]) {
+drawtext(const char *text, unsigned long col[ColLast])
+{
         int i, x, y, h, len, olen;
         char buf[256];
         XRectangle r = { dc.x, dc.y, dc.w, dc.h };
 
         XSetForeground(dpy, dc.gc, col[ColBG]);
         XFillRectangles(dpy, dc.drawable, dc.gc, &r, 1);
- if(!text)
+ if (!text)
                 return;
 
         olen = strlen(text);
_AT_@ -386,59 +384,62 @@ drawtext(const char *text, unsigned long col[ColLast]) {
         x = dc.x + (h / 2);
 
         /* shorten text if necessary */
- for(len = MIN(olen, sizeof(buf));
- len && textnw(text, len) > dc.w - h; len--);
- if(!len)
+ for (len = MIN(olen, sizeof(buf));
+ len && textnw(text, len) > dc.w - h; len--);
+
+ if (!len)
                 return;
 
         memcpy(buf, text, len);
- if(len < olen) {
+ if (len < olen)
                 for(i = len; i && i > len - 3; buf[--i] = '.');
- }
 
         XSetForeground(dpy, dc.gc, col[ColFG]);
- if(dc.font.set) {
+ if (dc.font.set)
                 XmbDrawString(dpy, dc.drawable, dc.font.set,
- dc.gc, x, y, buf, len);
- } else {
+ dc.gc, x, y, buf, len);
+ else
                 XDrawString(dpy, dc.drawable, dc.gc, x, y, buf, len);
- }
 }
 
 void *
-ecalloc(size_t n, size_t size) {
+ecalloc(size_t n, size_t size)
+{
         void *p;
 
- if(!(p = calloc(n, size)))
+ if (!(p = calloc(n, size)))
                 die("tabbed: cannot calloc\n");
         return p;
 }
 
 void *
-erealloc(void *o, size_t size) {
+erealloc(void *o, size_t size)
+{
         void *p;
 
- if(!(p = realloc(o, size)))
+ if (!(p = realloc(o, size)))
                 die("tabbed: cannot realloc\n");
         return p;
 }
 
 void
-expose(const XEvent *e) {
+expose(const XEvent *e)
+{
         const XExposeEvent *ev = &e->xexpose;
 
- if(ev->count == 0 && win == ev->window)
+ if (ev->count == 0 && win == ev->window)
                 drawbar();
 }
 
 void
-focus(int c) {
+focus(int c)
+{
         char buf[BUFSIZ] = "tabbed-"VERSION" ::";
         size_t i, n;
         XWMHints* wmh;
 
         /* If c, sel and clients are -1, raise tabbed-win itself */
- if(nclients == 0) {
+ if (nclients == 0) {
                 cmd[cmd_append_pos] = NULL;
                 for(i = 0, n = strlen(buf); cmd[i] && n < sizeof(buf); i++)
                         n += snprintf(&buf[n], sizeof(buf) - n, " %s", cmd[i]);
_AT_@ -449,7 +450,7 @@ focus(int c) {
                 return;
         }
 
- if(c < 0 || c >= nclients)
+ if (c < 0 || c >= nclients)
                 return;
 
         resize(c, ww, wh - bh);
_AT_@ -459,12 +460,12 @@ focus(int c) {
         sendxembed(c, XEMBED_WINDOW_ACTIVATE, 0, 0, 0);
         xsettitle(win, clients[c]->name);
 
- if(sel != c) {
+ if (sel != c) {
                 lastsel = sel;
                 sel = c;
         }
 
- if(clients[c]->urgent && (wmh = XGetWMHints(dpy, clients[c]->win))) {
+ if (clients[c]->urgent && (wmh = XGetWMHints(dpy, clients[c]->win))) {
                 wmh->flags &= ~XUrgencyHint;
                 XSetWMHints(dpy, clients[c]->win, wmh);
                 clients[c]->urgent = False;
_AT_@ -476,28 +477,32 @@ focus(int c) {
 }
 
 void
-focusin(const XEvent *e) {
+focusin(const XEvent *e)
+{
         const XFocusChangeEvent *ev = &e->xfocus;
         int dummy;
         Window focused;
 
- if(ev->mode != NotifyUngrab) {
+ if (ev->mode != NotifyUngrab) {
                 XGetInputFocus(dpy, &focused, &dummy);
- if(focused == win)
+ if (focused == win)
                         focus(sel);
         }
 }
 
 void
-focusonce(const Arg *arg) {
+focusonce(const Arg *arg)
+{
         nextfocus = True;
 }
 
 void
-focusurgent(const Arg *args) {
+focusurgent(const Arg *arg)
+{
         int c;
- for(c = (sel+1)%nclients; c != sel; c = (c+1)%nclients) {
- if(clients[c]->urgent) {
+
+ for (c = (sel + 1) % nclients; c != sel; c = (c + 1) % nclients) {
+ if (clients[c]->urgent) {
                         focus(c);
                         return;
                 }
_AT_@ -505,7 +510,8 @@ focusurgent(const Arg *args) {
 }
 
 void
-fullscreen(const Arg *arg) {
+fullscreen(const Arg *arg)
+{
         XEvent e;
 
         e.type = ClientMessage;
_AT_@ -519,7 +525,8 @@ fullscreen(const Arg *arg) {
 }
 
 char *
-getatom(int a) {
+getatom(int a)
+{
         static char buf[BUFSIZ];
         Atom adummy;
         int idummy;
_AT_@ -527,23 +534,23 @@ getatom(int a) {
         unsigned char *p = NULL;
 
         XGetWindowProperty(dpy, win, wmatom[a], 0L, BUFSIZ, False, XA_STRING,
- &adummy, &idummy, &ldummy, &ldummy, &p);
- if(p) {
+ &adummy, &idummy, &ldummy, &ldummy, &p);
+ if (p)
                 strncpy(buf, (char *)p, LENGTH(buf)-1);
- } else {
+ else
                 buf[0] = '\0';
- }
         XFree(p);
 
         return buf;
 }
 
 int
-getclient(Window w) {
+getclient(Window w)
+{
         int i;
 
- for(i = 0; i < nclients; i++) {
- if(clients[i]->win == w)
+ for (i = 0; i < nclients; i++) {
+ if (clients[i]->win == w)
                         return i;
         }
 
_AT_@ -551,53 +558,56 @@ getclient(Window w) {
 }
 
 unsigned long
-getcolor(const char *colstr) {
+getcolor(const char *colstr)
+{
         Colormap cmap = DefaultColormap(dpy, screen);
         XColor color;
 
- if(!XAllocNamedColor(dpy, cmap, colstr, &color, &color))
+ if (!XAllocNamedColor(dpy, cmap, colstr, &color, &color))
                 die("tabbed: cannot allocate color '%s'\n", colstr);
 
         return color.pixel;
 }
 
 int
-getfirsttab(void) {
+getfirsttab(void)
+{
         int cc, ret;
 
- if(sel < 0)
+ if (sel < 0)
                 return 0;
 
         cc = ww / tabwidth;
- if(nclients > cc)
+ if (nclients > cc)
                 cc = (ww - TEXTW(before) - TEXTW(after)) / tabwidth;
 
         ret = sel - cc / 2 + (cc + 1) % 2;
- return ret < 0 ? 0 : (ret + cc > nclients ? MAX(0, nclients - cc) : ret);
+ return ret < 0 ? 0 :
+ ret + cc > nclients ? MAX(0, nclients - cc) :
+ ret;
 }
 
 Bool
-gettextprop(Window w, Atom atom, char *text, unsigned int size) {
+gettextprop(Window w, Atom atom, char *text, unsigned int size)
+{
         char **list = NULL;
         int n;
         XTextProperty name;
 
- if(!text || size == 0)
+ if (!text || size == 0)
                 return False;
 
         text[0] = '\0';
         XGetTextProperty(dpy, w, &name, atom);
- if(!name.nitems)
+ if (!name.nitems)
                 return False;
 
- if(name.encoding == XA_STRING) {
+ if (name.encoding == XA_STRING) {
                 strncpy(text, (char *)name.value, size - 1);
- } else {
- if(XmbTextPropertyToTextList(dpy, &name, &list, &n) >= Success
- && n > 0 && *list) {
- strncpy(text, *list, size - 1);
- XFreeStringList(list);
- }
+ } else if (XmbTextPropertyToTextList(dpy, &name, &list, &n) >= Success
+ && n > 0 && *list) {
+ strncpy(text, *list, size - 1);
+ XFreeStringList(list);
         }
         text[size - 1] = '\0';
         XFree(name.value);
_AT_@ -606,38 +616,39 @@ gettextprop(Window w, Atom atom, char *text, unsigned int size) {
 }
 
 void
-initfont(const char *fontstr) {
+initfont(const char *fontstr)
+{
         char *def, **missing, **font_names;
- int i, n;
         XFontStruct **xfonts;
+ int i, n;
 
         missing = NULL;
- if(dc.font.set)
+ if (dc.font.set)
                 XFreeFontSet(dpy, dc.font.set);
 
         dc.font.set = XCreateFontSet(dpy, fontstr, &missing, &n, &def);
- if(missing) {
- while(n--)
+ if (missing) {
+ while (n--)
                         fprintf(stderr, "tabbed: missing fontset: %s\n", missing[n]);
                 XFreeStringList(missing);
         }
 
- if(dc.font.set) {
+ if (dc.font.set) {
                 dc.font.ascent = dc.font.descent = 0;
                 n = XFontsOfFontSet(dc.font.set, &xfonts, &font_names);
- for(i = 0, dc.font.ascent = 0, dc.font.descent = 0; i < n; i++) {
+ for (i = 0, dc.font.ascent = 0, dc.font.descent = 0;
+ i < n; i++) {
                         dc.font.ascent = MAX(dc.font.ascent, (*xfonts)->ascent);
                         dc.font.descent = MAX(dc.font.descent,(*xfonts)->descent);
                         xfonts++;
                 }
         } else {
- if(dc.font.xfont)
+ if (dc.font.xfont)
                         XFreeFont(dpy, dc.font.xfont);
                 dc.font.xfont = NULL;
- if(!(dc.font.xfont = XLoadQueryFont(dpy, fontstr))
- && !(dc.font.xfont = XLoadQueryFont(dpy, "fixed"))) {
+ if (!(dc.font.xfont = XLoadQueryFont(dpy, fontstr)) &&
+ !(dc.font.xfont = XLoadQueryFont(dpy, "fixed")))
                         die("tabbed: cannot load font: '%s'\n", fontstr);
- }
 
                 dc.font.ascent = dc.font.xfont->ascent;
                 dc.font.descent = dc.font.xfont->descent;
_AT_@ -646,14 +657,15 @@ initfont(const char *fontstr) {
 }
 
 Bool
-isprotodel(int c) {
+isprotodel(int c)
+{
         int i, n;
         Atom *protocols;
         Bool ret = False;
 
- if(XGetWMProtocols(dpy, clients[c]->win, &protocols, &n)) {
- for(i = 0; !ret && i < n; i++) {
- if(protocols[i] == wmatom[WMDelete])
+ if (XGetWMProtocols(dpy, clients[c]->win, &protocols, &n)) {
+ for (i = 0; !ret && i < n; i++) {
+ if (protocols[i] == wmatom[WMDelete])
                                 ret = True;
                 }
                 XFree(protocols);
_AT_@ -663,29 +675,30 @@ isprotodel(int c) {
 }
 
 void
-keypress(const XEvent *e) {
+keypress(const XEvent *e)
+{
         const XKeyEvent *ev = &e->xkey;
         unsigned int i;
         KeySym keysym;
 
         keysym = XkbKeycodeToKeysym(dpy, (KeyCode)ev->keycode, 0, 0);
- for(i = 0; i < LENGTH(keys); i++) {
- if(keysym == keys[i].keysym
- && CLEANMASK(keys[i].mod) == CLEANMASK(ev->state)
- && keys[i].func) {
+ for (i = 0; i < LENGTH(keys); i++) {
+ if (keysym == keys[i].keysym &&
+ CLEANMASK(keys[i].mod) == CLEANMASK(ev->state) &&
+ keys[i].func)
                         keys[i].func(&(keys[i].arg));
- }
         }
 }
 
 void
-killclient(const Arg *arg) {
+killclient(const Arg *arg)
+{
         XEvent ev;
 
- if(sel < 0)
+ if (sel < 0)
                 return;
 
- if(isprotodel(sel) && !clients[sel]->closed) {
+ if (isprotodel(sel) && !clients[sel]->closed) {
                 ev.type = ClientMessage;
                 ev.xclient.window = clients[sel]->win;
                 ev.xclient.message_type = wmatom[WMProtocols];
_AT_@ -700,29 +713,29 @@ killclient(const Arg *arg) {
 }
 
 void
-manage(Window w) {
+manage(Window w)
+{
         updatenumlockmask();
         {
                 int i, j, nextpos;
                 unsigned int modifiers[] = { 0, LockMask, numlockmask,
- numlockmask|LockMask };
+ numlockmask | LockMask };
                 KeyCode code;
                 Client *c;
                 XEvent e;
 
                 XWithdrawWindow(dpy, w, 0);
                 XReparentWindow(dpy, w, win, 0, bh);
- XSelectInput(dpy, w, PropertyChangeMask
- |StructureNotifyMask|EnterWindowMask);
+ XSelectInput(dpy, w, PropertyChangeMask |
+ StructureNotifyMask | EnterWindowMask);
                 XSync(dpy, False);
 
- for(i = 0; i < LENGTH(keys); i++) {
- if((code = XKeysymToKeycode(dpy, keys[i].keysym))) {
- for(j = 0; j < LENGTH(modifiers); j++) {
- XGrabKey(dpy, code, keys[i].mod
- | modifiers[j], w,
- True, GrabModeAsync,
- GrabModeAsync);
+ for (i = 0; i < LENGTH(keys); i++) {
+ if ((code = XKeysymToKeycode(dpy, keys[i].keysym))) {
+ for (j = 0; j < LENGTH(modifiers); j++) {
+ XGrabKey(dpy, code, keys[i].mod |
+ modifiers[j], w, True,
+ GrabModeAsync, GrabModeAsync);
                                 }
                         }
                 }
_AT_@ -736,22 +749,20 @@ manage(Window w) {
                 if(npisrelative) {
                         nextpos = sel + newposition;
                 } else {
- if(newposition < 0) {
+ if (newposition < 0)
                                 nextpos = nclients - newposition;
- } else {
+ else
                                 nextpos = newposition;
- }
                 }
- if(nextpos >= nclients)
+ if (nextpos >= nclients)
                         nextpos = nclients - 1;
- if(nextpos < 0)
+ if (nextpos < 0)
                         nextpos = 0;
 
- if(nclients > 1 && nextpos < nclients - 1) {
+ if (nclients > 1 && nextpos < nclients - 1)
                         memmove(&clients[nextpos + 1], &clients[nextpos],
- sizeof(Client *) *
- (nclients - nextpos - 1));
- }
+ sizeof(Client *) * (nclients - nextpos - 1));
+
                 clients[nextpos] = c;
                 updatetitle(nextpos);
 
_AT_@ -772,44 +783,51 @@ manage(Window w) {
                 XSync(dpy, False);
 
                 /* Adjust sel before focus does set it to lastsel. */
- if(sel >= nextpos)
+ if (sel >= nextpos)
                         sel++;
- focus((nextfocus)? nextpos : ((sel < 0)? 0 : sel));
+ focus(nextfocus ? nextpos :
+ sel < 0 ? 0 :
+ sel);
                 nextfocus = foreground;
         }
 }
 
 void
-maprequest(const XEvent *e) {
+maprequest(const XEvent *e)
+{
         const XMapRequestEvent *ev = &e->xmaprequest;
 
- if(getclient(ev->window) < 0)
+ if (getclient(ev->window) < 0)
                 manage(ev->window);
 }
 
 void
-move(const Arg *arg) {
- if(arg->i >= 0 && arg->i < nclients)
+move(const Arg *arg)
+{
+ if (arg->i >= 0 && arg->i < nclients)
                 focus(arg->i);
 }
 
 void
-movetab(const Arg *arg) {
+movetab(const Arg *arg)
+{
         int c;
         Client *new;
 
         c = (sel + arg->i) % nclients;
- if(c < 0)
+ if (c < 0)
                 c += nclients;
 
- if(sel < 0 || (c == sel))
+ if (sel < 0 || c == sel)
                 return;
 
         new = clients[sel];
- if(sel < c)
- memmove(&clients[sel], &clients[sel+1], sizeof(Client *) * (c - sel));
+ if (sel < c)
+ memmove(&clients[sel], &clients[sel+1],
+ sizeof(Client *) * (c - sel));
         else
- memmove(&clients[c+1], &clients[c], sizeof(Client *) * (sel - c));
+ memmove(&clients[c+1], &clients[c],
+ sizeof(Client *) * (sel - c));
         clients[c] = new;
         sel = c;
 
_AT_@ -817,16 +835,17 @@ movetab(const Arg *arg) {
 }
 
 void
-propertynotify(const XEvent *e) {
+propertynotify(const XEvent *e)
+{
         const XPropertyEvent *ev = &e->xproperty;
         XWMHints *wmh;
         int c;
         char* selection = NULL;
         Arg arg;
 
- if(ev->state == PropertyNewValue && ev->atom == wmatom[WMSelectTab]) {
+ if (ev->state == PropertyNewValue && ev->atom == wmatom[WMSelectTab]) {
                 selection = getatom(WMSelectTab);
- if(!strncmp(selection, "0x", 2)) {
+ if (!strncmp(selection, "0x", 2)) {
                         arg.i = getclient(strtoul(selection, NULL, 0));
                         move(&arg);
                 } else {
_AT_@ -834,38 +853,44 @@ propertynotify(const XEvent *e) {
                         arg.v = cmd;
                         spawn(&arg);
                 }
- } else if(ev->state == PropertyNewValue && ev->atom == XA_WM_HINTS
- && (c = getclient(ev->window)) > -1
- && (wmh = XGetWMHints(dpy, clients[c]->win))) {
- if(wmh->flags & XUrgencyHint) {
+ } else if (ev->state == PropertyNewValue && ev->atom == XA_WM_HINTS &&
+ (c = getclient(ev->window)) > -1 &&
+ (wmh = XGetWMHints(dpy, clients[c]->win))) {
+ if (wmh->flags & XUrgencyHint) {
                         XFree(wmh);
                         wmh = XGetWMHints(dpy, win);
- if(c != sel) {
- if(urgentswitch && wmh && !(wmh->flags & XUrgencyHint)) {
- /* only switch, if tabbed was focused since last urgency hint
- * if WMHints could not be received, default to no switch */
+ if (c != sel) {
+ if (urgentswitch && wmh &&
+ !(wmh->flags & XUrgencyHint)) {
+ /* only switch, if tabbed was focused
+ * since last urgency hint if WMHints
+ * could not be received,
+ * default to no switch */
                                         focus(c);
                                 } else {
- /* if no switch should be performed, mark tab as urgent */
+ /* if no switch should be performed,
+ * mark tab as urgent */
                                         clients[c]->urgent = True;
                                         drawbar();
                                 }
                         }
- if(wmh && !(wmh->flags & XUrgencyHint)) {
- /* update tabbed urgency hint if not set already */
+ if (wmh && !(wmh->flags & XUrgencyHint)) {
+ /* update tabbed urgency hint
+ * if not set already */
                                 wmh->flags |= XUrgencyHint;
                                 XSetWMHints(dpy, win, wmh);
                         }
                 }
                 XFree(wmh);
- } else if(ev->state != PropertyDelete && ev->atom == XA_WM_NAME
- && (c = getclient(ev->window)) > -1) {
+ } else if (ev->state != PropertyDelete && ev->atom == XA_WM_NAME &&
+ (c = getclient(ev->window)) > -1) {
                 updatetitle(c);
         }
 }
 
 void
-resize(int c, int w, int h) {
+resize(int c, int w, int h)
+{
         XConfigureEvent ce;
         XWindowChanges wc;
 
_AT_@ -881,51 +906,54 @@ resize(int c, int w, int h) {
         ce.override_redirect = False;
         ce.border_width = 0;
 
- XConfigureWindow(dpy, clients[c]->win, CWWidth|CWHeight, &wc);
+ XConfigureWindow(dpy, clients[c]->win, CWWidth | CWHeight, &wc);
         XSendEvent(dpy, clients[c]->win, False, StructureNotifyMask,
- (XEvent *)&ce);
+ (XEvent *)&ce);
 }
 
 void
-rotate(const Arg *arg) {
+rotate(const Arg *arg)
+{
         int nsel = -1;
 
- if(sel < 0)
+ if (sel < 0)
                 return;
 
- if(arg->i == 0) {
- if(lastsel > -1)
+ if (arg->i == 0) {
+ if (lastsel > -1)
                         focus(lastsel);
- } else if(sel > -1) {
+ } else if (sel > -1) {
                 /* Rotating in an arg->i step around the clients. */
                 nsel = sel + arg->i;
- while(nsel >= nclients)
+ while (nsel >= nclients)
                         nsel -= nclients;
- while(nsel < 0)
+ while (nsel < 0)
                         nsel += nclients;
                 focus(nsel);
         }
 }
 
 void
-run(void) {
+run(void)
+{
         XEvent ev;
 
         /* main event loop */
         XSync(dpy, False);
         drawbar();
- if(doinitspawn == True)
+ if (doinitspawn == True)
                 spawn(NULL);
 
- while(running) {
+ while (running) {
                 XNextEvent(dpy, &ev);
- if(handler[ev.type])
+ if (handler[ev.type])
                         (handler[ev.type])(&ev); /* call handler */
         }
 }
 
 void
-sendxembed(int c, long msg, long detail, long d1, long d2) {
+sendxembed(int c, long msg, long detail, long d1, long d2)
+{
         XEvent e = { 0 };
 
         e.xclient.window = clients[c]->win;
_AT_@ -941,21 +969,23 @@ sendxembed(int c, long msg, long detail, long d1, long d2) {
 }
 
 void
-setcmd(int argc, char *argv[], int replace) {
+setcmd(int argc, char *argv[], int replace)
+{
         int i;
 
- cmd = ecalloc(argc + 3, sizeof *cmd);
+ cmd = ecalloc(argc + 3, sizeof(*cmd));
         if (argc == 0)
                 return;
- for(i = 0; i < argc; i++)
+ for (i = 0; i < argc; i++)
                 cmd[i] = argv[i];
- cmd[(replace > 0)? replace : argc] = winid;
+ cmd[replace > 0 ? replace : argc] = winid;
         cmd_append_pos = argc + !replace;
- cmd[cmd_append_pos] = cmd[cmd_append_pos+1] = NULL;
+ cmd[cmd_append_pos] = cmd[cmd_append_pos + 1] = NULL;
 }
 
 void
-setup(void) {
+setup(void)
+{
         int bitm, tx, ty, tw, th, dh, dw, isfixed;
         XWMHints *wmh;
         XClassHint class_hint;
_AT_@ -971,13 +1001,14 @@ setup(void) {
         bh = dc.h = dc.font.height + 2;
 
         /* init atoms */
- wmatom[WMProtocols] = XInternAtom(dpy, "WM_PROTOCOLS", False);
         wmatom[WMDelete] = XInternAtom(dpy, "WM_DELETE_WINDOW", False);
- wmatom[XEmbed] = XInternAtom(dpy, "_XEMBED", False);
+ wmatom[WMFullscreen] = XInternAtom(dpy, "_NET_WM_STATE_FULLSCREEN",
+ False);
         wmatom[WMName] = XInternAtom(dpy, "_NET_WM_NAME", False);
- wmatom[WMState] = XInternAtom(dpy, "_NET_WM_STATE", False);
- wmatom[WMFullscreen] = XInternAtom(dpy, "_NET_WM_STATE_FULLSCREEN", False);
+ wmatom[WMProtocols] = XInternAtom(dpy, "WM_PROTOCOLS", False);
         wmatom[WMSelectTab] = XInternAtom(dpy, "_TABBED_SELECT_TAB", False);
+ wmatom[WMState] = XInternAtom(dpy, "_NET_WM_STATE", False);
+ wmatom[XEmbed] = XInternAtom(dpy, "_XEMBED", False);
 
         /* init appearance */
         wx = 0;
_AT_@ -986,30 +1017,30 @@ setup(void) {
         wh = 600;
         isfixed = 0;
 
- if(geometry) {
+ if (geometry) {
                 tx = ty = tw = th = 0;
                 bitm = XParseGeometry(geometry, &tx, &ty, (unsigned *)&tw,
- (unsigned *)&th);
- if(bitm & XValue)
+ (unsigned *)&th);
+ if (bitm & XValue)
                         wx = tx;
- if(bitm & YValue)
+ if (bitm & YValue)
                         wy = ty;
- if(bitm & WidthValue)
+ if (bitm & WidthValue)
                         ww = tw;
- if(bitm & HeightValue)
+ if (bitm & HeightValue)
                         wh = th;
- if(bitm & XNegative && wx == 0)
+ if (bitm & XNegative && wx == 0)
                         wx = -1;
- if(bitm & YNegative && wy == 0)
+ if (bitm & YNegative && wy == 0)
                         wy = -1;
- if(bitm & (HeightValue|WidthValue))
+ if (bitm & (HeightValue | WidthValue))
                         isfixed = 1;
 
                 dw = DisplayWidth(dpy, screen);
                 dh = DisplayHeight(dpy, screen);
- if(wx < 0)
+ if (wx < 0)
                         wx = dw + wx - ww - 1;
- if(wy < 0)
+ if (wy < 0)
                         wy = dh + wy - wh - 1;
         }
 
_AT_@ -1020,17 +1051,18 @@ setup(void) {
         dc.urg[ColBG] = getcolor(urgbgcolor);
         dc.urg[ColFG] = getcolor(urgfgcolor);
         dc.drawable = XCreatePixmap(dpy, root, ww, wh,
- DefaultDepth(dpy, screen));
+ DefaultDepth(dpy, screen));
         dc.gc = XCreateGC(dpy, root, 0, 0);
- if(!dc.font.set)
+ if (!dc.font.set)
                 XSetFont(dpy, dc.gc, dc.font.xfont->fid);
 
         win = XCreateSimpleWindow(dpy, root, wx, wy, ww, wh, 0,
- dc.norm[ColFG], dc.norm[ColBG]);
+ dc.norm[ColFG], dc.norm[ColBG]);
         XMapRaised(dpy, win);
- XSelectInput(dpy, win, SubstructureNotifyMask|FocusChangeMask|
- ButtonPressMask|ExposureMask|KeyPressMask|PropertyChangeMask|
- StructureNotifyMask|SubstructureRedirectMask);
+ XSelectInput(dpy, win, SubstructureNotifyMask | FocusChangeMask |
+ ButtonPressMask | ExposureMask | KeyPressMask |
+ PropertyChangeMask | StructureNotifyMask |
+ SubstructureRedirectMask);
         xerrorxlib = XSetErrorHandler(xerror);
 
         class_hint.res_name = wmname;
_AT_@ -1038,7 +1070,7 @@ setup(void) {
         XSetClassHint(dpy, win, &class_hint);
 
         size_hint = XAllocSizeHints();
- if(!isfixed) {
+ if (!isfixed) {
                 size_hint->flags = PSize;
                 size_hint->height = wh;
                 size_hint->width = ww;
_AT_@ -1062,24 +1094,26 @@ setup(void) {
 }
 
 void
-sigchld(int unused) {
- if(signal(SIGCHLD, sigchld) == SIG_ERR)
+sigchld(int unused)
+{
+ if (signal(SIGCHLD, sigchld) == SIG_ERR)
                 die("tabbed: cannot install SIGCHLD handler");
 
- while(0 < waitpid(-1, NULL, WNOHANG));
+ while (0 < waitpid(-1, NULL, WNOHANG));
 }
 
 void
-spawn(const Arg *arg) {
- if(fork() == 0) {
+spawn(const Arg *arg)
+{
+ if (fork() == 0) {
                 if(dpy)
                         close(ConnectionNumber(dpy));
 
                 setsid();
- if(arg && arg->v) {
+ if (arg && arg->v) {
                         execvp(((char **)arg->v)[0], (char **)arg->v);
                         fprintf(stderr, "tabbed: execvp %s",
- ((char **)arg->v)[0]);
+ ((char **)arg->v)[0]);
                 } else {
                         cmd[cmd_append_pos] = NULL;
                         execvp(cmd[0], cmd);
_AT_@ -1091,12 +1125,12 @@ spawn(const Arg *arg) {
 }
 
 int
-textnw(const char *text, unsigned int len) {
+textnw(const char *text, unsigned int len)
+{
         XRectangle r;
 
- if(dc.font.set) {
+ if (dc.font.set) {
                 XmbTextExtents(dc.font.set, text, len, NULL, &r);
-
                 return r.width;
         }
 
_AT_@ -1104,27 +1138,29 @@ textnw(const char *text, unsigned int len) {
 }
 
 void
-toggle(const Arg *arg) {
+toggle(const Arg *arg)
+{
     *(Bool*) arg->v = !*(Bool*) arg->v;
 }
 
 void
-unmanage(int c) {
- if(c < 0 || c >= nclients) {
+unmanage(int c)
+{
+ if (c < 0 || c >= nclients) {
                 drawbar();
                 XSync(dpy, False);
                 return;
         }
 
- if(!nclients)
+ if (!nclients)
                 return;
 
- if(c == 0) {
+ if (c == 0) {
                 /* First client. */
                 nclients--;
                 free(clients[0]);
                 memmove(&clients[0], &clients[1], sizeof(Client *) * nclients);
- } else if(c == nclients - 1) {
+ } else if (c == nclients - 1) {
                 /* Last client. */
                 nclients--;
                 free(clients[c]);
_AT_@ -1133,31 +1169,29 @@ unmanage(int c) {
                 /* Somewhere inbetween. */
                 free(clients[c]);
                 memmove(&clients[c], &clients[c+1],
- sizeof(Client *) * (nclients - (c + 1)));
+ sizeof(Client *) * (nclients - (c + 1)));
                 nclients--;
         }
 
- if(nclients <= 0) {
+ if (nclients <= 0) {
                 lastsel = sel = -1;
 
- if(closelastclient) {
+ if (closelastclient)
                         running = False;
- } else if(fillagain && running) {
+ else if (fillagain && running)
                         spawn(NULL);
- }
         } else {
- if(lastsel >= nclients) {
+ if (lastsel >= nclients)
                         lastsel = nclients - 1;
- } else if(lastsel > c) {
+ else if (lastsel > c)
                         lastsel--;
- }
 
- if(c == sel && lastsel >= 0) {
+ if (c == sel && lastsel >= 0) {
                         focus(lastsel);
                 } else {
- if(sel > c)
+ if (sel > c)
                                 sel--;
- if(sel >= nclients)
+ if (sel >= nclients)
                                 sel = nclients - 1;
 
                         focus(sel);
_AT_@ -1169,32 +1203,41 @@ unmanage(int c) {
 }
 
 void
-updatenumlockmask(void) {
+unmapnotify(const XEvent *e)
+{
+ const XUnmapEvent *ev = &e->xunmap;
+ int c;
+
+ if ((c = getclient(ev->window)) > -1)
+ unmanage(c);
+}
+
+void
+updatenumlockmask(void)
+{
         unsigned int i, j;
         XModifierKeymap *modmap;
 
         numlockmask = 0;
         modmap = XGetModifierMapping(dpy);
- for(i = 0; i < 8; i++) {
- for(j = 0; j < modmap->max_keypermod; j++) {
- if(modmap->modifiermap[i * modmap->max_keypermod + j]
- == XKeysymToKeycode(dpy,
- XK_Num_Lock)) {
+ for (i = 0; i < 8; i++) {
+ for (j = 0; j < modmap->max_keypermod; j++) {
+ if (modmap->modifiermap[i * modmap->max_keypermod + j]
+ == XKeysymToKeycode(dpy, XK_Num_Lock))
                                 numlockmask = (1 << i);
- }
                 }
         }
         XFreeModifiermap(modmap);
 }
 
 void
-updatetitle(int c) {
- if(!gettextprop(clients[c]->win, wmatom[WMName],
- clients[c]->name, sizeof(clients[c]->name))) {
- gettextprop(clients[c]->win, XA_WM_NAME,
- clients[c]->name, sizeof(clients[c]->name));
- }
- if(sel == c)
+updatetitle(int c)
+{
+ if (!gettextprop(clients[c]->win, wmatom[WMName], clients[c]->name,
+ sizeof(clients[c]->name)))
+ gettextprop(clients[c]->win, XA_WM_NAME, clients[c]->name,
+ sizeof(clients[c]->name));
+ if (sel == c)
                 xsettitle(win, clients[c]->name);
         drawbar();
 }
_AT_@ -1203,55 +1246,56 @@ updatetitle(int c) {
  * ignored (especially on UnmapNotify's). Other types of errors call Xlibs
  * default error handler, which may call exit. */
 int
-xerror(Display *dpy, XErrorEvent *ee) {
- if(ee->error_code == BadWindow
- || (ee->request_code == X_SetInputFocus
- && ee->error_code == BadMatch)
- || (ee->request_code == X_PolyText8
- && ee->error_code == BadDrawable)
- || (ee->request_code == X_PolyFillRectangle
- && ee->error_code == BadDrawable)
- || (ee->request_code == X_PolySegment
- && ee->error_code == BadDrawable)
- || (ee->request_code == X_ConfigureWindow
- && ee->error_code == BadMatch)
- || (ee->request_code == X_GrabButton
- && ee->error_code == BadAccess)
- || (ee->request_code == X_GrabKey
- && ee->error_code == BadAccess)
- || (ee->request_code == X_CopyArea
- && ee->error_code == BadDrawable)) {
+xerror(Display *dpy, XErrorEvent *ee)
+{
+ if (ee->error_code == BadWindow
+ || (ee->request_code == X_SetInputFocus &&
+ ee->error_code == BadMatch)
+ || (ee->request_code == X_PolyText8 &&
+ ee->error_code == BadDrawable)
+ || (ee->request_code == X_PolyFillRectangle &&
+ ee->error_code == BadDrawable)
+ || (ee->request_code == X_PolySegment &&
+ ee->error_code == BadDrawable)
+ || (ee->request_code == X_ConfigureWindow &&
+ ee->error_code == BadMatch)
+ || (ee->request_code == X_GrabButton &&
+ ee->error_code == BadAccess)
+ || (ee->request_code == X_GrabKey &&
+ ee->error_code == BadAccess)
+ || (ee->request_code == X_CopyArea &&
+ ee->error_code == BadDrawable))
                 return 0;
- }
 
         fprintf(stderr, "tabbed: fatal error: request code=%d, error code=%d\n",
- ee->request_code, ee->error_code);
+ ee->request_code, ee->error_code);
         return xerrorxlib(dpy, ee); /* may call exit */
 }
 
 void
-xsettitle(Window w, const char *str) {
+xsettitle(Window w, const char *str)
+{
         XTextProperty xtp;
 
- if(XmbTextListToTextProperty(dpy, (char **)&str, 1, XCompoundTextStyle,
- &xtp) == Success) {
+ if (XmbTextListToTextProperty(dpy, (char **)&str, 1,
+ XCompoundTextStyle, &xtp) == Success) {
                 XSetTextProperty(dpy, w, &xtp, wmatom[WMName]);
                 XSetTextProperty(dpy, w, &xtp, XA_WM_NAME);
                 XFree(xtp.value);
         }
 }
 
-char *argv0;
-
 void
-usage(void) {
- die("usage: %s [-dfhsv] [-g geometry] [-n name] [-p [s+/-]pos] [-r narg] "
- "[-o color] [-O color] [-t color] [-T color] [-u color] [-U color] "
- "command...\n", argv0);
+usage(void)
+{
+ die("usage: %s [-dfhsv] [-g geometry] [-n name] [-p [s+/-]pos] "
+ "[-r narg] [-o color] [-O color] [-t color] [-T color] "
+ "[-u color] [-U color] command...\n", argv0);
 }
 
 int
-main(int argc, char *argv[]) {
+main(int argc, char *argv[])
+{
         Bool detach = False;
         int replace = 0;
         char *pstr;
_AT_@ -1273,9 +1317,15 @@ main(int argc, char *argv[]) {
         case 'n':
                 wmname = EARGF(usage());
                 break;
+ case 'O':
+ normfgcolor = EARGF(usage());
+ break;
+ case 'o':
+ normbgcolor = EARGF(usage());
+ break;
         case 'p':
                 pstr = EARGF(usage());
- if(pstr[0] == 's') {
+ if (pstr[0] == 's') {
                         npisrelative = True;
                         newposition = atoi(&pstr[1]);
                 } else {
_AT_@ -1288,55 +1338,49 @@ main(int argc, char *argv[]) {
         case 's':
                 doinitspawn = False;
                 break;
- case 'o':
- normbgcolor = EARGF(usage());
- break;
- case 'O':
- normfgcolor = EARGF(usage());
+ case 'T':
+ selfgcolor = EARGF(usage());
                 break;
         case 't':
                 selbgcolor = EARGF(usage());
                 break;
- case 'T':
- selfgcolor = EARGF(usage());
+ case 'U':
+ urgfgcolor = EARGF(usage());
                 break;
         case 'u':
                 urgbgcolor = EARGF(usage());
                 break;
- case 'U':
- urgfgcolor = EARGF(usage());
- break;
         case 'v':
                 die("tabbed-"VERSION", © 2009-2016"
                         " tabbed engineers, see LICENSE"
                         " for details.\n");
                 break;
- default:
+ default: /* FALLTHROUGH */
         case 'h':
                 usage();
         } ARGEND;
 
- if(argc < 1) {
+ if (argc < 1) {
                 doinitspawn = False;
                 fillagain = False;
         }
 
         setcmd(argc, argv, replace);
 
- if(!setlocale(LC_CTYPE, "") || !XSupportsLocale())
- fprintf(stderr, "tabbed: no locale support\n");
- if(!(dpy = XOpenDisplay(NULL)))
- die("tabbed: cannot open display\n");
+ if (!setlocale(LC_CTYPE, "") || !XSupportsLocale())
+ fprintf(stderr, "%s: no locale support\n", argv0);
+ if (!(dpy = XOpenDisplay(NULL)))
+ die("%s: cannot open display\n", argv0);
 
         setup();
         printf("0x%lx\n", win);
         fflush(NULL);
 
- if(detach) {
- if(fork() == 0) {
+ if (detach) {
+ if (fork() == 0) {
                         fclose(stdout);
                 } else {
- if(dpy)
+ if (dpy)
                                 close(ConnectionNumber(dpy));
                         return EXIT_SUCCESS;
                 }
_AT_@ -1348,4 +1392,3 @@ main(int argc, char *argv[]) {
 
         return EXIT_SUCCESS;
 }
-
diff --git a/xembed.c b/xembed.c
index 33e8d92..cbb0e97 100644
--- a/xembed.c
+++ b/xembed.c
_AT_@ -8,21 +8,21 @@
 #include <unistd.h>
 
 int
-main(int argc, char **argv)
+main(int argc, char *argv[])
 {
         char *xembed;
         int tty;
         pid_t pgrp, tcpgrp;
 
- if(argc < 3) {
+ if (argc < 3) {
                 fprintf(stderr, "usage: %s flag cmd ...\n", argv[0]);
                 return 2;
         }
 
- if(!(xembed = getenv("XEMBED")))
+ if (!(xembed = getenv("XEMBED")))
                 goto noembed;
 
- if((tty = open("/dev/tty", O_RDONLY)) < 0)
+ if ((tty = open("/dev/tty", O_RDONLY)) < 0)
                 goto noembed;
 
         pgrp = getpgrp();
_AT_@ -30,7 +30,7 @@ main(int argc, char **argv)
 
         close(tty);
 
- if(pgrp == tcpgrp) { /* in foreground of tty */
+ if (pgrp == tcpgrp) { /* in foreground of tty */
                 argv[0] = argv[2];
                 argv[2] = xembed;
         } else {
_AT_@ -43,4 +43,3 @@ noembed:
         perror(argv[0]); /* failed to execute */
         return 1;
 }
-
Received on Sun Jan 03 2016 - 14:11:36 CET

This archive was generated by hypermail 2.3.0 : Sun Jan 03 2016 - 14:12:13 CET