<div dir="ltr"><div>New speed looks good to me!<br><br></div>(the Pink Plip approves)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 18, 2016 at 8:53 AM,  <span dir="ltr"><<a href="mailto:subversion@hamsterrepublic.com" target="_blank">subversion@hamsterrepublic.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">teeemcee<br>
2016-03-18 08:53:08 -0700 (Fri, 18 Mar 2016)<br>
1109<br>
Fix jump animations, properly this time.<br>
<br>
r3373, which was major bmod code cleanup, introduced a bug where when a hero<br>
jumps, they jump towards the right instead of to the left. It turns out that<br>
anim_relmove, which is only used for jumping, contained two major bugs, and<br>
James fixed one of them in r3373, causing the behaviour to change.<br>
In r4278 I fixed the jump problem by simply reverting the bugfix in r3373,<br>
clearly not understanding the code. The r4278 change was:<br>
<br>
--- a/bmod.bas<br>
+++ b/bmod.bas<br>
@@ -445,7 +445,7 @@ SUB battle_attack_anim_playback (BYREF attack AS AttackData, BYREF bat AS Battle<br>
    movestep.x = popw<br>
    movestep.y = popw<br>
    with bslot(ww)<br>
-    if movestep.x <> 0 then .xspeed = movedist.x / movestep.x<br>
+    if movedist.y <> 0 then .xspeed = movedist.x / movestep.x<br>
     if movestep.y <> 0 then .yspeed = movedist.y / movestep.y<br>
     .xmov = movedist.x<br>
     .ymov = movedist.y<br>
<br>
The other bug was that the wrong thing was assigned to .xmov/.ymov.<br>
<br>
As a result of this fix I had to change the jump animation, so I made it<br>
faster. I always thought it looked bad because it was too slow<br>
---<br>
U   wip/bmod.rbas<br>
U   wip/bmodsubs.bas<br>
_______________________________________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org">ohrrpgce@lists.motherhamster.org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org</a><br>
</blockquote></div><br></div>