[PATCH weston] weston-launch: cleanup - return failures in main with EXIT_FAILURE

Kristian Høgsberg hoegsberg at gmail.com
Tue Mar 19 11:41:21 PDT 2013


On Tue, Mar 12, 2013 at 02:36:52AM +0530, Siddharth Heroor wrote:
> 
> Signed-off-by: Siddharth Heroor <heroor at gmail.com>

Thanks, committed.

Kristian

>  src/weston-launch.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/src/weston-launch.c b/src/weston-launch.c
> index bc7f8a2..528c345 100644
> --- a/src/weston-launch.c
> +++ b/src/weston-launch.c
> @@ -542,7 +542,7 @@ main(int argc, char *argv[])
>  			break;
>  		case 'h':
>  			help("weston-launch");
> -			exit(1);
> +			exit(EXIT_FAILURE);
>  		}
>  	}
>  
> @@ -572,20 +572,20 @@ main(int argc, char *argv[])
>  		      " - or add yourself to the 'weston-launch' group.");
>  
>  	if (setup_tty(&wl, tty) < 0)
> -		return 1;
> +		exit(EXIT_FAILURE);
>  
>  	if (setup_pam(&wl) < 0)
> -		return 1;
> +		exit(EXIT_FAILURE);
>  
>  	wl.epollfd = epoll_create1(EPOLL_CLOEXEC);
>  	if (wl.epollfd < 0)
>  		error(1, errno, "epoll create failed");
>  
>  	if (setup_launcher_socket(&wl) < 0)
> -		return 1;
> +		exit(EXIT_FAILURE);
>  
>  	if (setup_signals(&wl) < 0)
> -		return 1;
> +		exit(EXIT_FAILURE);
>  
>  	switch ((wl.child = fork())) {
>  	case -1:
> -- 
> 1.8.1.3
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list