On Fri, 2008-12-05 at 10:45 +1000, Peter Hutterer wrote:

>      if (num_valuators) {
> -        if ((num_valuators / 6) + 1 > MAX_VALUATOR_EVENTS)
> +        if (((num_valuators - 1) / 6) + 1 > MAX_VALUATOR_EVENTS)
>              num_valuators = MAX_VALUATOR_EVENTS;

This still looks wrong to me; why would you assign MAX_VALUATOR_EVENTS t
num_valuators? (no, I haven't looked at the surrounding code, just this
patch).

-- 
[EMAIL PROTECTED]

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
xorg mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/xorg

Reply via email to