]> git.sur5r.net Git - i3/i3/blobdiff - src/util.c
Don’t rely on libxcb-wm any longer, as it got removed in libxcb 0.3.4
[i3/i3] / src / util.c
index 8de4d665853a02fe0f2f2c98e61141723813bdfa..276f76f92be4cbd6fb6ac3321c19ea884e2c3376 100644 (file)
@@ -19,6 +19,8 @@
 #include <assert.h>
 #include <iconv.h>
 
+#include <xcb/xcb_icccm.h>
+
 #include "i3.h"
 #include "data.h"
 #include "table.h"
@@ -27,6 +29,8 @@
 #include "xcb.h"
 
 static iconv_t conversion_descriptor = 0;
+struct keyvalue_table_head by_parent = TAILQ_HEAD_INITIALIZER(by_parent);
+struct keyvalue_table_head by_child = TAILQ_HEAD_INITIALIZER(by_child);
 
 int min(int a, int b) {
         return (a < b ? a : b);
@@ -94,6 +98,44 @@ char *sstrdup(const char *str) {
         return result;
 }
 
+/*
+ * The table_* functions emulate the behaviour of libxcb-wm, which in libxcb 0.3.4 suddenly
+ * vanished. Great.
+ *
+ */
+bool table_put(struct keyvalue_table_head *head, uint32_t key, void *value) {
+        struct keyvalue_element *element = scalloc(sizeof(struct keyvalue_element));
+        element->key = key;
+        element->value = value;
+
+        TAILQ_INSERT_TAIL(head, element, elements);
+        return true;
+}
+
+void *table_remove(struct keyvalue_table_head *head, uint32_t key) {
+        struct keyvalue_element *element;
+
+        TAILQ_FOREACH(element, head, elements)
+                if (element->key == key) {
+                        void *value = element->value;
+                        TAILQ_REMOVE(head, element, elements);
+                        free(element);
+                        return value;
+                }
+
+        return NULL;
+}
+
+void *table_get(struct keyvalue_table_head *head, uint32_t key) {
+        struct keyvalue_element *element;
+
+        TAILQ_FOREACH(element, head, elements)
+                if (element->key == key)
+                        return element->value;
+
+        return NULL;
+}
+
 /*
  * Starts the given application by passing it through a shell. We use double fork
  * to avoid zombie processes. As the started application’s parent exits (immediately),
@@ -145,11 +187,10 @@ void check_error(xcb_connection_t *conn, xcb_void_cookie_t cookie, char *err_mes
  * returned. It has to be freed when done.
  *
  */
-char *convert_utf8_to_ucs2(const char *input, int *real_strlen) {
+char *convert_utf8_to_ucs2(char *input, int *real_strlen) {
        size_t input_size = strlen(input) + 1;
        /* UCS-2 consumes exactly two bytes for each glyph */
        int buffer_size = input_size * 2;
-       printf("reserving %d bytes\n", buffer_size);
 
        char *buffer = smalloc(buffer_size);
        size_t output_size = buffer_size;
@@ -169,11 +210,11 @@ char *convert_utf8_to_ucs2(const char *input, int *real_strlen) {
        iconv(conversion_descriptor, NULL, NULL, NULL, NULL);
 
        /* Convert our text */
-       int rc = iconv(conversion_descriptor, &input, &input_size, &output, &output_size);
+       int rc = iconv(conversion_descriptor, (void*)&input, &input_size, &output, &output_size);
         if (rc == (size_t)-1) {
-               fprintf(stderr, "Converting to UCS-2 failed\n");
-                perror("erron\n");
-               exit(1);
+                perror("Converting to UCS-2 failed");
+               *real_strlen = 0;
+                return NULL;
        }
 
        *real_strlen = ((buffer_size - output_size) / 2) - 1;
@@ -189,6 +230,8 @@ char *convert_utf8_to_ucs2(const char *input, int *real_strlen) {
 void remove_client_from_container(xcb_connection_t *conn, Client *client, Container *container) {
         CIRCLEQ_REMOVE(&(container->clients), client, clients);
 
+        SLIST_REMOVE(&(container->workspace->focus_stack), client, Client, focus_clients);
+
         /* If the container will be empty now and is in stacking mode, we need to
            unmap the stack_win */
         if (CIRCLEQ_EMPTY(&(container->clients)) && container->mode == MODE_STACK) {
@@ -198,23 +241,39 @@ void remove_client_from_container(xcb_connection_t *conn, Client *client, Contai
         }
 }
 
+/*
+ * Returns the client which comes next in focus stack (= was selected before) for
+ * the given container, optionally excluding the given client.
+ *
+ */
+Client *get_last_focused_client(xcb_connection_t *conn, Container *container, Client *exclude) {
+        Client *current;
+        SLIST_FOREACH(current, &(container->workspace->focus_stack), focus_clients)
+                if ((current->container == container) && ((exclude == NULL) || (current != exclude)))
+                        return current;
+        return NULL;
+}
+
 /*
  * Sets the given client as focused by updating the data structures correctly,
  * updating the X input focus and finally re-decorating both windows (to signalize
  * the user the new focus situation)
  *
  */
-void set_focus(xcb_connection_t *conn, Client *client) {
-        /* The dock window cannot be focused */
-        /* TODO: does this play well with dzen2’s popup menus? or do we just need to set the input
-           focus but not update our internal structures? */
+void set_focus(xcb_connection_t *conn, Client *client, bool set_anyways) {
+        /* The dock window cannot be focused, but enter notifies are still handled correctly */
         if (client->dock)
                 return;
 
         /* Store the old client */
         Client *old_client = CUR_CELL->currently_focused;
 
-        /* TODO: check if the focus needs to be changed at all */
+        /* Check if the focus needs to be changed at all */
+        if (!set_anyways && (old_client == client)) {
+                LOG("old_client == client, not changing focus\n");
+                return;
+        }
+
         /* Store current_row/current_col */
         c_ws->current_row = current_row;
         c_ws->current_col = current_col;
@@ -231,9 +290,17 @@ void set_focus(xcb_connection_t *conn, Client *client) {
         xcb_set_input_focus(conn, XCB_INPUT_FOCUS_POINTER_ROOT, client->child, XCB_CURRENT_TIME);
         //xcb_warp_pointer(conn, XCB_NONE, client->child, 0, 0, 0, 0, 10, 10);
 
+        /* Get the client which was last focused in this particular container, it may be a different
+           one than old_client */
+        Client *last_focused = get_last_focused_client(conn, client->container, NULL);
+
+        /* If it is the same one as old_client, we save us the unnecessary redecorate */
+        if ((last_focused != NULL) && (last_focused != old_client))
+                redecorate_window(conn, last_focused);
+
         /* If we’re in stacking mode, this renders the container to update changes in the title
            bars and to raise the focused client */
-        if ((old_client != NULL) && (old_client != client))
+        if ((old_client != NULL) && (old_client != client) && !old_client->dock)
                 redecorate_window(conn, old_client);
 
         SLIST_REMOVE(&(client->container->workspace->focus_stack), client, Client, focus_clients);
@@ -285,7 +352,8 @@ void switch_layout_mode(xcb_connection_t *conn, Container *container, int mode)
 
                 /* We want to know when… */
                 mask |= XCB_CW_EVENT_MASK;
-                values[1] =     XCB_EVENT_MASK_BUTTON_PRESS |   /* …mouse is pressed */
+                values[1] =     XCB_EVENT_MASK_ENTER_WINDOW |   /* …mouse is moved into our window */
+                                XCB_EVENT_MASK_BUTTON_PRESS |   /* …mouse is pressed */
                                 XCB_EVENT_MASK_EXPOSURE;        /* …our window needs to be redrawn */
 
                 struct Stack_Window *stack_win = &(container->stack_win);
@@ -336,11 +404,13 @@ void toggle_fullscreen(xcb_connection_t *conn, Client *client) {
 
         Workspace *workspace = client->container->workspace;
 
-        workspace->fullscreen_client = (client->fullscreen ? NULL : client);
-
-        client->fullscreen = !client->fullscreen;
-
-        if (client->fullscreen) {
+        if (!client->fullscreen) {
+                if (workspace->fullscreen_client != NULL) {
+                        LOG("Not entering fullscreen mode, there already is a fullscreen client.\n");
+                        return;
+                }
+                client->fullscreen = true;
+                workspace->fullscreen_client = client;
                 LOG("Entering fullscreen mode...\n");
                 /* We just entered fullscreen mode, let’s configure the window */
                  uint32_t mask = XCB_CONFIG_WINDOW_X |
@@ -366,14 +436,70 @@ void toggle_fullscreen(xcb_connection_t *conn, Client *client) {
                 values[0] = XCB_STACK_MODE_ABOVE;
                 xcb_configure_window(conn, client->frame, XCB_CONFIG_WINDOW_STACK_MODE, values);
 
+                Rect child_rect = workspace->rect;
+                child_rect.x = child_rect.y = 0;
+                fake_configure_notify(conn, child_rect, client->child);
         } else {
                 LOG("leaving fullscreen mode\n");
+                client->fullscreen = false;
+                workspace->fullscreen_client = NULL;
                 /* Because the coordinates of the window haven’t changed, it would not be
                    re-configured if we don’t set the following flag */
                 client->force_reconfigure = true;
-                /* We left fullscreen mode, redraw the container */
-                render_container(conn, client->container);
+                /* We left fullscreen mode, redraw the whole layout to ensure enternotify events are disabled */
+                render_layout(conn);
+        }
+
+        xcb_flush(conn);
+}
+
+/*
+ * Returns true if the client supports the given protocol atom (like WM_DELETE_WINDOW)
+ *
+ */
+static bool client_supports_protocol(xcb_connection_t *conn, Client *client, xcb_atom_t atom) {
+        xcb_get_property_cookie_t cookie;
+        xcb_get_wm_protocols_reply_t protocols;
+        bool result = false;
+
+        cookie = xcb_get_wm_protocols_unchecked(conn, client->child, atoms[WM_PROTOCOLS]);
+        if (xcb_get_wm_protocols_reply(conn, cookie, &protocols, NULL) != 1)
+                return false;
+
+        /* Check if the client’s protocols have the requested atom set */
+        for (uint32_t i = 0; i < protocols.atoms_len; i++)
+                if (protocols.atoms[i] == atom)
+                        result = true;
+
+        xcb_get_wm_protocols_reply_wipe(&protocols);
+
+        return result;
+}
+
+/*
+ * Kills the given window using WM_DELETE_WINDOW or xcb_kill_window
+ *
+ */
+void kill_window(xcb_connection_t *conn, Client *window) {
+        /* If the client does not support WM_DELETE_WINDOW, we kill it the hard way */
+        if (!client_supports_protocol(conn, window, atoms[WM_DELETE_WINDOW])) {
+                LOG("Killing window the hard way\n");
+                xcb_kill_client(conn, window->child);
+                return;
         }
 
+        xcb_client_message_event_t ev;
+
+        memset(&ev, 0, sizeof(xcb_client_message_event_t));
+
+        ev.response_type = XCB_CLIENT_MESSAGE;
+        ev.window = window->child;
+        ev.type = atoms[WM_PROTOCOLS];
+        ev.format = 32;
+        ev.data.data32[0] = atoms[WM_DELETE_WINDOW];
+        ev.data.data32[1] = XCB_CURRENT_TIME;
+
+        LOG("Sending WM_DELETE to the client\n");
+        xcb_send_event(conn, false, window->child, XCB_EVENT_MASK_NO_EVENT, (char*)&ev);
         xcb_flush(conn);
 }