From 19826222f6dc85116145acaf8210c83c2c91dab6 Mon Sep 17 00:00:00 2001
From: okan <okan>
Date: Thu, 27 Aug 2015 18:40:09 +0000
Subject: Move client cycle grab/ungrab into a more relevant place; while here,
 update comments about why we need to grab/ungrab the keyboard.

---
 kbfunc.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

(limited to 'kbfunc.c')

diff --git a/kbfunc.c b/kbfunc.c
index 8ff230d..8d220a6 100644
--- a/kbfunc.c
+++ b/kbfunc.c
@@ -219,13 +219,7 @@ kbfunc_menu_group(struct client_ctx *cc, union arg *arg)
 void
 kbfunc_client_cycle(struct client_ctx *cc, union arg *arg)
 {
-	struct screen_ctx	*sc = cc->sc;
-
-	/* XXX for X apps that ignore events */
-	XGrabKeyboard(X_Dpy, sc->rootwin, True,
-	    GrabModeAsync, GrabModeAsync, CurrentTime);
-
-	client_cycle(sc, arg->i);
+	client_cycle(cc->sc, arg->i);
 }
 
 void
@@ -464,7 +458,7 @@ void
 kbfunc_client_grouptoggle(struct client_ctx *cc, union arg *arg)
 {
 	if (arg->i == 0) {
-		/* XXX for stupid X apps like xpdf and gvim */
+		/* For X apps that steal events. */
 		XGrabKeyboard(X_Dpy, cc->win, True,
 		    GrabModeAsync, GrabModeAsync, CurrentTime);
 	}
-- 
cgit 1.4.1