[airstrike] BUG: engine/math.c: vnormalize() is not vector normalization

Erik Auerswald auerswal at unix-ag.uni-kl.de
Sat Feb 18 11:21:06 EST 2006


Hi,

On Sat, Feb 18, 2006 at 05:08:50PM +0100, Ulf Ekstrom wrote:
> > while working on the seeker AI I found out that vnormalize() from
> > engine/maths.c is not vector normalization.
> 
> Hi! It's really strange that we didn't discover this bug earlier;
> thanks for the find! It should be sqrt(t) of course. Maybe now I can
> make the physics better as well. Please commit the fix, I will also
> test a bit for things that break.

I'll soon commit my proposed change, i.e. renaming vnormalize to
vnormalize2 and introducing a correct vnormalize function that should be
used from now on.

Of course the old code could be modified to use the correct function (at
least the used constants need to be changed), but I won't do that for
now.

Erik



More information about the airstrike mailing list