[PATCH weston] eventdemo: use %u for uint32_t printing
Benoit Gschwind
gschwind at gnu-log.net
Fri May 13 11:50:17 UTC 2016
Hello Pekka,
Look sane.
Reviewed-by: Benoit Gschwind <gschwind at gnu-log.net>
On 13/05/2016 13:38, Pekka Paalanen wrote:
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
>
> I was confused why timestamp was printed negative. This fixes it, and
> others while at it.
>
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> ---
> clients/eventdemo.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/clients/eventdemo.c b/clients/eventdemo.c
> index 64b3d01..f04e39b 100644
> --- a/clients/eventdemo.c
> +++ b/clients/eventdemo.c
> @@ -206,7 +206,7 @@ key_handler(struct window *window, struct input *input, uint32_t time,
> if (!log_key)
> return;
>
> - printf("key key: %d, unicode: %d, state: %s, modifiers: 0x%x\n",
> + printf("key key: %u, unicode: %u, state: %s, modifiers: 0x%x\n",
> key, unicode,
> (state == WL_KEYBOARD_KEY_STATE_PRESSED) ? "pressed" :
> "released",
> @@ -235,7 +235,7 @@ button_handler(struct widget *widget, struct input *input, uint32_t time,
> e->print_pointer_frame = true;
>
> input_get_position(input, &x, &y);
> - printf("button time: %d, button: %d, state: %s, x: %d, y: %d\n",
> + printf("button time: %u, button: %u, state: %s, x: %d, y: %d\n",
> time, button,
> (state == WL_POINTER_BUTTON_STATE_PRESSED) ? "pressed" :
> "released",
> @@ -262,7 +262,7 @@ axis_handler(struct widget *widget, struct input *input, uint32_t time,
>
> e->print_pointer_frame = true;
>
> - printf("axis time: %d, axis: %s, value: %f\n",
> + printf("axis time: %u, axis: %s, value: %f\n",
> time,
> axis == WL_POINTER_AXIS_VERTICAL_SCROLL ? "vertical" :
> "horizontal",
> @@ -322,7 +322,7 @@ axis_stop_handler(struct widget *widget, struct input *input,
> return;
>
> e->print_pointer_frame = true;
> - printf("axis stop time: %d, axis: %s\n",
> + printf("axis stop time: %u, axis: %s\n",
> time,
> axis == WL_POINTER_AXIS_VERTICAL_SCROLL ? "vertical" :
> "horizontal");
> @@ -338,7 +338,7 @@ axis_discrete_handler(struct widget *widget, struct input *input,
> return;
>
> e->print_pointer_frame = true;
> - printf("axis discrete axis: %d value: %d\n", axis, discrete);
> + printf("axis discrete axis: %u value: %d\n", axis, discrete);
> }
>
> /**
> @@ -361,7 +361,7 @@ motion_handler(struct widget *widget, struct input *input, uint32_t time,
> struct eventdemo *e = data;
>
> if (log_motion) {
> - printf("motion time: %d, x: %f, y: %f\n", time, x, y);
> + printf("motion time: %u, x: %f, y: %f\n", time, x, y);
> e->print_pointer_frame = true;
> }
>
>
More information about the wayland-devel
mailing list