From 8b88f00117c6c251efd720aedb97afd55fdf76f6 Mon Sep 17 00:00:00 2001 From: Orestis Floros Date: Sat, 30 Mar 2019 13:31:59 +0200 Subject: [PATCH] window.c: Reduce code in window_update_* functions --- src/window.c | 40 ++++++++++++---------------------------- 1 file changed, 12 insertions(+), 28 deletions(-) diff --git a/src/window.c b/src/window.c index 799488c6..8c3ae850 100644 --- a/src/window.c +++ b/src/window.c @@ -51,14 +51,10 @@ void window_update_class(i3Window *win, xcb_get_property_reply_t *prop, bool bef 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); + } } /* @@ -92,14 +88,10 @@ void window_update_name(i3Window *win, xcb_get_property_reply_t *prop, bool befo win->uses_net_wm_name = true; - if (before_mgmt) { - free(prop); - return; - } - - run_assignments(win); - free(prop); + if (!before_mgmt) { + run_assignments(win); + } } /* @@ -141,14 +133,10 @@ void window_update_name_legacy(i3Window *win, xcb_get_property_reply_t *prop, bo win->name_x_changed = true; - if (before_mgmt) { - free(prop); - return; - } - - run_assignments(win); - free(prop); + if (!before_mgmt) { + run_assignments(win); + } } /* @@ -244,14 +232,10 @@ void window_update_role(i3Window *win, xcb_get_property_reply_t *prop, bool befo 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); + } } /* -- 2.39.2