all repos — dmenu @ 32f7fe483538be60e529415c8cbd9cfbb4ab86d9

my build of dmenu

implemented early keyboard grab for dmenu with a timeout for stdin data writers to prevent endless grabbings of the keyboard
Anselm R. Garbe arg@10kloc.org
Fri, 08 Sep 2006 07:33:20 +0200
commit

32f7fe483538be60e529415c8cbd9cfbb4ab86d9

parent

0fa5a339ffac530eb596cc1b93c4ee696ef0ec74

3 files changed, 21 insertions(+), 4 deletions(-)

jump to
M config.arg.hconfig.arg.h

@@ -8,3 +8,4 @@ #define SELBGCOLOR "#333366"

#define SELFGCOLOR "#eeeeee" #define NORMBGCOLOR "#333333" #define NORMFGCOLOR "#dddddd" +#define STDIN_TIMEOUT 3 /* seconds */
M config.default.hconfig.default.h

@@ -8,3 +8,4 @@ #define SELBGCOLOR "#666699"

#define SELFGCOLOR "#eeeeee" #define NORMBGCOLOR "#333366" #define NORMFGCOLOR "#cccccc" +#define STDIN_TIMEOUT 3 /* seconds */
M main.cmain.c

@@ -11,6 +11,8 @@ #include <stdlib.h>

#include <stdio.h> #include <string.h> #include <unistd.h> +#include <sys/select.h> +#include <sys/time.h> #include <X11/cursorfont.h> #include <X11/Xutil.h> #include <X11/keysym.h>

@@ -290,6 +292,8 @@ int

main(int argc, char *argv[]) { char *maxname; + fd_set rd; + struct timeval timeout; Item *i; XEvent ev; XSetWindowAttributes wa;

@@ -307,12 +311,22 @@ eprint("dmenu: cannot open display\n");

screen = DefaultScreen(dpy); root = RootWindow(dpy, screen); - maxname = readstdin(); - - /* grab as early as possible, but after reading all items!!! */ + /* Note, the select() construction allows to grab all keypresses as + * early as possible, to not loose them. But if there is no standard + * input supplied, we will make sure to exit after MAX_WAIT_STDIN + * seconds. This is convenience behavior for rapid typers. + */ while(XGrabKeyboard(dpy, root, True, GrabModeAsync, GrabModeAsync, CurrentTime) != GrabSuccess) usleep(1000); + + timeout.tv_usec = 0; + timeout.tv_sec = STDIN_TIMEOUT; + FD_ZERO(&rd); + FD_SET(STDIN_FILENO, &rd); + if(select(ConnectionNumber(dpy) + 1, &rd, NULL, NULL, &timeout) < 1) + goto UninitializedEnd; + maxname = readstdin(); /* style */ dc.sel[ColBG] = getcolor(SELBGCOLOR);

@@ -366,7 +380,6 @@ break;

} } - XUngrabKeyboard(dpy, CurrentTime); while(allitems) { i = allitems->next; free(allitems->text);

@@ -380,6 +393,8 @@ XFreeFont(dpy, dc.font.xfont);

XFreePixmap(dpy, dc.drawable); XFreeGC(dpy, dc.gc); XDestroyWindow(dpy, win); +UninitializedEnd: + XUngrabKeyboard(dpy, CurrentTime); XCloseDisplay(dpy); return ret;