diff options
| author | Derek Foreman <derekf@osg.samsung.com> | 2015-04-07 12:12:13 -0500 |
|---|---|---|
| committer | Pekka Paalanen <pekka.paalanen@collabora.co.uk> | 2015-04-08 15:21:17 +0300 |
| commit | b4deec6490cea7bb20abe91f0d3aca0cad8a64ae (patch) | |
| tree | e41b157329788777b282b41ad925ffb4cecd2e44 /xwayland | |
| parent | 8cb258766b03b6c364b79a45de9eedf99724617d (diff) | |
| download | weston-b4deec6490cea7bb20abe91f0d3aca0cad8a64ae.tar.gz | |
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 <derekf@osg.samsung.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Diffstat (limited to 'xwayland')
| -rw-r--r-- | xwayland/window-manager.c | 6 |
1 files 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, |