Sat 17 Apr 2010 12:03:53 AM UTC, original submission:
paakku> it looks like KillUnitAt expects to get exactly two arguments, but uses more
<Kyran> is it an easy fix?
<paakku> looks easy, but there are more bugs, instead of KillUnitAt(unit, player, quantity, {x1,y1}, {x2,y2}), the code expects KillUnitAt(unit, player, quantity, {x1,y1,x2,y2}) so, four args rather than five
<Kyran> for consistency with the rest of the scripts it should be {x1,y1}, {x2,y2}
<paakku> hoh, the number-of-arguments check was changed in 2005 to use the LuaCheckArgs macro, but the bug was there before that AFAICT, the original implementation of this function with SIOD or Guile did not have the bug. it was introduced during the conversion to lua
|