[PATCH weston] eventdemo: use %u for uint32_t printing

Pekka Paalanen ppaalanen at gmail.com
Mon May 16 12:04:35 UTC 2016


On Fri, 13 May 2016 13:50:17 +0200
Benoit Gschwind <gschwind at gnu-log.net> wrote:

> Hello Pekka,
> 
> Look sane.
> 
> Reviewed-by: Benoit Gschwind <gschwind at gnu-log.net>

And pushed:
   0baffb0..901ac32  master -> master


Thanks,
pq

> 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;
> >  	}
> >  
> >   

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20160516/85e9ca1f/attachment.sig>


More information about the wayland-devel mailing list