X-Git-Url: https://git.sur5r.net/?a=blobdiff_plain;f=src%2Fxcb.c;h=900840a20a419b19832680d4d9a9c20195227065;hb=1994eea5da1c83f35e5793ef6ca72a91d09eb824;hp=5f3b22abd1c03aaea4ccf923d5095c3e49103d4c;hpb=af0b33bcc377a35274ccb4ab4aef90bb68486a34;p=i3%2Fi3 diff --git a/src/xcb.c b/src/xcb.c index 5f3b22ab..900840a2 100644 --- a/src/xcb.c +++ b/src/xcb.c @@ -1,379 +1,355 @@ /* - * vim:ts=8:expandtab + * vim:ts=4:sw=4:expandtab * * i3 - an improved dynamic tiling window manager - * - * © 2009 Michael Stapelberg and contributors - * - * See file LICENSE for license information. + * © 2009 Michael Stapelberg and contributors (see also: LICENSE) * * xcb.c: Helper functions for easier usage of XCB * */ -#include -#include -#include -#include - -#include -#include - -#include "i3.h" -#include "util.h" -#include "xcb.h" -#include "log.h" +#include "all.h" -TAILQ_HEAD(cached_fonts_head, Font) cached_fonts = TAILQ_HEAD_INITIALIZER(cached_fonts); unsigned int xcb_numlock_mask; /* - * Loads a font for usage, getting its height. This function is used very often, so it - * maintains a cache. + * Convenience wrapper around xcb_create_window which takes care of depth, generating an ID and checking + * for errors. * */ -i3Font *load_font(xcb_connection_t *conn, const char *pattern) { - /* Check if we got the font cached */ - i3Font *font; - TAILQ_FOREACH(font, &cached_fonts, fonts) - if (strcmp(font->pattern, pattern) == 0) - return font; - - i3Font *new = smalloc(sizeof(i3Font)); - xcb_void_cookie_t font_cookie; - xcb_list_fonts_with_info_cookie_t info_cookie; - - /* Send all our requests first */ - new->id = xcb_generate_id(conn); - font_cookie = xcb_open_font_checked(conn, new->id, strlen(pattern), pattern); - info_cookie = xcb_list_fonts_with_info(conn, 1, strlen(pattern), pattern); - - check_error(conn, font_cookie, "Could not open font"); - - /* Get information (height/name) for this font */ - xcb_list_fonts_with_info_reply_t *reply = xcb_list_fonts_with_info_reply(conn, info_cookie, NULL); - exit_if_null(reply, "Could not load font \"%s\"\n", pattern); - - if (asprintf(&(new->name), "%.*s", xcb_list_fonts_with_info_name_length(reply), - xcb_list_fonts_with_info_name(reply)) == -1) - die("asprintf() failed\n"); - new->pattern = sstrdup(pattern); - new->height = reply->font_ascent + reply->font_descent; - - /* Insert into cache */ - TAILQ_INSERT_TAIL(&cached_fonts, new, fonts); - - return new; +xcb_window_t create_window(xcb_connection_t *conn, Rect dims, + uint16_t depth, xcb_visualid_t visual, uint16_t window_class, + enum xcursor_cursor_t cursor, bool map, uint32_t mask, uint32_t *values) { + xcb_window_t result = xcb_generate_id(conn); + + /* If the window class is XCB_WINDOW_CLASS_INPUT_ONLY, we copy depth and + * visual id from the parent window. */ + if (window_class == XCB_WINDOW_CLASS_INPUT_ONLY) { + depth = XCB_COPY_FROM_PARENT; + visual = XCB_COPY_FROM_PARENT; + } + + xcb_create_window(conn, + depth, + result, /* the window id */ + root, /* parent == root */ + dims.x, dims.y, dims.width, dims.height, /* dimensions */ + 0, /* border = 0, we draw our own */ + window_class, + visual, + mask, + values); + + /* Set the cursor */ + if (xcursor_supported) { + mask = XCB_CW_CURSOR; + values[0] = xcursor_get_cursor(cursor); + xcb_change_window_attributes(conn, result, mask, values); + } else { + xcb_cursor_t cursor_id = xcb_generate_id(conn); + i3Font cursor_font = load_font("cursor", false); + int xcb_cursor = xcursor_get_xcb_cursor(cursor); + xcb_create_glyph_cursor(conn, cursor_id, cursor_font.specific.xcb.id, + cursor_font.specific.xcb.id, xcb_cursor, xcb_cursor + 1, 0, 0, 0, + 65535, 65535, 65535); + xcb_change_window_attributes(conn, result, XCB_CW_CURSOR, &cursor_id); + xcb_free_cursor(conn, cursor_id); + } + + /* Map the window (= make it visible) */ + if (map) + xcb_map_window(conn, result); + + return result; } /* - * Returns the colorpixel to use for the given hex color (think of HTML). - * - * The hex_color has to start with #, for example #FF00FF. + * Draws a line from x,y to to_x,to_y using the given color * - * NOTE that get_colorpixel() does _NOT_ check the given color code for validity. - * This has to be done by the caller. + */ +void xcb_draw_line(xcb_connection_t *conn, xcb_drawable_t drawable, xcb_gcontext_t gc, + uint32_t colorpixel, uint32_t x, uint32_t y, uint32_t to_x, uint32_t to_y) { + xcb_change_gc(conn, gc, XCB_GC_FOREGROUND, (uint32_t[]){colorpixel}); + xcb_poly_line(conn, XCB_COORD_MODE_ORIGIN, drawable, gc, 2, + (xcb_point_t[]){{x, y}, {to_x, to_y}}); +} + +/* + * Draws a rectangle from x,y with width,height using the given color * */ -uint32_t get_colorpixel(xcb_connection_t *conn, char *hex) { - char strgroups[3][3] = {{hex[1], hex[2], '\0'}, - {hex[3], hex[4], '\0'}, - {hex[5], hex[6], '\0'}}; - uint32_t rgb16[3] = {(strtol(strgroups[0], NULL, 16)), - (strtol(strgroups[1], NULL, 16)), - (strtol(strgroups[2], NULL, 16))}; - - return (rgb16[0] << 16) + (rgb16[1] << 8) + rgb16[2]; +void xcb_draw_rect(xcb_connection_t *conn, xcb_drawable_t drawable, xcb_gcontext_t gc, + uint32_t colorpixel, uint32_t x, uint32_t y, uint32_t width, uint32_t height) { + xcb_change_gc(conn, gc, XCB_GC_FOREGROUND, (uint32_t[]){colorpixel}); + xcb_rectangle_t rect = {x, y, width, height}; + xcb_poly_fill_rectangle(conn, drawable, gc, 1, &rect); } /* - * Convenience wrapper around xcb_create_window which takes care of depth, generating an ID and checking - * for errors. + * Generates a configure_notify_event with absolute coordinates (relative to the X root + * window, not to the client’s frame) for the given client. * */ -xcb_window_t create_window(xcb_connection_t *conn, Rect dims, uint16_t window_class, int cursor, - bool map, uint32_t mask, uint32_t *values) { - xcb_window_t root = xcb_setup_roots_iterator(xcb_get_setup(conn)).data->root; - xcb_window_t result = xcb_generate_id(conn); - xcb_cursor_t cursor_id = xcb_generate_id(conn); +void fake_absolute_configure_notify(Con *con) { + xcb_rectangle_t absolute; + if (con->window == NULL) + return; - /* If the window class is XCB_WINDOW_CLASS_INPUT_ONLY, depth has to be 0 */ - uint16_t depth = (window_class == XCB_WINDOW_CLASS_INPUT_ONLY ? 0 : XCB_COPY_FROM_PARENT); + absolute.x = con->rect.x + con->window_rect.x; + absolute.y = con->rect.y + con->window_rect.y; + absolute.width = con->window_rect.width; + absolute.height = con->window_rect.height; - /* Use the default cursor (left pointer) */ - if (cursor > -1) { - i3Font *cursor_font = load_font(conn, "cursor"); - xcb_create_glyph_cursor(conn, cursor_id, cursor_font->id, cursor_font->id, - XCB_CURSOR_LEFT_PTR, XCB_CURSOR_LEFT_PTR + 1, - 0, 0, 0, 65535, 65535, 65535); - } + DLOG("fake rect = (%d, %d, %d, %d)\n", absolute.x, absolute.y, absolute.width, absolute.height); - xcb_create_window(conn, - depth, - result, /* the window id */ - root, /* parent == root */ - dims.x, dims.y, dims.width, dims.height, /* dimensions */ - 0, /* border = 0, we draw our own */ - window_class, - XCB_WINDOW_CLASS_COPY_FROM_PARENT, /* copy visual from parent */ - mask, - values); - - if (cursor > -1) - xcb_change_window_attributes(conn, result, XCB_CW_CURSOR, &cursor_id); - - /* Map the window (= make it visible) */ - if (map) - xcb_map_window(conn, result); - - return result; + fake_configure_notify(conn, absolute, con->window->id, con->border_width); } /* - * Changes a single value in the graphic context (so one doesn’t have to define an array of values) + * Sends the WM_TAKE_FOCUS ClientMessage to the given window * */ -void xcb_change_gc_single(xcb_connection_t *conn, xcb_gcontext_t gc, uint32_t mask, uint32_t value) { - xcb_change_gc(conn, gc, mask, &value); +void send_take_focus(xcb_window_t window, xcb_timestamp_t timestamp) { + /* Every X11 event is 32 bytes long. Therefore, XCB will copy 32 bytes. + * In order to properly initialize these bytes, we allocate 32 bytes even + * though we only need less for an xcb_configure_notify_event_t */ + void *event = scalloc(32, 1); + xcb_client_message_event_t *ev = event; + + ev->response_type = XCB_CLIENT_MESSAGE; + ev->window = window; + ev->type = A_WM_PROTOCOLS; + ev->format = 32; + ev->data.data32[0] = A_WM_TAKE_FOCUS; + ev->data.data32[1] = timestamp; + + DLOG("Sending WM_TAKE_FOCUS to the client\n"); + xcb_send_event(conn, false, window, XCB_EVENT_MASK_NO_EVENT, (char *)ev); + free(event); } /* - * Draws a line from x,y to to_x,to_y using the given color + * Raises the given window (typically client->frame) above all other windows * */ -void xcb_draw_line(xcb_connection_t *conn, xcb_drawable_t drawable, xcb_gcontext_t gc, - uint32_t colorpixel, uint32_t x, uint32_t y, uint32_t to_x, uint32_t to_y) { - xcb_change_gc_single(conn, gc, XCB_GC_FOREGROUND, colorpixel); - xcb_point_t points[] = {{x, y}, {to_x, to_y}}; - xcb_poly_line(conn, XCB_COORD_MODE_ORIGIN, drawable, gc, 2, points); +void xcb_raise_window(xcb_connection_t *conn, xcb_window_t window) { + uint32_t values[] = {XCB_STACK_MODE_ABOVE}; + xcb_configure_window(conn, window, XCB_CONFIG_WINDOW_STACK_MODE, values); } /* - * Draws a rectangle from x,y with width,height using the given color + * Configures the given window to have the size/position specified by given rect * */ -void xcb_draw_rect(xcb_connection_t *conn, xcb_drawable_t drawable, xcb_gcontext_t gc, - uint32_t colorpixel, uint32_t x, uint32_t y, uint32_t width, uint32_t height) { - xcb_change_gc_single(conn, gc, XCB_GC_FOREGROUND, colorpixel); - xcb_rectangle_t rect = {x, y, width, height}; - xcb_poly_fill_rectangle(conn, drawable, gc, 1, &rect); +void xcb_set_window_rect(xcb_connection_t *conn, xcb_window_t window, Rect r) { + xcb_void_cookie_t cookie; + cookie = xcb_configure_window(conn, window, + XCB_CONFIG_WINDOW_X | + XCB_CONFIG_WINDOW_Y | + XCB_CONFIG_WINDOW_WIDTH | + XCB_CONFIG_WINDOW_HEIGHT, + &(r.x)); + /* ignore events which are generated because we configured a window */ + add_ignore_event(cookie.sequence, -1); } /* - * Generates a configure_notify event and sends it to the given window - * Applications need this to think they’ve configured themselves correctly. - * The truth is, however, that we will manage them. + * Returns the first supported _NET_WM_WINDOW_TYPE atom. * */ -void fake_configure_notify(xcb_connection_t *conn, Rect r, xcb_window_t window) { - xcb_configure_notify_event_t generated_event; - - generated_event.event = window; - generated_event.window = window; - generated_event.response_type = XCB_CONFIGURE_NOTIFY; - - generated_event.x = r.x; - generated_event.y = r.y; - generated_event.width = r.width; - generated_event.height = r.height; - - generated_event.border_width = 0; - generated_event.above_sibling = XCB_NONE; - generated_event.override_redirect = false; +xcb_atom_t xcb_get_preferred_window_type(xcb_get_property_reply_t *reply) { + if (reply == NULL || xcb_get_property_value_length(reply) == 0) + return XCB_NONE; + + xcb_atom_t *atoms; + if ((atoms = xcb_get_property_value(reply)) == NULL) + return XCB_NONE; + + for (int i = 0; i < xcb_get_property_value_length(reply) / (reply->format / 8); i++) { + if (atoms[i] == A__NET_WM_WINDOW_TYPE_NORMAL || + atoms[i] == A__NET_WM_WINDOW_TYPE_DIALOG || + atoms[i] == A__NET_WM_WINDOW_TYPE_UTILITY || + atoms[i] == A__NET_WM_WINDOW_TYPE_TOOLBAR || + atoms[i] == A__NET_WM_WINDOW_TYPE_SPLASH || + atoms[i] == A__NET_WM_WINDOW_TYPE_MENU || + atoms[i] == A__NET_WM_WINDOW_TYPE_DROPDOWN_MENU || + atoms[i] == A__NET_WM_WINDOW_TYPE_POPUP_MENU || + atoms[i] == A__NET_WM_WINDOW_TYPE_TOOLTIP || + atoms[i] == A__NET_WM_WINDOW_TYPE_NOTIFICATION) { + return atoms[i]; + } + } - xcb_send_event(conn, false, window, XCB_EVENT_MASK_STRUCTURE_NOTIFY, (char*)&generated_event); - xcb_flush(conn); + return XCB_NONE; } /* - * Generates a configure_notify_event with absolute coordinates (relative to the X root - * window, not to the client’s frame) for the given client. + * Returns true if the given reply contains the given atom. * */ -void fake_absolute_configure_notify(xcb_connection_t *conn, Client *client) { - Rect absolute; +bool xcb_reply_contains_atom(xcb_get_property_reply_t *prop, xcb_atom_t atom) { + if (prop == NULL || xcb_get_property_value_length(prop) == 0) + return false; + + xcb_atom_t *atoms; + if ((atoms = xcb_get_property_value(prop)) == NULL) + return false; - absolute.x = client->rect.x + client->child_rect.x; - absolute.y = client->rect.y + client->child_rect.y; - absolute.width = client->child_rect.width; - absolute.height = client->child_rect.height; + for (int i = 0; i < xcb_get_property_value_length(prop) / (prop->format / 8); i++) + if (atoms[i] == atom) + return true; - fake_configure_notify(conn, absolute, client->child); + return false; } -/* - * Finds out which modifier mask is the one for numlock, as the user may change this. +/** + * Moves the mouse pointer into the middle of rect. * */ -void xcb_get_numlock_mask(xcb_connection_t *conn) { - xcb_key_symbols_t *keysyms; - xcb_get_modifier_mapping_cookie_t cookie; - xcb_get_modifier_mapping_reply_t *reply; - xcb_keycode_t *modmap; - int mask, i; - const int masks[8] = { XCB_MOD_MASK_SHIFT, - XCB_MOD_MASK_LOCK, - XCB_MOD_MASK_CONTROL, - XCB_MOD_MASK_1, - XCB_MOD_MASK_2, - XCB_MOD_MASK_3, - XCB_MOD_MASK_4, - XCB_MOD_MASK_5 }; - - /* Request the modifier map */ - cookie = xcb_get_modifier_mapping_unchecked(conn); - - /* Get the keysymbols */ - keysyms = xcb_key_symbols_alloc(conn); - - if ((reply = xcb_get_modifier_mapping_reply(conn, cookie, NULL)) == NULL) { - xcb_key_symbols_free(keysyms); - return; - } +void xcb_warp_pointer_rect(xcb_connection_t *conn, Rect *rect) { + int mid_x = rect->x + (rect->width / 2); + int mid_y = rect->y + (rect->height / 2); - modmap = xcb_get_modifier_mapping_keycodes(reply); - - /* Get the keycode for numlock */ -#ifdef OLD_XCB_KEYSYMS_API - xcb_keycode_t numlock = xcb_key_symbols_get_keycode(keysyms, XCB_NUM_LOCK); -#else - /* For now, we only use the first keysymbol. */ - xcb_keycode_t *numlock_syms = xcb_key_symbols_get_keycode(keysyms, XCB_NUM_LOCK); - if (numlock_syms == NULL) - return; - xcb_keycode_t numlock = *numlock_syms; - free(numlock_syms); -#endif - - /* Check all modifiers (Mod1-Mod5, Shift, Control, Lock) */ - for (mask = 0; mask < 8; mask++) - for (i = 0; i < reply->keycodes_per_modifier; i++) - if (modmap[(mask * reply->keycodes_per_modifier) + i] == numlock) - xcb_numlock_mask = masks[mask]; - - xcb_key_symbols_free(keysyms); - free(reply); + LOG("warp pointer to: %d %d\n", mid_x, mid_y); + xcb_warp_pointer(conn, XCB_NONE, root, 0, 0, 0, 0, mid_x, mid_y); } /* - * Raises the given window (typically client->frame) above all other windows + * Set the cursor of the root window to the given cursor id. + * This function should only be used if xcursor_supported == false. + * Otherwise, use xcursor_set_root_cursor(). * */ -void xcb_raise_window(xcb_connection_t *conn, xcb_window_t window) { - uint32_t values[] = { XCB_STACK_MODE_ABOVE }; - xcb_configure_window(conn, window, XCB_CONFIG_WINDOW_STACK_MODE, values); +void xcb_set_root_cursor(int cursor) { + xcb_cursor_t cursor_id = xcb_generate_id(conn); + i3Font cursor_font = load_font("cursor", false); + int xcb_cursor = xcursor_get_xcb_cursor(cursor); + xcb_create_glyph_cursor(conn, cursor_id, cursor_font.specific.xcb.id, + cursor_font.specific.xcb.id, xcb_cursor, xcb_cursor + 1, 0, 0, 0, + 65535, 65535, 65535); + xcb_change_window_attributes(conn, root, XCB_CW_CURSOR, &cursor_id); + xcb_free_cursor(conn, cursor_id); + xcb_flush(conn); } /* - * - * Prepares the given Cached_Pixmap for usage (checks whether the size of the - * object this pixmap is related to (e.g. a window) has changed and re-creates - * the pixmap if so). + * Get depth of visual specified by visualid * */ -void cached_pixmap_prepare(xcb_connection_t *conn, struct Cached_Pixmap *pixmap) { - DLOG("preparing pixmap\n"); - - /* If the Rect did not change, the pixmap does not need to be recreated */ - if (memcmp(&(pixmap->rect), pixmap->referred_rect, sizeof(Rect)) == 0) - return; - - memcpy(&(pixmap->rect), pixmap->referred_rect, sizeof(Rect)); - - if (pixmap->id == 0 || pixmap->gc == 0) { - DLOG("Creating new pixmap...\n"); - pixmap->id = xcb_generate_id(conn); - pixmap->gc = xcb_generate_id(conn); - } else { - DLOG("Re-creating this pixmap...\n"); - xcb_free_gc(conn, pixmap->gc); - xcb_free_pixmap(conn, pixmap->id); +uint16_t get_visual_depth(xcb_visualid_t visual_id) { + xcb_depth_iterator_t depth_iter; + + depth_iter = xcb_screen_allowed_depths_iterator(root_screen); + for (; depth_iter.rem; xcb_depth_next(&depth_iter)) { + xcb_visualtype_iterator_t visual_iter; + + visual_iter = xcb_depth_visuals_iterator(depth_iter.data); + for (; visual_iter.rem; xcb_visualtype_next(&visual_iter)) { + if (visual_id == visual_iter.data->visual_id) { + return depth_iter.data->depth; + } } + } + return 0; +} - xcb_create_pixmap(conn, root_depth, pixmap->id, - pixmap->referred_drawable, pixmap->rect.width, pixmap->rect.height); - - xcb_create_gc(conn, pixmap->gc, pixmap->id, 0, 0); +/* + * Get visual type specified by visualid + * + */ +xcb_visualtype_t *get_visualtype_by_id(xcb_visualid_t visual_id) { + xcb_depth_iterator_t depth_iter; + + depth_iter = xcb_screen_allowed_depths_iterator(root_screen); + for (; depth_iter.rem; xcb_depth_next(&depth_iter)) { + xcb_visualtype_iterator_t visual_iter; + + visual_iter = xcb_depth_visuals_iterator(depth_iter.data); + for (; visual_iter.rem; xcb_visualtype_next(&visual_iter)) { + if (visual_id == visual_iter.data->visual_id) { + return visual_iter.data; + } + } + } + return 0; } /* - * Returns the xcb_charinfo_t for the given character (specified by row and - * column in the lookup table) if existing, otherwise the minimum bounds. + * Get visualid with specified depth * */ -static xcb_charinfo_t *get_charinfo(int col, int row, xcb_query_font_reply_t *font_info, - xcb_charinfo_t *table, bool dont_fallback) { - xcb_charinfo_t *result; - - /* Bounds checking */ - if (row < font_info->min_byte1 || row > font_info->max_byte1 || - col < font_info->min_char_or_byte2 || col > font_info->max_char_or_byte2) - return NULL; - - /* If we don’t have a table to lookup the infos per character, return the - * minimum bounds */ - if (table == NULL) - return &font_info->min_bounds; - - result = &table[((row - font_info->min_byte1) * - (font_info->max_char_or_byte2 - font_info->min_char_or_byte2 + 1)) + - (col - font_info->min_char_or_byte2)]; - - /* If the character has an entry in the table, return it */ - if (result->character_width != 0 || - (result->right_side_bearing | - result->left_side_bearing | - result->ascent | - result->descent) != 0) - return result; - - /* Otherwise, get the default character and return its charinfo */ - if (dont_fallback) - return NULL; - - return get_charinfo((font_info->default_char >> 8), - (font_info->default_char & 0xFF), - font_info, - table, - true); +xcb_visualid_t get_visualid_by_depth(uint16_t depth) { + xcb_depth_iterator_t depth_iter; + + depth_iter = xcb_screen_allowed_depths_iterator(root_screen); + for (; depth_iter.rem; xcb_depth_next(&depth_iter)) { + if (depth_iter.data->depth != depth) + continue; + + xcb_visualtype_iterator_t visual_iter; + + visual_iter = xcb_depth_visuals_iterator(depth_iter.data); + if (!visual_iter.rem) + continue; + return visual_iter.data->visual_id; + } + return 0; } /* - * Calculate the width of the given text (16-bit characters, UCS) with given - * real length (amount of glyphs) using the given font. + * Add an atom to a list of atoms the given property defines. + * This is useful, for example, for manipulating _NET_WM_STATE. * */ -int predict_text_width(xcb_connection_t *conn, const char *font_pattern, char *text, int length) { - xcb_query_font_reply_t *font_info; - xcb_charinfo_t *table; - xcb_generic_error_t *error; - int i, width = 0; - i3Font *font = load_font(conn, font_pattern); - - font_info = xcb_query_font_reply(conn, xcb_query_font(conn, font->id), &error); - if (error != NULL) { - fprintf(stderr, "ERROR: query font (X error code %d)\n", error->error_code); - /* We return the rather safe guess of 7 pixels, because a - * rendering error is better than a crash. Plus, the user will - * see the error on his stderr. */ - return 7; - } +void xcb_add_property_atom(xcb_connection_t *conn, xcb_window_t window, xcb_atom_t property, xcb_atom_t atom) { + xcb_change_property(conn, XCB_PROP_MODE_APPEND, window, property, XCB_ATOM_ATOM, 32, 1, (uint32_t[]){atom}); +} - /* If no per-char info is available for this font, we use the default */ - if (xcb_query_font_char_infos_length(font_info) == 0) { - DLOG("Falling back on default char_width of %d pixels\n", font_info->max_bounds.character_width); - return (font_info->max_bounds.character_width * length); +/* + * Remove an atom from a list of atoms the given property defines without + * removing any other potentially set atoms. This is useful, for example, for + * manipulating _NET_WM_STATE. + * + */ +void xcb_remove_property_atom(xcb_connection_t *conn, xcb_window_t window, xcb_atom_t property, xcb_atom_t atom) { + xcb_grab_server(conn); + + xcb_get_property_reply_t *reply = + xcb_get_property_reply(conn, + xcb_get_property(conn, false, window, property, XCB_GET_PROPERTY_TYPE_ANY, 0, 4096), NULL); + if (reply == NULL || xcb_get_property_value_length(reply) == 0) + goto release_grab; + xcb_atom_t *atoms = xcb_get_property_value(reply); + if (atoms == NULL) { + goto release_grab; + } + + { + int num = 0; + const int current_size = xcb_get_property_value_length(reply) / (reply->format / 8); + xcb_atom_t values[current_size]; + for (int i = 0; i < current_size; i++) { + if (atoms[i] != atom) + values[num++] = atoms[i]; } - table = xcb_query_font_char_infos(font_info); - - for (i = 0; i < 2 * length; i += 2) { - xcb_charinfo_t *info = get_charinfo(text[i+1], text[i], font_info, table, false); - if (info == NULL) - continue; - width += info->character_width; - } + xcb_change_property(conn, XCB_PROP_MODE_REPLACE, window, property, XCB_ATOM_ATOM, 32, num, values); + } - free(font_info); +release_grab: + FREE(reply); + xcb_ungrab_server(conn); +} - return width; +/* + * Grab the specified buttons on a window when managing it. + * + */ +void xcb_grab_buttons(xcb_connection_t *conn, xcb_window_t window, int *buttons) { + int i = 0; + while (buttons[i] > 0) { + xcb_grab_button(conn, false, window, XCB_EVENT_MASK_BUTTON_PRESS, XCB_GRAB_MODE_SYNC, + XCB_GRAB_MODE_ASYNC, root, XCB_NONE, buttons[i], XCB_BUTTON_MASK_ANY); + + i++; + } }