From b4deec6490cea7bb20abe91f0d3aca0cad8a64ae Mon Sep 17 00:00:00 2001 From: Derek Foreman Date: Tue, 7 Apr 2015 12:12:13 -0500 Subject: [PATCH] xwm: fix extra break The first break in TYPE_WM_PROTOCOLS was almost certainly intended to be nested within the if statement. Even if it wasn't, it makes sense there. Signed-off-by: Derek Foreman Reviewed-by: Pekka Paalanen --- xwayland/window-manager.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/xwayland/window-manager.c b/xwayland/window-manager.c index 7018c92f..83ebfaeb 100644 --- a/xwayland/window-manager.c +++ b/xwayland/window-manager.c @@ -459,10 +459,10 @@ weston_wm_window_read_properties(struct weston_wm_window *window) case TYPE_WM_PROTOCOLS: atom = xcb_get_property_value(reply); for (i = 0; i < reply->value_len; i++) - if (atom[i] == wm->atom.wm_delete_window) + if (atom[i] == wm->atom.wm_delete_window) { window->delete_window = 1; - break; - + break; + } break; case TYPE_WM_NORMAL_HINTS: memcpy(&window->size_hints,