Re: [PATCH] weston-launch: Drop redundant exit()

2015-07-10 Thread Bryce Harrington
On Wed, Jul 01, 2015 at 11:52:28AM +0200, Marek Chalupa wrote: > Reviewed-by: Marek Chalupa Thanks, pushed as commit 430aee1c23a641cf3227bcde08419458ef2ea252 > > On Sat, Jun 20, 2015 at 12:47 AM, Bryce Harrington > wrote: > > > error(1, ...) already will exit, per man page: "If status has a

Re: [PATCH] weston-launch: Drop redundant exit()

2015-07-01 Thread Marek Chalupa
Reviewed-by: Marek Chalupa On Sat, Jun 20, 2015 at 12:47 AM, Bryce Harrington wrote: > error(1, ...) already will exit, per man page: "If status has a nonzero > value, then error() calls exit(3) to terminate the program using the > given value as the exit status." So exit(EXIT_FAILURE) is nev

[PATCH] weston-launch: Drop redundant exit()

2015-06-19 Thread Bryce Harrington
error(1, ...) already will exit, per man page: "If status has a nonzero value, then error() calls exit(3) to terminate the program using the given value as the exit status." So exit(EXIT_FAILURE) is never reached. The EXIT_FAILURE macro is guaranteed to be non-zero. Typically it's just 1, but on