Hi BigJJ,
I just looked at your patch.
Well... it's not entirely obvious to me what your intention is with variable
Runspon, or why you need it at all, given all the other information in the various local and member variables.
But I think the big question is if the code in
should (all) be in this place. It looks as if some of it should have been in the above
Code: Select all
if (Keys & PCK_MoveForward) { ... }
? (But remember that the meaning of
Runspon is not really clear to me.)
Also, certainly unintended and cause of problems is that you test
Runspon!=3 and in the body of the same condition
Runspon==3:
Code: Select all
if (Keys & PCK_Walk)
{
if (Runspon!=3)
{
WishVelocity=scale(WishVelocity, 2.0);
if (Runspon==3) Runspon=4;
else Runspon=1;
}
}
Please review these pieces of code, your problem is certainly somewhere in this (flawed) logic.
Also, some general tips:
- Configure your editor to use 4 spaces, not TABs for indentation. As you can see in your patch, the TABs break the alignment.
- Update your code to the latest revision in SVN trunk (it's just a matter of entering svn update at the command line). Your patch is missing fixes for bugs that have been addressed in the meanwhile.