[PATCH weston v2] removed extra sigemptyset and updated documentation

Kristian Høgsberg hoegsberg at gmail.com
Wed Mar 27 12:32:34 PDT 2013


On Wed, Mar 27, 2013 at 03:45:39PM +0100, Peter Maatman wrote:
> From: blackwolf12333 <blackwolf12333 at gmail.com>

Thanks, this is a useful feature.  However, following up to a patch
review typically means taking the suggested changes and adding them
into the original patch, using something like git commit --amend -a to
update the original patch.  At the end of the day, there should just
be one patch for me to commit that incorporates the feedback from the
review.

thanks,
Kristian

>  src/compositor.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/src/compositor.c b/src/compositor.c
> index ec3d0e3..5d4c9b2 100644
> --- a/src/compositor.c
> +++ b/src/compositor.c
> @@ -3401,8 +3401,6 @@ catch_signals()
>  	action.sa_sigaction = on_caught_signal;
>  	sigemptyset(&action.sa_mask);
>  	sigaction(SIGSEGV, &action, NULL);
> -	
> -	sigemptyset(&action.sa_mask);
>  	sigaction(SIGABRT, &action, NULL);
>  }
>  
> -- 
> 1.8.2
> 
> _______________________________________________
> 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