[Commits] [wesnoth/wesnoth] de4479: Make [*time_area] Lua API functions take a single ...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth
  Commit: de44790d7d96262d470d8ca5adc2e2ca6964e4b1
      https://github.com/wesnoth/wesnoth/commit/de44790d7d96262d470d8ca5adc2e2ca6964e4b1
  Author: Ignacio R. Morelle <shadowm at wesnoth.org>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M changelog
    M src/scripting/game_lua_kernel.cpp

  Log Message:
  -----------
  Make [*time_area] Lua API functions take a single time area id

This means the Lua code implementing the WML tags are now solely
responsible for parsing the comma-separated id list. It just so happens
that I added code for that without realizing that I was duplicating
existing functionality on the C++ side.

As a result:
 * wesnoth.add_time_area and wesnoth.remove_time_area can operate on
   time area ids that contain commas (if someone feels this is needed
   for some weird reason)
 * [time_area] no longer warns about adding time areas with commas in
   their ids (actually adding the first id of the list instead)
 * [time_area] remove=yes and [remove_time_area] still handle
   comma-separated lists of ids, in the Lua side instead of C++.

This makes the Lua API for this consistent with [event]/[remove_event].

Since the Lua API functions in question are undocumented at this moment,
it's unlikely this impacts anyone, really.