diff options
| -rw-r--r-- | calmwm.h | 6 | ||||
| -rw-r--r-- | client.c | 19 | ||||
| -rw-r--r-- | conf.c | 1 | ||||
| -rw-r--r-- | util.c | 5 |
4 files changed, 10 insertions, 21 deletions
diff --git a/calmwm.h b/calmwm.h index 112c8dd..a99c334 100644 --- a/calmwm.h +++ b/calmwm.h @@ -307,8 +307,8 @@ __dead void usage(void); void client_applysizehints(struct client_ctx *); struct client_ctx *client_current(void); -struct client_ctx *client_cycle(struct screen_ctx *, int); -int client_delete(struct client_ctx *); +void client_cycle(struct screen_ctx *, int); +void client_delete(struct client_ctx *); void client_draw_border(struct client_ctx *); struct client_ctx *client_find(Window); void client_freeze(struct client_ctx *); @@ -463,7 +463,7 @@ void xu_setstate(struct client_ctx *, int); void xu_setwmname(struct screen_ctx *); void u_exec(char *); -int u_spawn(char *); +void u_spawn(char *); void *xcalloc(size_t, size_t); void xfree(void *); diff --git a/client.c b/client.c index a18ffcc..de6cb1b 100644 --- a/client.c +++ b/client.c @@ -137,7 +137,7 @@ client_new(Window win, struct screen_ctx *sc, int mapped) return (cc); } -int +void client_delete(struct client_ctx *cc) { struct screen_ctx *sc = cc->sc; @@ -189,8 +189,6 @@ client_delete(struct client_ctx *cc) client_freehints(cc); xfree(cc); - - return (0); } void @@ -550,9 +548,8 @@ client_setname(struct client_ctx *cc) char *newname; if (!xu_getstrprop(cc->win, _NET_WM_NAME, &newname)) - xu_getstrprop(cc->win, XA_WM_NAME, &newname); - if (newname == NULL) - newname = emptystring; + if (!xu_getstrprop(cc->win, XA_WM_NAME, &newname)) + newname = emptystring; TAILQ_FOREACH(wn, &cc->nameq, entry) if (strcmp(wn->name, newname) == 0) { @@ -580,11 +577,9 @@ match: xfree(wn); cc->nameqlen--; } - - return; } -struct client_ctx * +void client_cycle(struct screen_ctx *sc, int reverse) { struct client_ctx *oldcc, *newcc; @@ -594,7 +589,7 @@ client_cycle(struct screen_ctx *sc, int reverse) /* If no windows then you cant cycle */ if (TAILQ_EMPTY(&sc->mruq)) - return (NULL); + return; if (oldcc == NULL) oldcc = (reverse ? TAILQ_LAST(&sc->mruq, cycle_entry_q) : @@ -614,7 +609,7 @@ client_cycle(struct screen_ctx *sc, int reverse) /* Is oldcc the only non-hidden window? */ if (newcc == oldcc) { if (again) - return (NULL); /* No windows visible. */ + return; /* No windows visible. */ break; } @@ -624,8 +619,6 @@ client_cycle(struct screen_ctx *sc, int reverse) sc->altpersist = 1; client_ptrsave(oldcc); client_ptrwarp(newcc); - - return (newcc); } static struct client_ctx * diff --git a/conf.c b/conf.c index 46478b2..1bd1788 100644 --- a/conf.c +++ b/conf.c @@ -521,7 +521,6 @@ conf_bindname(struct conf *c, char *name, char *binding) current_binding->flags = 0; conf_grab(c, current_binding); TAILQ_INSERT_TAIL(&c->keybindingq, current_binding, entry); - return; } static void diff --git a/util.c b/util.c index be4b68a..b8ea87f 100644 --- a/util.c +++ b/util.c @@ -32,7 +32,7 @@ #define MAXARGLEN 20 -int +void u_spawn(char *argstr) { switch (fork()) { @@ -42,12 +42,9 @@ u_spawn(char *argstr) break; case -1: warn("fork"); - return (-1); default: break; } - - return (0); } void |