]> Pileus Git - wmpus/blobdiff - sys-x11.c
Formatting and debugging updates
[wmpus] / sys-x11.c
index 08e80e986c1f3dcb42c4fc8b7e9c5af52e9ca590..dbab94820cc3dce31adeb1d1f9f310b25773768b 100644 (file)
--- a/sys-x11.c
+++ b/sys-x11.c
@@ -4,16 +4,22 @@
 
 #include <X11/Xlib.h>
 #include <X11/Xproto.h>
+#include <X11/Xatom.h>
 #include <X11/keysym.h>
 
 #include "util.h"
 #include "sys.h"
 #include "wm.h"
 
+#define BORDER 2
+
 /* Internal structures */
 struct win_sys {
        Window   xid;
        Display *dpy;
+       struct {
+               int left, right, top, bottom;
+       } strut;
 };
 
 typedef struct {
@@ -22,16 +28,21 @@ typedef struct {
 } keymap_t;
 
 typedef enum {
-       wm_proto, wm_focus, natoms
+       wm_proto, wm_focus, net_strut, natoms
 } atom_t;
 
-Atom atoms[natoms];
+typedef enum {
+       clr_focus, clr_unfocus, clr_urgent, ncolors
+} color_t;
 
 /* Global data */
-void *win_cache = NULL;
+static void *win_cache;
+static Atom atoms[natoms];
+static int (*xerrorxlib)(Display *, XErrorEvent *);
+static unsigned long colors[ncolors];
 
 /* Conversion functions */
-keymap_t key2sym[] = {
+static keymap_t key2sym[] = {
        {key_left    , XK_Left },
        {key_right   , XK_Right},
        {key_up      , XK_Up   },
@@ -55,7 +66,7 @@ keymap_t key2sym[] = {
 };
 
 /* - Modifiers */
-mod_t x2mod(unsigned int state, int up)
+static mod_t x2mod(unsigned int state, int up)
 {
        return (mod_t){
               .alt   = !!(state & Mod1Mask   ),
@@ -66,7 +77,7 @@ mod_t x2mod(unsigned int state, int up)
        };
 }
 
-unsigned int mod2x(mod_t mod)
+static unsigned int mod2x(mod_t mod)
 {
        return (mod.alt   ? Mod1Mask    : 0)
             | (mod.ctrl  ? ControlMask : 0)
@@ -75,35 +86,36 @@ unsigned int mod2x(mod_t mod)
 }
 
 /* - Keycodes */
-Key_t x2key(KeySym sym)
+static Key_t x2key(KeySym sym)
 {
        keymap_t *km = map_getr(key2sym,sym);
        return km ? km->key : sym;
 }
 
-KeySym key2x(Key_t key)
+static KeySym key2x(Key_t key)
 {
        keymap_t *km = map_get(key2sym,key);
        return km ? km->sym : key;
 }
 
-Key_t x2btn(int btn)
+static Key_t x2btn(int btn)
 {
        return btn + key_mouse0;
 }
 
-int btn2x(Key_t key)
+static int btn2x(Key_t key)
 {
        return key - key_mouse0;
 }
 
 /* - Pointers */
-ptr_t x2ptr(XEvent *_ev)
+static ptr_t x2ptr(XEvent *_ev)
 {
        XKeyEvent *ev = &_ev->xkey;
        return (ptr_t){ev->x, ev->y, ev->x_root, ev->y_root};
 }
-Window getfocus(win_t *root, XEvent *event)
+
+static Window getfocus(win_t *root, XEvent *event)
 {
        int revert;
        Window focus = PointerRoot;
@@ -116,8 +128,56 @@ Window getfocus(win_t *root, XEvent *event)
        return focus;
 }
 
+/* Helpers */
+static int add_strut(win_t *root, win_t *win)
+{
+       /* Get X11 strut data */
+       Atom ret_type;
+       int ret_size;
+       unsigned long ret_items, bytes_left;
+       unsigned char *xdata;
+       int status = XGetWindowProperty(win->sys->dpy, win->sys->xid,
+                       atoms[net_strut], 0L, 4L, False, XA_CARDINAL,
+                       &ret_type, &ret_size, &ret_items, &bytes_left, &xdata);
+       if (status != Success || ret_size != 32 || ret_items != 4)
+               return 0;
+
+       int left   = ((int*)xdata)[0];
+       int right  = ((int*)xdata)[1];
+       int top    = ((int*)xdata)[2];
+       int bottom = ((int*)xdata)[3];
+       if (left == 0 && right == 0 && top == 0 && bottom == 0)
+               return 0;
+
+       win->sys->strut.left   = left;
+       win->sys->strut.right  = right;
+       win->sys->strut.top    = top;
+       win->sys->strut.bottom = bottom;
+       root->x += left;
+       root->y += top;
+       root->w -= left+right;
+       root->h -= top+bottom;
+       return 1;
+}
+
+static int del_strut(win_t *root, win_t *win)
+{
+       int left   = win->sys->strut.left;
+       int right  = win->sys->strut.right;
+       int top    = win->sys->strut.top;
+       int bottom = win->sys->strut.bottom;
+       if (left == 0 && right == 0 && top == 0 && bottom == 0)
+               return 0;
+
+       root->x -= left;
+       root->y -= top;
+       root->w += left+right;
+       root->h += top+bottom;
+       return 1;
+}
+
 /* Window functions */
-win_t *win_new(Display *dpy, Window xid)
+static win_t *win_new(Display *dpy, Window xid)
 {
        XWindowAttributes attr;
        if (XGetWindowAttributes(dpy, xid, &attr))
@@ -131,11 +191,13 @@ win_t *win_new(Display *dpy, Window xid)
        win->sys      = new0(win_sys_t);
        win->sys->dpy = dpy;
        win->sys->xid = xid;
-       printf("%p = %p, %d\n", win, dpy, (int)xid);
+       printf("win_new: %p = %p, %d (%d,%d %dx%d)\n",
+                       win, dpy, (int)xid,
+                       win->x, win->y, win->w, win->h);
        return win;
 }
 
-int win_cmp(const void *_a, const void *_b)
+static int win_cmp(const void *_a, const void *_b)
 {
        const win_t *a = _a, *b = _b;
        if (a->sys->dpy < b->sys->dpy) return -1;
@@ -145,7 +207,7 @@ int win_cmp(const void *_a, const void *_b)
        return 0;
 }
 
-win_t *win_find(Display *dpy, Window xid, int create)
+static win_t *win_find(Display *dpy, Window xid, int create)
 {
        if (!dpy || !xid)
                return NULL;
@@ -160,14 +222,14 @@ win_t *win_find(Display *dpy, Window xid, int create)
        return new;
 }
 
-void win_remove(win_t *win)
+static void win_remove(win_t *win)
 {
        tdelete(win, &win_cache, win_cmp);
        free(win->sys);
        free(win);
 }
 
-int win_viewable(win_t *win)
+static int win_viewable(win_t *win)
 {
        XWindowAttributes attr;
        if (XGetWindowAttributes(win->sys->dpy, win->sys->xid, &attr))
@@ -176,8 +238,18 @@ int win_viewable(win_t *win)
                return True;
 }
 
+/* Drawing functions */
+unsigned long get_color(Display *dpy, const char *name)
+{
+       XColor color;
+       int screen = DefaultScreen(dpy);
+       Colormap cmap = DefaultColormap(dpy, screen);
+       XAllocNamedColor(dpy, cmap, name, &color, &color);
+       return color.pixel;
+}
+
 /* Callbacks */
-void process_event(int type, XEvent *ev, win_t *root)
+static void process_event(int type, XEvent *ev, win_t *root)
 {
        Display  *dpy = root->sys->dpy;
        win_t *win = NULL;
@@ -205,9 +277,18 @@ void process_event(int type, XEvent *ev, win_t *root)
                //printf("release: %d\n", type);
        }
        else if (type == ButtonPress) {
-               wm_handle_key(win, x2btn(ev->xbutton.button), mod, ptr);
-               XGrabPointer(dpy, ev->xbutton.root, True, PointerMotionMask|ButtonReleaseMask,
-                               GrabModeAsync, GrabModeAsync, None, None, CurrentTime);
+               if (wm_handle_key(win, x2btn(ev->xbutton.button), mod, ptr))
+                       XGrabPointer(dpy, ev->xbutton.root, True, PointerMotionMask|ButtonReleaseMask,
+                                       GrabModeAsync, GrabModeAsync, None, None, CurrentTime);
+               else {
+                       printf("resending event\n");
+                       XSendEvent(win->sys->dpy, ev->xbutton.window,    True,  NoEventMask, ev);
+                       XSendEvent(win->sys->dpy, ev->xbutton.window,    False, NoEventMask, ev);
+                       XSendEvent(win->sys->dpy, ev->xbutton.root,      True,  NoEventMask, ev);
+                       XSendEvent(win->sys->dpy, ev->xbutton.root,      False, NoEventMask, ev);
+                       XSendEvent(win->sys->dpy, ev->xbutton.subwindow, True,  NoEventMask, ev);
+                       XSendEvent(win->sys->dpy, ev->xbutton.subwindow, False, NoEventMask, ev);
+               }
        }
        else if (type == ButtonRelease) {
                XUngrabPointer(dpy, CurrentTime);
@@ -230,36 +311,46 @@ void process_event(int type, XEvent *ev, win_t *root)
                        wm_handle_key(win, key, MOD(), PTR());
        }
        else if (type == ConfigureNotify) {
-               //printf("configure: %d\n", type);
+               printf("configure: %d\n", type);
        }
        else if (type == MapNotify) {
-               //printf("map: %d\n", type);
-       }
-       else if (type == DestroyNotify) {
-               //printf("destory: %d\n", type);
-               if ((win = win_find(dpy,ev->xdestroywindow.window,0)))
-                       win_remove(win);
+               printf("map: %d\n", type);
        }
        else if (type == UnmapNotify) {
                //printf("unmap: %d\n", type);
-               if ((win = win_find(dpy,ev->xmap.window,0)))
-                       wm_remove(win);
+               if ((win = win_find(dpy,ev->xunmap.window,0))) {
+                       if (!del_strut(root, win))
+                               wm_remove(win);
+                       else
+                               wm_update();
+                       win_remove(win);
+               }
        }
        else if (type == ConfigureRequest) {
                XConfigureRequestEvent *cre = &ev->xconfigurerequest;
-               XWindowChanges wc = {
-                       .x     = cre->x,     .y      = cre->y,
-                       .width = cre->width, .height = cre->height,
-               };
                printf("configure_req: %d - %x, (0x%lx) %dx%d @ %d,%d\n",
                                type, (int)cre->window, cre->value_mask,
                                cre->height, cre->width, cre->x, cre->y);
-               XConfigureWindow(dpy, cre->window, cre->value_mask, &wc);
+               XConfigureWindow(dpy, cre->window, cre->value_mask, &(XWindowChanges){
+                       .x      = cre->x,
+                       .y      = cre->y,
+                       .width  = cre->width,
+                       .height = cre->height,
+               });
+
+               /* This seems necessasairy for, but causes flicker
+                * there could be a better way to do this */
+               if ((win = win_find(dpy,ev->xmaprequest.window,0)))
+                       sys_move(win, win->x, win->y, win->w, win->h);
        }
        else if (type == MapRequest) {
                printf("map_req: %d\n", type);
-               if ((win = win_find(dpy,ev->xmaprequest.window,1)))
-                       wm_insert(win);
+               if ((win = win_find(dpy,ev->xmaprequest.window,1))) {
+                       if (!add_strut(root, win))
+                               wm_insert(win);
+                       else
+                               wm_update();
+               }
                XMapWindow(dpy,ev->xmaprequest.window);
        }
        else {
@@ -267,9 +358,19 @@ void process_event(int type, XEvent *ev, win_t *root)
        }
 }
 
-int xwmerror(Display *dpy, XErrorEvent *err)
+static int xerror(Display *dpy, XErrorEvent *err)
 {
-       return error("another window manager is running?");
+       if (err->error_code == BadWindow ||
+           (err->request_code == X_SetInputFocus     && err->error_code == BadMatch   ) ||
+           (err->request_code == X_PolyText8         && err->error_code == BadDrawable) ||
+           (err->request_code == X_PolyFillRectangle && err->error_code == BadDrawable) ||
+           (err->request_code == X_PolySegment       && err->error_code == BadDrawable) ||
+           (err->request_code == X_ConfigureWindow   && err->error_code == BadMatch   ) ||
+           (err->request_code == X_GrabButton        && err->error_code == BadAccess  ) ||
+           (err->request_code == X_GrabKey           && err->error_code == BadAccess  ) ||
+           (err->request_code == X_CopyArea          && err->error_code == BadDrawable))
+               return 0;
+       return xerrorxlib(dpy, err);
 }
 
 /*****************
@@ -278,8 +379,10 @@ int xwmerror(Display *dpy, XErrorEvent *err)
 void sys_move(win_t *win, int x, int y, int w, int h)
 {
        //printf("sys_move: %p - %d,%d  %dx%d\n", win, x, y, w, h);
-       win->x = x; win->y = y;
-       win->w = w; win->h = h;
+       int b = 2*BORDER;
+       win->x = MAX(x,0);   win->y = MAX(y,0);
+       win->w = MAX(w,1+b); win->h = MAX(h,1+b);
+       w      = MAX(w-b,1); h      = MAX(h-b,1);
        XMoveResizeWindow(win->sys->dpy, win->sys->xid, x, y, w, h);
 
        /* Flush events, so moving window doesn't cuase re-focus
@@ -299,39 +402,55 @@ void sys_raise(win_t *win)
 void sys_focus(win_t *win)
 {
        printf("sys_focus: %p\n", win);
-       XEvent ev = {
+
+       /* Set border on focused window */
+       static win_t *last = NULL;
+       if (last)
+               XSetWindowBorder(last->sys->dpy, last->sys->xid, colors[clr_unfocus]);
+       XSetWindowBorder(win->sys->dpy, win->sys->xid, colors[clr_focus]);
+       XSetWindowBorderWidth(win->sys->dpy, win->sys->xid, BORDER);
+       last = win;
+
+       /* Set actual focus */
+       XSetInputFocus(win->sys->dpy, win->sys->xid,
+                       RevertToPointerRoot, CurrentTime);
+       XSendEvent(win->sys->dpy, win->sys->xid, False, NoEventMask, &(XEvent){
                .type                 = ClientMessage,
                .xclient.window       = win->sys->xid,
                .xclient.message_type = atoms[wm_proto],
                .xclient.format       = 32,
                .xclient.data.l[0]    = atoms[wm_focus],
                .xclient.data.l[1]    = CurrentTime,
-       };
-       XSetInputFocus(win->sys->dpy, win->sys->xid,
-                       RevertToPointerRoot, CurrentTime);
-       //XSetInputFocus(win->sys->dpy, PointerRoot,
-       //              RevertToPointerRoot, CurrentTime);
-       XSendEvent(win->sys->dpy, win->sys->xid, False, NoEventMask, &ev);
+       });
 }
 
 void sys_watch(win_t *win, Key_t key, mod_t mod)
 {
        //printf("sys_watch: %p - %x %hhx\n", win, key, mod);
+       XWindowAttributes attr;
+       XGetWindowAttributes(win->sys->dpy, win->sys->xid, &attr);
+       long mask = attr.your_event_mask;
        if (key_mouse0 <= key && key <= key_mouse7)
-               XGrabButton(win->sys->dpy, btn2x(key), mod2x(mod), win->sys->xid, True,
+               XGrabButton(win->sys->dpy, btn2x(key), mod2x(mod), win->sys->xid, False,
                                mod.up ? ButtonReleaseMask : ButtonPressMask,
                                GrabModeAsync, GrabModeAsync, None, None);
        else if (key == key_enter)
-               XSelectInput(win->sys->dpy, win->sys->xid, EnterWindowMask);
+               XSelectInput(win->sys->dpy, win->sys->xid, EnterWindowMask|mask);
        else if (key == key_leave)
-               XSelectInput(win->sys->dpy, win->sys->xid, LeaveWindowMask);
+               XSelectInput(win->sys->dpy, win->sys->xid, LeaveWindowMask|mask);
        else if (key == key_focus || key == key_unfocus)
-               XSelectInput(win->sys->dpy, win->sys->xid, FocusChangeMask);
+               XSelectInput(win->sys->dpy, win->sys->xid, FocusChangeMask|mask);
        else
                XGrabKey(win->sys->dpy, XKeysymToKeycode(win->sys->dpy, key2x(key)),
                                mod2x(mod), win->sys->xid, True, GrabModeAsync, GrabModeAsync);
 }
 
+void sys_unwatch(win_t *win, Key_t key, mod_t mod)
+{
+       if (key_mouse0 <= key && key <= key_mouse7)
+               XUngrabButton(win->sys->dpy, btn2x(key), mod2x(mod), win->sys->xid);
+}
+
 win_t *sys_init(void)
 {
        Display *dpy;
@@ -340,10 +459,16 @@ win_t *sys_init(void)
                error("Unable to get display");
        if (!(xid = DefaultRootWindow(dpy)))
                error("Unable to get root window");
-       atoms[wm_proto] = XInternAtom(dpy, "WM_PROTOCOLS",  False);
-       atoms[wm_focus] = XInternAtom(dpy, "WM_TAKE_FOCUS", False);
+       atoms[wm_proto]  = XInternAtom(dpy, "WM_PROTOCOLS",  False);
+       atoms[wm_focus]  = XInternAtom(dpy, "WM_TAKE_FOCUS", False);
+       atoms[net_strut] = XInternAtom(dpy, "_NET_WM_STRUT", False);
+       colors[clr_focus]   = get_color(dpy, "#a0a0ff");
+       colors[clr_unfocus] = get_color(dpy, "#101066");
+       colors[clr_urgent]  = get_color(dpy, "#ff0000");
+       printf("colors = #%06lx #%06lx #%06lx\n", colors[0], colors[1], colors[2]);
        XSelectInput(dpy, xid, SubstructureRedirectMask|SubstructureNotifyMask);
-       //XSetInputFocus(dpy, None, RevertToNone, CurrentTime);
+       XSetInputFocus(dpy, None, RevertToNone, CurrentTime);
+       xerrorxlib = XSetErrorHandler(xerror);
        return win_find(dpy, xid, 1);
 }
 
@@ -356,9 +481,10 @@ void sys_run(win_t *root)
                                &par, &xid, &kids, &nkids))
                for(int i = 0; i < nkids; i++) {
                        win_t *win = win_find(root->sys->dpy, kids[i], 1);
-                       if (win && win_viewable(win))
+                       if (win && win_viewable(win) && !add_strut(root,win))
                                wm_insert(win);
                }
+       wm_update(); // For struts
 
        /* Main loop */
        for(;;)