[PATCH] Fix Bug in DeadZone calculation

Kuba Ober kuba at mareimbrium.org
Wed Aug 15 13:26:02 CDT 2007


On Wednesday 15 August 2007, Andrew Talbot wrote:
> Kuba Ober wrote:
> >> You calculating center wrong:
> >> > +  ret = (props->lMax-props->lMin)/2;
> >>
> >> This won't work for min=1000 max=2000.
> >
> > But it does. Maybe you meant if min/max were switched? In such case
> >
> > ret = (props->lMax-props->lMin)/2;
> > if (props->lMax < props->lMin) ret = -ret;
> >
> > Cheers, Kuba
>
> If ret is meant to have a value that is half way between props->lMax and
> props->lMin, then I believe there should be a "+" between them, not a "-".
> In other words:
>
>     ret = (props->lMax + props->lMin) / 2;

Yeah, another case of one (me) reading what one wants to see, rather than 
reading what's really there. I feel hilarious, having suggested a non-fix 
(ret = -ret) ;)

Cheers, Kuba



More information about the wine-devel mailing list