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

2015-04-08 Thread Pekka Paalanen
On Tue, 7 Apr 2015 12:12:13 -0500 Derek Foreman wrote: > 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 > --- > xwayland/window-manager.c | 6 +++--- > 1

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

2015-04-07 Thread Derek Foreman
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 --- xwayland/window-manager.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/xwayland/window-