[PATCH weston 1/3] xwm: fix extra break

Derek Foreman derekf at osg.samsung.com
Tue Apr 7 10:12:13 PDT 2015


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 <derekf at osg.samsung.com>
---
 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 7018c92..83ebfae 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,
-- 
2.1.4



More information about the wayland-devel mailing list