-#undef I3__FILE__
-#define I3__FILE__ "window.c"
/*
* vim:ts=4:sw=4:expandtab
*
*/
#include "all.h"
+/*
+ * Frees an i3Window and all its members.
+ *
+ */
+void window_free(i3Window *win) {
+ FREE(win->class_class);
+ FREE(win->class_instance);
+ i3string_free(win->name);
+ FREE(win->ran_assignments);
+ FREE(win);
+}
+
/*
* Updates the WM_CLASS (consisting of the class and instance) for the
* given window.
LOG("WM_CLASS changed to %s (instance), %s (class)\n",
win->class_instance, win->class_class);
- if (before_mgmt) {
- free(prop);
- return;
- }
-
- run_assignments(win);
-
free(prop);
+ if (!before_mgmt) {
+ run_assignments(win);
+ }
}
/*
}
i3string_free(win->name);
- win->name = i3string_from_utf8_with_length(xcb_get_property_value(prop),
- xcb_get_property_value_length(prop));
- if (win->title_format != NULL)
- ewmh_update_visible_name(win->id, i3string_as_utf8(window_parse_title_format(win)));
+
+ /* Truncate the name at the first zero byte. See #3515. */
+ const int len = xcb_get_property_value_length(prop);
+ char *name = sstrndup(xcb_get_property_value(prop), len);
+ win->name = i3string_from_utf8(name);
+ free(name);
+
+ Con *con = con_by_window_id(win->id);
+ if (con != NULL && con->title_format != NULL) {
+ i3String *name = con_parse_title_format(con);
+ ewmh_update_visible_name(win->id, i3string_as_utf8(name));
+ I3STRING_FREE(name);
+ }
win->name_x_changed = true;
LOG("_NET_WM_NAME changed to \"%s\"\n", i3string_as_utf8(win->name));
win->uses_net_wm_name = true;
- if (before_mgmt) {
- free(prop);
- return;
- }
-
- run_assignments(win);
-
free(prop);
+ if (!before_mgmt) {
+ run_assignments(win);
+ }
}
/*
}
i3string_free(win->name);
- win->name = i3string_from_utf8_with_length(xcb_get_property_value(prop),
- xcb_get_property_value_length(prop));
- if (win->title_format != NULL)
- ewmh_update_visible_name(win->id, i3string_as_utf8(window_parse_title_format(win)));
+ const int len = xcb_get_property_value_length(prop);
+ char *name = sstrndup(xcb_get_property_value(prop), len);
+ win->name = i3string_from_utf8(name);
+ free(name);
+
+ Con *con = con_by_window_id(win->id);
+ if (con != NULL && con->title_format != NULL) {
+ i3String *name = con_parse_title_format(con);
+ ewmh_update_visible_name(win->id, i3string_as_utf8(name));
+ I3STRING_FREE(name);
+ }
LOG("WM_NAME changed to \"%s\"\n", i3string_as_utf8(win->name));
LOG("Using legacy window title. Note that in order to get Unicode window "
win->name_x_changed = true;
- if (before_mgmt) {
- free(prop);
- return;
- }
-
- run_assignments(win);
-
free(prop);
+ if (!before_mgmt) {
+ run_assignments(win);
+ }
}
/*
win->role = new_role;
LOG("WM_WINDOW_ROLE changed to \"%s\"\n", win->role);
- if (before_mgmt) {
- free(prop);
- return;
- }
-
- run_assignments(win);
-
free(prop);
+ if (!before_mgmt) {
+ run_assignments(win);
+ }
}
/*
*/
void window_update_type(i3Window *window, xcb_get_property_reply_t *reply) {
xcb_atom_t new_type = xcb_get_preferred_window_type(reply);
+ free(reply);
if (new_type == XCB_NONE) {
DLOG("cannot read _NET_WM_WINDOW_TYPE from window.\n");
return;
run_assignments(window);
}
+/*
+ * Updates the WM_NORMAL_HINTS
+ *
+ */
+bool window_update_normal_hints(i3Window *win, xcb_get_property_reply_t *reply, xcb_get_geometry_reply_t *geom) {
+ bool changed = false;
+ xcb_size_hints_t size_hints;
+
+ /* If the hints were already in this event, use them, if not, request them */
+ bool success;
+ if (reply != NULL) {
+ success = xcb_icccm_get_wm_size_hints_from_reply(&size_hints, reply);
+ } else {
+ success = xcb_icccm_get_wm_normal_hints_reply(conn, xcb_icccm_get_wm_normal_hints_unchecked(conn, win->id), &size_hints, NULL);
+ }
+ if (!success) {
+ DLOG("Could not get WM_NORMAL_HINTS\n");
+ return false;
+ }
+
+#define ASSIGN_IF_CHANGED(original, new) \
+ do { \
+ if (original != new) { \
+ original = new; \
+ changed = true; \
+ } \
+ } while (0)
+
+ if ((size_hints.flags & XCB_ICCCM_SIZE_HINT_P_MIN_SIZE)) {
+ DLOG("Minimum size: %d (width) x %d (height)\n", size_hints.min_width, size_hints.min_height);
+
+ ASSIGN_IF_CHANGED(win->min_width, size_hints.min_width);
+ ASSIGN_IF_CHANGED(win->min_height, size_hints.min_height);
+ }
+
+ if ((size_hints.flags & XCB_ICCCM_SIZE_HINT_P_MAX_SIZE)) {
+ DLOG("Maximum size: %d (width) x %d (height)\n", size_hints.max_width, size_hints.max_height);
+
+ int max_width = max(0, size_hints.max_width);
+ int max_height = max(0, size_hints.max_height);
+
+ ASSIGN_IF_CHANGED(win->max_width, max_width);
+ ASSIGN_IF_CHANGED(win->max_height, max_height);
+ } else {
+ DLOG("Clearing maximum size \n");
+
+ ASSIGN_IF_CHANGED(win->max_width, 0);
+ ASSIGN_IF_CHANGED(win->max_height, 0);
+ }
+
+ if ((size_hints.flags & XCB_ICCCM_SIZE_HINT_P_RESIZE_INC)) {
+ DLOG("Size increments: %d (width) x %d (height)\n", size_hints.width_inc, size_hints.height_inc);
+
+ if (size_hints.width_inc > 0 && size_hints.width_inc < 0xFFFF) {
+ ASSIGN_IF_CHANGED(win->width_increment, size_hints.width_inc);
+ } else {
+ ASSIGN_IF_CHANGED(win->width_increment, 0);
+ }
+
+ if (size_hints.height_inc > 0 && size_hints.height_inc < 0xFFFF) {
+ ASSIGN_IF_CHANGED(win->height_increment, size_hints.height_inc);
+ } else {
+ ASSIGN_IF_CHANGED(win->height_increment, 0);
+ }
+ } else {
+ DLOG("Clearing size increments\n");
+
+ ASSIGN_IF_CHANGED(win->width_increment, 0);
+ ASSIGN_IF_CHANGED(win->height_increment, 0);
+ }
+
+ /* The base width / height is the desired size of the window. */
+ if (size_hints.flags & XCB_ICCCM_SIZE_HINT_BASE_SIZE &&
+ (win->base_width >= 0) && (win->base_height >= 0)) {
+ DLOG("Base size: %d (width) x %d (height)\n", size_hints.base_width, size_hints.base_height);
+
+ ASSIGN_IF_CHANGED(win->base_width, size_hints.base_width);
+ ASSIGN_IF_CHANGED(win->base_height, size_hints.base_height);
+ } else {
+ DLOG("Clearing base size\n");
+
+ ASSIGN_IF_CHANGED(win->base_width, 0);
+ ASSIGN_IF_CHANGED(win->base_height, 0);
+ }
+
+ if (geom != NULL &&
+ (size_hints.flags & XCB_ICCCM_SIZE_HINT_US_POSITION || size_hints.flags & XCB_ICCCM_SIZE_HINT_P_POSITION) &&
+ (size_hints.flags & XCB_ICCCM_SIZE_HINT_US_SIZE || size_hints.flags & XCB_ICCCM_SIZE_HINT_P_SIZE)) {
+ DLOG("Setting geometry x=%d y=%d w=%d h=%d\n", size_hints.x, size_hints.y, size_hints.width, size_hints.height);
+ geom->x = size_hints.x;
+ geom->y = size_hints.y;
+ geom->width = size_hints.width;
+ geom->height = size_hints.height;
+ }
+
+ /* If no aspect ratio was set or if it was invalid, we ignore the hints */
+ if (size_hints.flags & XCB_ICCCM_SIZE_HINT_P_ASPECT &&
+ (size_hints.min_aspect_num >= 0) && (size_hints.min_aspect_den > 0) &&
+ (size_hints.max_aspect_num >= 0) && (size_hints.max_aspect_den > 0)) {
+ /* Convert numerator/denominator to a double */
+ double min_aspect = (double)size_hints.min_aspect_num / size_hints.min_aspect_den;
+ double max_aspect = (double)size_hints.max_aspect_num / size_hints.max_aspect_den;
+ DLOG("Aspect ratio set: minimum %f, maximum %f\n", min_aspect, max_aspect);
+ if (fabs(win->min_aspect_ratio - min_aspect) > DBL_EPSILON) {
+ win->min_aspect_ratio = min_aspect;
+ changed = true;
+ }
+ if (fabs(win->max_aspect_ratio - max_aspect) > DBL_EPSILON) {
+ win->max_aspect_ratio = max_aspect;
+ changed = true;
+ }
+ } else {
+ DLOG("Clearing aspect ratios\n");
+
+ ASSIGN_IF_CHANGED(win->min_aspect_ratio, 0.0);
+ ASSIGN_IF_CHANGED(win->max_aspect_ratio, 0.0);
+ }
+
+ return changed;
+}
+
/*
* Updates the WM_HINTS (we only care about the input focus handling part).
*
*
*/
void window_update_motif_hints(i3Window *win, xcb_get_property_reply_t *prop, border_style_t *motif_border_style) {
-/* This implementation simply mirrors Gnome's Metacity. Official
+ /* This implementation simply mirrors Gnome's Metacity. Official
* documentation of this hint is nowhere to be found.
* For more information see:
* https://people.gnome.org/~tthurman/docs/metacity/xprops_8h-source.html
- * http://stackoverflow.com/questions/13787553/detect-if-a-x11-window-has-decorations
+ * https://stackoverflow.com/questions/13787553/detect-if-a-x11-window-has-decorations
*/
+#define MWM_HINTS_FLAGS_FIELD 0
+#define MWM_HINTS_DECORATIONS_FIELD 2
+
#define MWM_HINTS_DECORATIONS (1 << 1)
#define MWM_DECOR_ALL (1 << 0)
#define MWM_DECOR_BORDER (1 << 1)
return;
}
- /* The property consists of an array of 5 uint64_t's. The first value is a bit
- * mask of what properties the hint will specify. We are only interested in
- * MWM_HINTS_DECORATIONS because it indicates that the second value of the
+ /* The property consists of an array of 5 uint32_t's. The first value is a
+ * bit mask of what properties the hint will specify. We are only interested
+ * in MWM_HINTS_DECORATIONS because it indicates that the third value of the
* array tells us which decorations the window should have, each flag being
- * a particular decoration. */
- uint64_t *motif_hints = (uint64_t *)xcb_get_property_value(prop);
-
- if (motif_border_style != NULL && motif_hints[0] & MWM_HINTS_DECORATIONS) {
- if (motif_hints[1] & MWM_DECOR_ALL || motif_hints[1] & MWM_DECOR_TITLE)
+ * a particular decoration. Notice that X11 (Xlib) often mentions 32-bit
+ * fields which in reality are implemented using unsigned long variables
+ * (64-bits long on amd64 for example). On the other hand,
+ * xcb_get_property_value() behaves strictly according to documentation,
+ * i.e. returns 32-bit data fields. */
+ uint32_t *motif_hints = (uint32_t *)xcb_get_property_value(prop);
+
+ if (motif_border_style != NULL &&
+ motif_hints[MWM_HINTS_FLAGS_FIELD] & MWM_HINTS_DECORATIONS) {
+ if (motif_hints[MWM_HINTS_DECORATIONS_FIELD] & MWM_DECOR_ALL ||
+ motif_hints[MWM_HINTS_DECORATIONS_FIELD] & MWM_DECOR_TITLE)
*motif_border_style = BS_NORMAL;
- else if (motif_hints[1] & MWM_DECOR_BORDER)
+ else if (motif_hints[MWM_HINTS_DECORATIONS_FIELD] & MWM_DECOR_BORDER)
*motif_border_style = BS_PIXEL;
else
*motif_border_style = BS_NONE;
FREE(prop);
+#undef MWM_HINTS_FLAGS_FIELD
+#undef MWM_HINTS_DECORATIONS_FIELD
#undef MWM_HINTS_DECORATIONS
#undef MWM_DECOR_ALL
#undef MWM_DECOR_BORDER
#undef MWM_DECOR_TITLE
}
-
-/*
- * Returns the window title considering the current title format.
- * If no format is set, this will simply return the window's name.
- *
- */
-i3String *window_parse_title_format(i3Window *win) {
- /* We need to ensure that we only escape the window title if pango
- * is used by the current font. */
- const bool is_markup = font_is_pango();
-
- char *format = win->title_format;
- if (format == NULL)
- return i3string_copy(win->name);
-
- /* We initialize these lazily so we only escape them if really necessary. */
- const char *escaped_title = NULL;
- const char *escaped_class = NULL;
- const char *escaped_instance = NULL;
-
- /* We have to first iterate over the string to see how much buffer space
- * we need to allocate. */
- int buffer_len = strlen(format) + 1;
- for (char *walk = format; *walk != '\0'; walk++) {
- if (STARTS_WITH(walk, "%title")) {
- if (escaped_title == NULL)
- escaped_title = win->name == NULL ? "" : i3string_as_utf8(is_markup ? i3string_escape_markup(win->name) : win->name);
-
- buffer_len = buffer_len - strlen("%title") + strlen(escaped_title);
- walk += strlen("%title") - 1;
- } else if (STARTS_WITH(walk, "%class")) {
- if (escaped_class == NULL)
- escaped_class = is_markup ? g_markup_escape_text(win->class_class, -1) : win->class_class;
-
- buffer_len = buffer_len - strlen("%class") + strlen(escaped_class);
- walk += strlen("%class") - 1;
- } else if (STARTS_WITH(walk, "%instance")) {
- if (escaped_instance == NULL)
- escaped_instance = is_markup ? g_markup_escape_text(win->class_instance, -1) : win->class_instance;
-
- buffer_len = buffer_len - strlen("%instance") + strlen(escaped_instance);
- walk += strlen("%instance") - 1;
- }
- }
-
- /* Now we can parse the format string. */
- char buffer[buffer_len];
- char *outwalk = buffer;
- for (char *walk = format; *walk != '\0'; walk++) {
- if (*walk != '%') {
- *(outwalk++) = *walk;
- continue;
- }
-
- if (STARTS_WITH(walk + 1, "title")) {
- outwalk += sprintf(outwalk, "%s", escaped_title);
- walk += strlen("title");
- } else if (STARTS_WITH(walk + 1, "class")) {
- outwalk += sprintf(outwalk, "%s", escaped_class);
- walk += strlen("class");
- } else if (STARTS_WITH(walk + 1, "instance")) {
- outwalk += sprintf(outwalk, "%s", escaped_instance);
- walk += strlen("instance");
- } else {
- *(outwalk++) = *walk;
- }
- }
- *outwalk = '\0';
-
- i3String *formatted = i3string_from_utf8(buffer);
- i3string_set_markup(formatted, is_markup);
- return formatted;
-}
projects / i3 / i3 / blobdiff