Re: [PATCH weston] xwayland: Remove pointless newline in X11 lock

2016-11-17 Thread Daniel Stone
Hi, On 16 November 2016 at 21:59, Eric Engestrom wrote: > On Wednesday, 2016-11-16 16:37:59 +, Daniel Stone wrote: >> The newline was always chopped off by snprintf: pid is only 11 > > Don't you mean 10? > Reviewed-by: Eric Engestrom Apparently I got this somewhat backwards whilst flipping

Re: [PATCH weston] xwayland: Remove pointless newline in X11 lock

2016-11-16 Thread Eric Engestrom
On Wednesday, 2016-11-16 16:37:59 +, Daniel Stone wrote: > The newline was always chopped off by snprintf: pid is only 11 Don't you mean 10? Reviewed-by: Eric Engestrom > characters long, and we were asking to print 11 characters plus the > terminating NULL. Hence snprintf would always chop

[PATCH weston] xwayland: Remove pointless newline in X11 lock

2016-11-16 Thread Daniel Stone
The newline was always chopped off by snprintf: pid is only 11 characters long, and we were asking to print 11 characters plus the terminating NULL. Hence snprintf would always chop the newline anyway. Signed-off-by: Daniel Stone Cc: Pekka Paalanen --- xwayland/launcher.c | 2 +- 1 file changed