[PATCH] dix: rename shadowing loop variable in TouchResizeQueue

Chase Douglas chase.douglas at canonical.com
Tue May 15 08:29:27 PDT 2012


On 05/15/2012 03:28 AM, Peter Hutterer wrote:
> second 'i' shadows the function-wide one, rename to 'j'
> 
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
>  dix/touch.c |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/dix/touch.c b/dix/touch.c
> index eeb158e..95a7b06 100644
> --- a/dix/touch.c
> +++ b/dix/touch.c
> @@ -103,11 +103,11 @@ TouchResizeQueue(ClientPtr client, pointer closure)
>  
>          tmp = realloc(dev->last.touches, size * sizeof(*dev->last.touches));
>          if (tmp) {
> -            int i;
> +            int j;
>  
>              dev->last.touches = tmp;
> -            for (i = dev->last.num_touches; i < size; i++)
> -                TouchInitDDXTouchPoint(dev, &dev->last.touches[i]);
> +            for (j = dev->last.num_touches; j < size; j++)
> +                TouchInitDDXTouchPoint(dev, &dev->last.touches[j]);
>              dev->last.num_touches = size;
>          }

One warning down, 834729 to go!

Reviewed-by: Chase Douglas <chase.douglas at canonical.com>


More information about the xorg-devel mailing list