[Commits] [wesnoth/wesnoth] 0dfb39: Fix wesnoth.set_village_owner ignoring the fire_ev...

GitHub noreply at github.com
Sat Feb 17 06:23:51 UTC 2018


  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth
  Commit: 0dfb390a844f80ce898c5c641fa6ec45bab7add2
      https://github.com/wesnoth/wesnoth/commit/0dfb390a844f80ce898c5c641fa6ec45bab7add2
  Author: Celtic Minstrel <CelticMinstrel at users.noreply.github.com>
  Date:   2018-02-17 (Sat, 17 Feb 2018)

  Changed paths:
    M src/scripting/game_lua_kernel.cpp

  Log Message:
  -----------
  Fix wesnoth.set_village_owner ignoring the fire_event parameter

Looks like this was lost in the refactor that introduced the luaW_checklocation helper function.
If the location is specified as two separate parameters for x and y, this helper function
actually pops the y parameter from the stack so that further parameter tests can act
as if the location was a single parameter. Thus, the fire_event parameter, which used to
be 4th on the stack, is now moved down to 3rd.





More information about the Commits mailing list