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

[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