[PATCH libinput] tools: libinput-debug-events stop variable needs volatile

Peter Hutterer peter.hutterer at who-t.net
Wed Mar 14 22:54:20 UTC 2018


On Tue, Mar 13, 2018 at 09:28:22PM +0100, Peter Seiderer wrote:
> The stop variable set in the signal handler needs
> volatile (and use the defined sig_atomic_t instead
> of unsigned int).
> 
> Signed-off-by: Peter Seiderer <ps.report at gmx.net>
> ---
>  tools/libinput-debug-events.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/libinput-debug-events.c b/tools/libinput-debug-events.c
> index 33f51d0e..32782c21 100644
> --- a/tools/libinput-debug-events.c
> +++ b/tools/libinput-debug-events.c
> @@ -48,7 +48,7 @@ static const uint32_t screen_width = 100;
>  static const uint32_t screen_height = 100;
>  static struct tools_options options;
>  static bool show_keycodes;
> -static unsigned int stop = 0;
> +static volatile sig_atomic_t stop = 0;
>  static bool be_quiet = false;
>  
>  #define printq(...) ({ if (!be_quiet)  printf(__VA_ARGS__); })
> -- 
> 2.16.2

pushed, thanks

Cheers,
   Peter


More information about the wayland-devel mailing list