[PATCH 01/15] Fix pressure->motion property format

walter harms wharms at bfs.de
Fri Jun 10 01:07:39 PDT 2011


The code says INT32. Something to worry about ?

Why you are not using (u)int32_t ?

re,
 wh


Am 09.06.2011 21:57, schrieb Daniel Stone:
> From: Derek Foreman <derek.foreman at collabora.co.uk>
> 
> UINT32, not float.
> 
> Signed-off-by: Derek Foreman <derek.foreman at collabora.co.uk>
> Reviewed-by: Daniel Stone <daniel at fooishbar.org>
> ---
>  src/properties.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/properties.c b/src/properties.c
> index 299a444..8370c0b 100644
> --- a/src/properties.c
> +++ b/src/properties.c
> @@ -646,11 +646,11 @@ SetProperty(DeviceIntPtr dev, Atom property, XIPropertyValuePtr prop,
>          para->coasting_friction = coast_speeds[1];
>      } else if (property == prop_pressuremotion)
>      {
> -        float *press;
> -        if (prop->size != 2 || prop->format != 32 || prop->type != float_type)
> +        INT32 *press;
> +        if (prop->size != 2 || prop->format != 32 || prop->type != XA_INTEGER)
>              return BadMatch;
>  
> -        press = (float*)prop->data;
> +        press = (INT32*)prop->data;
>          if (press[0] > press[1])
>              return BadValue;
>  


More information about the xorg-devel mailing list