Follow-up Comment #6, bug #6933 (project wormux):

OK. No duplicate calls and a good night for me :)

In PhysicalObj::NotifyMove(oldPos,newPos), i found a candidate bug. We start
the test with an initialized value of oldPos + offset :

...
  // Compute increments to move the object step by step from the old
  // to the new position.
  offset = (newPos - oldPos) / lg;

  // First iteration position.
  pos = oldPos + offset;

I think pos must be initialized with oldPos value :

  pos = oldPos;

Because, when we start moving, we are force to move at least one time ... and
maybe overlapse.

    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?6933>

_______________________________________________
  Message posté via/par Gna!
  http://gna.org/


_______________________________________________
Wormux-gna mailing list
[email protected]
https://mail.gna.org/listinfo/wormux-gna

Reply via email to