since the root and event window are the same in the case of a button

event on the screen's root window, there's no need to pass down the
entire XButtonEvent event, at least to group_menu(), the only callback
which takes an argument at this point; instead use the already populated
screen.
This commit is contained in:
okan 2013-07-09 01:24:49 +00:00
parent 9bf2498d7e
commit dd10412804
3 changed files with 3 additions and 6 deletions

View File

@ -360,7 +360,7 @@ void group_autogroup(struct client_ctx *);
void group_cycle(struct screen_ctx *, int);
void group_hidetoggle(struct screen_ctx *, int);
void group_init(struct screen_ctx *);
void group_menu(XButtonEvent *);
void group_menu(struct screen_ctx *);
void group_movetogroup(struct client_ctx *, int);
void group_only(struct screen_ctx *, int);
void group_sticky(struct client_ctx *);

View File

@ -309,16 +309,13 @@ group_cycle(struct screen_ctx *sc, int flags)
}
void
group_menu(XButtonEvent *e)
group_menu(struct screen_ctx *sc)
{
struct screen_ctx *sc;
struct group_ctx *gc;
struct menu *mi;
struct menu_q menuq;
int i;
sc = screen_fromroot(e->root);
TAILQ_INIT(&menuq);
for (i = 0; i < CALMWM_NGROUPS; i++) {

View File

@ -216,7 +216,7 @@ mousefunc_client_rcyclegroup(struct client_ctx *cc, void *arg)
void
mousefunc_menu_group(struct client_ctx *cc, void *arg)
{
group_menu(arg);
group_menu(cc->sc);
}
void