making copyright notice more compact

[dmenu.git] / main.c

diff --git a/main.c b/main.c
index ee32011..c63ac08 100644 (file)
--- a/main.c
+++ b/main.c
@@ -1,20 +1,18 @@
-/*
- * (C)opyright MMVI Anselm R. Garbe <garbeam at gmail dot com>
- * (C)opyright MMVI Sander van Dijk <a dot h dot vandijk at gmail dot com>
- * See LICENSE file for license details.
- */
-
+/* © 2006-2007 Anselm R. Garbe <garbeam at gmail dot com>
+ * © 2006-2007 Sander van Dijk <a dot h dot vandijk at gmail dot com>
+ * See LICENSE file for license details. */

 #include "dmenu.h"
 #include "dmenu.h"

-

 #include <ctype.h>
 #include <ctype.h>

+#include <locale.h>

 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
 #include <unistd.h>
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
 #include <unistd.h>

-#include <X11/cursorfont.h>

 #include <X11/Xutil.h>
 #include <X11/keysym.h>
 
 #include <X11/Xutil.h>
 #include <X11/keysym.h>
 

+#define CLEANMASK(mask) (mask & ~(numlockmask | LockMask))
+

 typedef struct Item Item;
 struct Item {
        Item *next;             /* traverses all items */
 typedef struct Item Item;
 struct Item {
        Item *next;             /* traverses all items */


@@ -25,11 +23,14 @@ struct Item {
 /* static */
 
 static char text[4096];
 /* static */
 
 static char text[4096];

-static int mx, my, mw, mh;
+static char *prompt = NULL;
+static int mw, mh;

 static int ret = 0;
 static int nitem = 0;
 static unsigned int cmdw = 0;
 static int ret = 0;
 static int nitem = 0;
 static unsigned int cmdw = 0;

-static Bool done = False;
+static unsigned int promptw = 0;
+static unsigned int numlockmask = 0;
+static Bool running = True;

 static Item *allitems = NULL;  /* first of all items */
 static Item *item = NULL;      /* first of pattern matching items */
 static Item *sel = NULL;
 static Item *allitems = NULL;  /* first of all items */
 static Item *item = NULL;      /* first of pattern matching items */
 static Item *sel = NULL;


@@ -40,14 +41,12 @@ static Window root;
 static Window win;
 
 static void
 static Window win;
 
 static void

-calcoffsets()
-{
+calcoffsets(void) {

        unsigned int tw, w;
 
        if(!curr)
                return;
        unsigned int tw, w;
 
        if(!curr)
                return;

-
-       w = cmdw + 2 * SPACE;
+       w = promptw + cmdw + 2 * SPACE;

        for(next = curr; next; next=next->right) {
                tw = textw(next->text);
                if(tw > mw / 3)
        for(next = curr; next; next=next->right) {
                tw = textw(next->text);
                if(tw > mw / 3)


@@ -56,8 +55,7 @@ calcoffsets()
                if(w > mw)
                        break;
        }
                if(w > mw)
                        break;
        }

-
-       w = cmdw + 2 * SPACE;
+       w = promptw + cmdw + 2 * SPACE;

        for(prev = curr; prev && prev->left; prev=prev->left) {
                tw = textw(prev->left->text);
                if(tw > mw / 3)
        for(prev = curr; prev && prev->left; prev=prev->left) {
                tw = textw(prev->left->text);
                if(tw > mw / 3)


@@ -69,58 +67,117 @@ calcoffsets()
 }
 
 static void
 }
 
 static void

-drawmenu()
-{
+drawmenu(void) {

        Item *i;
 
        dc.x = 0;
        dc.y = 0;
        dc.w = mw;
        dc.h = mh;
        Item *i;
 
        dc.x = 0;
        dc.y = 0;
        dc.w = mw;
        dc.h = mh;

-       drawtext(NULL, False, False);
-
+       drawtext(NULL, dc.norm);
+       /* print prompt? */
+       if(promptw) {
+               dc.w = promptw;
+               drawtext(prompt, dc.sel);
+       }
+       dc.x += promptw;
+       dc.w = mw - promptw;

        /* print command */
        if(cmdw && item)
                dc.w = cmdw;
        /* print command */
        if(cmdw && item)
                dc.w = cmdw;

-       drawtext(text[0] ? text : NULL, False, False);
+       drawtext(text[0] ? text : NULL, dc.norm);

        dc.x += cmdw;
        dc.x += cmdw;

-

        if(curr) {
                dc.w = SPACE;
        if(curr) {
                dc.w = SPACE;

-               drawtext((curr && curr->left) ? "<" : NULL, False, False);
+               drawtext((curr && curr->left) ? "<" : NULL, dc.norm);

                dc.x += dc.w;
                dc.x += dc.w;

-

                /* determine maximum items */
                for(i = curr; i != next; i=i->right) {
                /* determine maximum items */
                for(i = curr; i != next; i=i->right) {

-                       dc.border = False;

                        dc.w = textw(i->text);
                        if(dc.w > mw / 3)
                                dc.w = mw / 3;
                        dc.w = textw(i->text);
                        if(dc.w > mw / 3)
                                dc.w = mw / 3;

-                       drawtext(i->text, sel == i, sel == i);
+                       drawtext(i->text, (sel == i) ? dc.sel : dc.norm);

                        dc.x += dc.w;
                }
                        dc.x += dc.w;
                }

-

                dc.x = mw - SPACE;
                dc.w = SPACE;
                dc.x = mw - SPACE;
                dc.w = SPACE;

-               drawtext(next ? ">" : NULL, False, False);
+               drawtext(next ? ">" : NULL, dc.norm);

        }
        XCopyArea(dpy, dc.drawable, win, dc.gc, 0, 0, mw, mh, 0, 0);
        XFlush(dpy);
 }
 
        }
        XCopyArea(dpy, dc.drawable, win, dc.gc, 0, 0, mw, mh, 0, 0);
        XFlush(dpy);
 }
 

+static Bool
+grabkeyboard(void) {
+       unsigned int len;
+
+       for(len = 1000; len; len--) {
+               if(XGrabKeyboard(dpy, root, True, GrabModeAsync, GrabModeAsync, CurrentTime)
+                       == GrabSuccess)
+                       break;
+               usleep(1000);
+       }
+       return len > 0;
+}
+
+static unsigned long
+initcolor(const char *colstr) {
+       Colormap cmap = DefaultColormap(dpy, screen);
+       XColor color;
+
+       if(!XAllocNamedColor(dpy, cmap, colstr, &color, &color))
+               eprint("error, cannot allocate color '%s'\n", colstr);
+       return color.pixel;
+}
+

 static void
 static void

-match(char *pattern)
-{
+initfont(const char *fontstr) {
+       char *def, **missing;
+       int i, n;
+
+       missing = NULL;
+       if(dc.font.set)
+               XFreeFontSet(dpy, dc.font.set);
+       dc.font.set = XCreateFontSet(dpy, fontstr, &missing, &n, &def);
+       if(missing)
+               XFreeStringList(missing);
+       if(dc.font.set) {
+               XFontSetExtents *font_extents;
+               XFontStruct **xfonts;
+               char **font_names;
+               dc.font.ascent = dc.font.descent = 0;
+               font_extents = XExtentsOfFontSet(dc.font.set);
+               n = XFontsOfFontSet(dc.font.set, &xfonts, &font_names);
+               for(i = 0, dc.font.ascent = 0, dc.font.descent = 0; i < n; i++) {
+                       if(dc.font.ascent < (*xfonts)->ascent)
+                               dc.font.ascent = (*xfonts)->ascent;
+                       if(dc.font.descent < (*xfonts)->descent)
+                               dc.font.descent = (*xfonts)->descent;
+                       xfonts++;
+               }
+       }
+       else {
+               if(dc.font.xfont)
+                       XFreeFont(dpy, dc.font.xfont);
+               dc.font.xfont = NULL;
+               if(!(dc.font.xfont = XLoadQueryFont(dpy, fontstr)))
+                       eprint("error, cannot load font: '%s'\n", fontstr);
+               dc.font.ascent = dc.font.xfont->ascent;
+               dc.font.descent = dc.font.xfont->descent;
+       }
+       dc.font.height = dc.font.ascent + dc.font.descent;
+}
+
+static void
+match(char *pattern) {

        unsigned int plen;
        Item *i, *j;
 
        if(!pattern)
                return;
        unsigned int plen;
        Item *i, *j;
 
        if(!pattern)
                return;

-

        plen = strlen(pattern);
        item = j = NULL;
        nitem = 0;
        plen = strlen(pattern);
        item = j = NULL;
        nitem = 0;

-

        for(i = allitems; i; i=i->next)
                if(!plen || !strncmp(pattern, i->text, plen)) {
                        if(!j)
        for(i = allitems; i; i=i->next)
                if(!plen || !strncmp(pattern, i->text, plen)) {
                        if(!j)


@@ -144,50 +201,131 @@ match(char *pattern)
                        j = i;
                        nitem++;
                }
                        j = i;
                        nitem++;
                }

-

        curr = prev = next = sel = item;
        calcoffsets();
 }
 
 static void
        curr = prev = next = sel = item;
        calcoffsets();
 }
 
 static void

-kpress(XKeyEvent * e)
-{
+kpress(XKeyEvent * e) {

        char buf[32];
        char buf[32];

-       int num, prev_nitem;
-       unsigned int i, len;
+       int i, num;
+       unsigned int len;

        KeySym ksym;
 
        len = strlen(text);
        buf[0] = 0;
        KeySym ksym;
 
        len = strlen(text);
        buf[0] = 0;

-       num = XLookupString(e, buf, sizeof(buf), &ksym, 0);
-
+       num = XLookupString(e, buf, sizeof buf, &ksym, 0);
+       if(IsKeypadKey(ksym)) { 
+               if(ksym == XK_KP_Enter) {
+                       ksym = XK_Return;
+               } else if(ksym >= XK_KP_0 && ksym <= XK_KP_9) {
+                       ksym = (ksym - XK_KP_0) + XK_0;
+               }
+       }

        if(IsFunctionKey(ksym) || IsKeypadKey(ksym)
                        || IsMiscFunctionKey(ksym) || IsPFKey(ksym)
                        || IsPrivateKeypadKey(ksym))
                return;
        if(IsFunctionKey(ksym) || IsKeypadKey(ksym)
                        || IsMiscFunctionKey(ksym) || IsPFKey(ksym)
                        || IsPrivateKeypadKey(ksym))
                return;

-

        /* first check if a control mask is omitted */
        if(e->state & ControlMask) {
                switch (ksym) {
                default:        /* ignore other control sequences */
                        return;
        /* first check if a control mask is omitted */
        if(e->state & ControlMask) {
                switch (ksym) {
                default:        /* ignore other control sequences */
                        return;

+               case XK_bracketleft:
+                       ksym = XK_Escape;

                        break;
                case XK_h:
                        break;
                case XK_h:

+               case XK_H:

                        ksym = XK_BackSpace;
                        break;
                        ksym = XK_BackSpace;
                        break;

-               case XK_U:
+               case XK_i:
+               case XK_I:
+                       ksym = XK_Tab;
+                       break;
+               case XK_j:
+               case XK_J:
+                       ksym = XK_Return;
+                       break;

                case XK_u:
                case XK_u:

+               case XK_U:

                        text[0] = 0;
                        match(text);
                        drawmenu();
                        return;
                        text[0] = 0;
                        match(text);
                        drawmenu();
                        return;

+               case XK_w:
+               case XK_W:
+                       if(len) {
+                               i = len - 1;
+                               while(i >= 0 && text[i] == ' ')
+                                       text[i--] = 0;
+                               while(i >= 0 && text[i] != ' ')
+                                       text[i--] = 0;
+                               match(text);
+                               drawmenu();
+                       }
+                       return;
+               }
+       }
+       if(CLEANMASK(e->state) & Mod1Mask) {
+               switch(ksym) {
+               default: return;
+               case XK_h:
+                       ksym = XK_Left;

                        break;
                        break;

-               case XK_bracketleft:
-                       ksym = XK_Escape;
+               case XK_l:
+                       ksym = XK_Right;
+                       break;
+               case XK_j:
+                       ksym = XK_Next;
+                       break;
+               case XK_k:
+                       ksym = XK_Prior;
+                       break;
+               case XK_g:
+                       ksym = XK_Home;
+                       break;
+               case XK_G:
+                       ksym = XK_End;

                        break;
                }
        }
        switch(ksym) {
                        break;
                }
        }
        switch(ksym) {

+       default:
+               if(num && !iscntrl((int) buf[0])) {
+                       buf[num] = 0;
+                       if(len > 0)
+                               strncat(text, buf, sizeof text);
+                       else
+                               strncpy(text, buf, sizeof text);
+                       match(text);
+               }
+               break;
+       case XK_BackSpace:
+               if(len) {
+                       text[--len] = 0;
+                       match(text);
+               }
+               break;
+       case XK_End:
+               if(!item)
+                       return;
+               while(next) {
+                       sel = curr = next;
+                       calcoffsets();
+               }
+               while(sel && sel->right)
+                       sel = sel->right;
+               break;
+       case XK_Escape:
+               ret = 1;
+               running = False;
+               break;
+       case XK_Home:
+               if(!item)
+                       return;
+               sel = curr = item;
+               calcoffsets();
+               break;

        case XK_Left:
                if(!(sel && sel->left))
                        return;
        case XK_Left:
                if(!(sel && sel->left))
                        return;


@@ -197,70 +335,56 @@ kpress(XKeyEvent * e)
                        calcoffsets();
                }
                break;
                        calcoffsets();
                }
                break;

-       case XK_Tab:
-               if(!sel)
+       case XK_Next:
+               if(!next)

                        return;
                        return;

-               strncpy(text, sel->text, sizeof(text));
-               match(text);
+               sel = curr = next;
+               calcoffsets();

                break;
                break;

-       case XK_Right:
-               if(!(sel && sel->right))
+       case XK_Prior:
+               if(!prev)

                        return;
                        return;

-               sel=sel->right;
-               if(sel == next) {
-                       curr = next;
-                       calcoffsets();
-               }
+               sel = curr = prev;
+               calcoffsets();

                break;
        case XK_Return:
                break;
        case XK_Return:

-               if(e->state & ShiftMask) {
-                       if(text)
-                               fprintf(stdout, "%s", text);
-               }
+               if((e->state & ShiftMask) && text)
+                       fprintf(stdout, "%s", text);

                else if(sel)
                        fprintf(stdout, "%s", sel->text);
                else if(text)
                        fprintf(stdout, "%s", text);
                fflush(stdout);
                else if(sel)
                        fprintf(stdout, "%s", sel->text);
                else if(text)
                        fprintf(stdout, "%s", text);
                fflush(stdout);

-               done = True;
+               running = False;

                break;
                break;

-       case XK_Escape:
-               ret = 1;
-               done = True;
-               break;
-       case XK_BackSpace:
-               if((i = len)) {
-                       prev_nitem = nitem;
-                       do {
-                               text[--i] = 0;
-                               match(text);
-                       } while(i && nitem && prev_nitem == nitem);
-                       match(text);
+       case XK_Right:
+               if(!(sel && sel->right))
+                       return;
+               sel=sel->right;
+               if(sel == next) {
+                       curr = next;
+                       calcoffsets();

                }
                break;
                }
                break;

-       default:
-               if(num && !iscntrl((int) buf[0])) {
-                       buf[num] = 0;
-                       if(len > 0)
-                               strncat(text, buf, sizeof(text));
-                       else
-                               strncpy(text, buf, sizeof(text));
-                       match(text);
-               }
+       case XK_Tab:
+               if(!sel)
+                       return;
+               strncpy(text, sel->text, sizeof text);
+               match(text);
+               break;

        }
        drawmenu();
 }
 
 static char *
        }
        drawmenu();
 }
 
 static char *

-readstdin()
-{
+readstdin(void) {

        static char *maxname = NULL;
        char *p, buf[1024];
        unsigned int len = 0, max = 0;
        Item *i, *new;
 
        i = 0;
        static char *maxname = NULL;
        char *p, buf[1024];
        unsigned int len = 0, max = 0;
        Item *i, *new;
 
        i = 0;

-       while(fgets(buf, sizeof(buf), stdin)) {
+       while(fgets(buf, sizeof buf, stdin)) {

                len = strlen(buf);
                if (buf[len - 1] == '\n')
                        buf[len - 1] = 0;
                len = strlen(buf);
                if (buf[len - 1] == '\n')
                        buf[len - 1] = 0;


@@ -269,7 +393,6 @@ readstdin()
                        maxname = p;
                        max = len;
                }
                        maxname = p;
                        max = len;
                }

-

                new = emalloc(sizeof(Item));
                new->next = new->left = new->right = NULL;
                new->text = p;
                new = emalloc(sizeof(Item));
                new->next = new->left = new->right = NULL;
                new->text = p;


@@ -283,6 +406,12 @@ readstdin()
        return maxname;
 }
 
        return maxname;
 }
 

+static void
+usage(void) {
+       eprint("usage: dmenu [-b] [-fn <font>] [-nb <color>] [-nf <color>]\n"
+               "             [-p <prompt>] [-sb <color>] [-sf <color>] [-v]\n");
+}
+

 /* extern */
 
 int screen;
 /* extern */
 
 int screen;


@@ -290,61 +419,102 @@ Display *dpy;
 DC dc = {0};
 
 int
 DC dc = {0};
 
 int

-main(int argc, char *argv[])
-{
+main(int argc, char *argv[]) {
+       Bool bottom = False;
+       char *font = FONT;

        char *maxname;
        char *maxname;

+       char *normbg = NORMBGCOLOR;
+       char *normfg = NORMFGCOLOR;
+       char *selbg = SELBGCOLOR;
+       char *selfg = SELFGCOLOR;
+       int i, j;
+       Item *itm;

        XEvent ev;
        XEvent ev;

+       XModifierKeymap *modmap;

        XSetWindowAttributes wa;
 
        XSetWindowAttributes wa;
 

-       if(argc == 2 && !strncmp("-v", argv[1], 3)) {
-               fputs("dmenu-"VERSION", (C)opyright MMVI Anselm R. Garbe\n", stdout);
-               exit(EXIT_SUCCESS);
-       }
-       else if(argc != 1)
-               eprint("usage: dmenu [-v]\n");
-
+       /* command line args */
+       for(i = 1; i < argc; i++)
+               if(!strncmp(argv[i], "-b", 3)) {
+                       bottom = True;
+               }
+               else if(!strncmp(argv[i], "-fn", 4)) {
+                       if(++i < argc) font = argv[i];
+               }
+               else if(!strncmp(argv[i], "-nb", 4)) {
+                       if(++i < argc) normbg = argv[i];
+               }
+               else if(!strncmp(argv[i], "-nf", 4)) {
+                       if(++i < argc) normfg = argv[i];
+               }
+               else if(!strncmp(argv[i], "-p", 3)) {
+                       if(++i < argc) prompt = argv[i];
+               }
+               else if(!strncmp(argv[i], "-sb", 4)) {
+                       if(++i < argc) selbg = argv[i];
+               }
+               else if(!strncmp(argv[i], "-sf", 4)) {
+                       if(++i < argc) selfg = argv[i];
+               }
+               else if(!strncmp(argv[i], "-v", 3))
+                       eprint("dmenu-"VERSION", © 2006-2007 Anselm R. Garbe, Sander van Dijk\n");
+               else
+                       usage();
+       setlocale(LC_CTYPE, "");

        dpy = XOpenDisplay(0);
        if(!dpy)
                eprint("dmenu: cannot open display\n");
        screen = DefaultScreen(dpy);
        root = RootWindow(dpy, screen);
        dpy = XOpenDisplay(0);
        if(!dpy)
                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!!! */
-       while(XGrabKeyboard(dpy, root, True, GrabModeAsync,
-                        GrabModeAsync, CurrentTime) != GrabSuccess)
-               usleep(1000);
-
+       if(isatty(STDIN_FILENO)) {
+               maxname = readstdin();
+               running = grabkeyboard();
+       }
+       else { /* prevent keypress loss */
+               running = grabkeyboard();
+               maxname = readstdin();
+       }
+       /* init modifier map */
+       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))
+                               numlockmask = (1 << i);
+               }
+       }
+       XFreeModifiermap(modmap);

        /* style */
        /* style */

-       dc.bg = getcolor(BGCOLOR);
-       dc.fg = getcolor(FGCOLOR);
-       dc.border = getcolor(BORDERCOLOR);
-       setfont(FONT);
-
+       dc.norm[ColBG] = initcolor(normbg);
+       dc.norm[ColFG] = initcolor(normfg);
+       dc.sel[ColBG] = initcolor(selbg);
+       dc.sel[ColFG] = initcolor(selfg);
+       initfont(font);
+       /* menu window */

        wa.override_redirect = 1;
        wa.background_pixmap = ParentRelative;
        wa.event_mask = ExposureMask | ButtonPressMask | KeyPressMask;
        wa.override_redirect = 1;
        wa.background_pixmap = ParentRelative;
        wa.event_mask = ExposureMask | ButtonPressMask | KeyPressMask;

-
-       mx = my = 0;

        mw = DisplayWidth(dpy, screen);
        mw = DisplayWidth(dpy, screen);

-       mh = dc.font.height + 4;
-
-       win = XCreateWindow(dpy, root, mx, my, mw, mh, 0,
+       mh = dc.font.height + 2;
+       win = XCreateWindow(dpy, root, 0,
+                       bottom ? DisplayHeight(dpy, screen) - mh : 0, mw, mh, 0,

                        DefaultDepth(dpy, screen), CopyFromParent,
                        DefaultVisual(dpy, screen),
                        CWOverrideRedirect | CWBackPixmap | CWEventMask, &wa);
                        DefaultDepth(dpy, screen), CopyFromParent,
                        DefaultVisual(dpy, screen),
                        CWOverrideRedirect | CWBackPixmap | CWEventMask, &wa);

-       XDefineCursor(dpy, win, XCreateFontCursor(dpy, XC_xterm));
-

        /* pixmap */
        dc.drawable = XCreatePixmap(dpy, root, mw, mh, DefaultDepth(dpy, screen));
        dc.gc = XCreateGC(dpy, root, 0, 0);
        /* pixmap */
        dc.drawable = XCreatePixmap(dpy, root, mw, mh, DefaultDepth(dpy, screen));
        dc.gc = XCreateGC(dpy, root, 0, 0);

-
+       XSetLineAttributes(dpy, dc.gc, 1, LineSolid, CapButt, JoinMiter);
+       if(!dc.font.set)
+               XSetFont(dpy, dc.gc, dc.font.xfont->fid);

        if(maxname)
                cmdw = textw(maxname);
        if(cmdw > mw / 3)
                cmdw = mw / 3;
        if(maxname)
                cmdw = textw(maxname);
        if(cmdw > mw / 3)
                cmdw = mw / 3;

-
+       if(prompt)
+               promptw = textw(prompt);
+       if(promptw > mw / 5)
+               promptw = mw / 5;

        text[0] = 0;
        match(text);
        XMapRaised(dpy, win);
        text[0] = 0;
        match(text);
        XMapRaised(dpy, win);


@@ -352,8 +522,10 @@ main(int argc, char *argv[])
        XSync(dpy, False);
 
        /* main event loop */
        XSync(dpy, False);
 
        /* main event loop */

-       while(!done && !XNextEvent(dpy, &ev)) {
+       while(running && !XNextEvent(dpy, &ev))

                switch (ev.type) {
                switch (ev.type) {

+               default:        /* ignore all crap */
+                       break;

                case KeyPress:
                        kpress(&ev.xkey);
                        break;
                case KeyPress:
                        kpress(&ev.xkey);
                        break;


@@ -361,16 +533,23 @@ main(int argc, char *argv[])
                        if(ev.xexpose.count == 0)
                                drawmenu();
                        break;
                        if(ev.xexpose.count == 0)
                                drawmenu();
                        break;

-               default:
-                       break;

                }
                }

-       }

 
 

-       XUngrabKeyboard(dpy, CurrentTime);
+       /* cleanup */
+       while(allitems) {
+               itm = allitems->next;
+               free(allitems->text);
+               free(allitems);
+               allitems = itm;
+       }
+       if(dc.font.set)
+               XFreeFontSet(dpy, dc.font.set);
+       else
+               XFreeFont(dpy, dc.font.xfont);

        XFreePixmap(dpy, dc.drawable);
        XFreeGC(dpy, dc.gc);
        XDestroyWindow(dpy, win);
        XFreePixmap(dpy, dc.drawable);
        XFreeGC(dpy, dc.gc);
        XDestroyWindow(dpy, win);

+       XUngrabKeyboard(dpy, CurrentTime);

        XCloseDisplay(dpy);
        XCloseDisplay(dpy);

-

        return ret;
 }
        return ret;
 }