Update of patch #3682 (project wesnoth):

                  Status:                    None => In Progress            
             Assigned to:                    None => boucman                

    _______________________________________________________

Follow-up Comment #1:

ok, good job for all the bugs catched and the syntax upgrade, awesome

i get this error message when using your patch 

20130215 23:52:03 error general: Square bracket lengths do not match up:
units/human-outlaws/thief-idle-[7~12,11~8].png:[300,100*10]


the loop parameter is definitely not a missing feature. it should be similar
to progressive string in every aspect...



* you defined a does_not_change function for the new parameters but you don't
use it. does_not_change() is used to tell the animation engine that a unit
needs to be redrawn because it moved. without that the animtion engine will
leave the unit as is unless there is another reason to redraw it. a quick grep
around should allow you to fix this

* after that there is a remaining bug in the fencer standing wml, but it's
trivial to fix

I'll wait for an updated patch with all WML updated. please be carefull since
it's easy to get it wrong


quick note about looping anims. We don't have looping animations in wesnoth,
the only thing we have is animations that end and are restarted right away :)
i.e looping is handle at the animation choice level, not by the animation
itself

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?3682>

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


_______________________________________________
Wesnoth-bugs mailing list
[email protected]
https://mail.gna.org/listinfo/wesnoth-bugs

Reply via email to