[wesnoth-commits] [wesnoth/wesnoth] 915e5c: Fix some typos

Tue Feb 23 05:37:57 UTC 2021

  Branch: refs/heads/lua_gamemap
  Home:   https://github.com/wesnoth/wesnoth
  Commit: 915e5c725c31b3694c60c7ef9237bfec2ec1261a
      https://github.com/wesnoth/wesnoth/commit/915e5c725c31b3694c60c7ef9237bfec2ec1261a
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M src/scripting/lua_terrainfilter.cpp
    M src/scripting/lua_terrainfilter.hpp
    M src/scripting/lua_terrainmap.cpp
    M src/scripting/lua_terrainmap.hpp
    M src/scripting/mapgen_lua_kernel.cpp

  Log Message:
  -----------
  Fix some typos

  Commit: e20ec2ece2a2f11b150c03d3396d4fb971ba543c
      https://github.com/wesnoth/wesnoth/commit/e20ec2ece2a2f11b150c03d3396d4fb971ba543c
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    A data/lua/core/map.lua

  Log Message:
  -----------
  Add some utility functions to help clarify the merge mode being used when assigning terrains

  Commit: ce3e30d0072a22190583ae3f0f2e20fe6f6d56ec
      https://github.com/wesnoth/wesnoth/commit/ce3e30d0072a22190583ae3f0f2e20fe6f6d56ec
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/lua/core/map.lua
    M data/lua/location_set.lua

  Log Message:
  -----------
  Add a utility function to extract a location from the front of a variadic parameter pack

The purpose of this is to make it easy for functions implemented in Lua to handle locations
in the same way as functions implemented in C++.

The location_set module has been updated to make use of this functionality in its setter functions.

Usage example:

Imagine a function foo with the following signature:

foo(bar : string, home : location, mode : string, target : location, moo : boolean) -> boolean

With the new read_location function it could be implemented as follows:

function foo(...)
	-- First argument goes in bar
	local bar = ...
	-- Read location starting at the second argument
	local home, n = wesnoth.mP.read_location(select(2, ...))
	-- note: n will be 0 if a location wasn't found at that position
	-- This could be an error, or it could be handled as an optional parameter
	-- Next argument after that goes in mode
	local mode = select(n + 2, ...)
	-- Then read a location into target
	local target, m = wesnoth.map.read_location(select(n + 2, ...))
	-- Finally, read a parameter into moo
	local moo = select(m + n + 2, ...)
	-- Do stuff with all these parameters
	return true
end

With that code, all the following invocations of foo work:

foo('a', 'b', true) -- both optional locations omitted
foo('a', 1, 2, 'q', 5, 6, false) -- locations given as separate integer parameters
foo('a', 'm', {1, 7},  true) -- first location omitted, second given as 2-element array
foo('a', some_unit, 'z', {x = 5, y = 10}, false) -- a unit also functions as a location
foo('a', 7, 12, 'q', my_leader, true) -- mixing different forms also works

  Commit: 7ce225f4f82c17cbad8cc94bc5b63d59a497f092
      https://github.com/wesnoth/wesnoth/commit/7ce225f4f82c17cbad8cc94bc5b63d59a497f092
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/lua/core/map.lua
    M src/editor/action/action.cpp
    M src/editor/action/mouse/mouse_action.cpp
    M src/editor/map/editor_map.cpp
    M src/game_board.cpp
    M src/game_board.hpp
    M src/map/map.cpp
    M src/map/map.hpp
    M src/scripting/game_lua_kernel.cpp
    M src/scripting/game_lua_kernel.hpp
    M src/scripting/lua_terrainfilter.cpp
    M src/scripting/lua_terrainfilter.hpp
    M src/scripting/lua_terrainmap.cpp
    M src/scripting/lua_terrainmap.hpp
    M src/scripting/mapgen_lua_kernel.cpp

  Log Message:
  -----------
  Refactor the game map to permit exposing it to Lua via wesnoth.current.map

The method of accessing terrain on the map has drastically changed.
- wesnoth.get_terrain and wesnoth.set_terrain are both deprecated
- wesnoth.terrain_mask still works but is moved into the wesnoth.map module and now takes the map object as the first parameter
- The map's terrain is now accessed exclusively via indexing on the map object, ie map[{x,y}]
- You set terrain by assigning a terrain code; the position of ^ in the terrain code now determines the merge mode
- The replace_if_failed option is now manifested as a function that converts any terrain code into a special value that, when assigned to a location on the map, uses the replace if failed logic.

The map object has a few attributes in it:
- width and height are the total size, including borders
- playable_width and playable_height are the values returned from wesnoth.get_map_size, which is now deprecated
- border_size is the third value from wesnoth.get_map_size
- data converts the map to a string
- Special locations are now part of the map object. The length operator is deprecated.
- other than that, wesnoth.map is treated as if it were the metatable of the map object

  Commit: b59cd9bb8c81cefa0bcd2de5440acb59b7a5228d
      https://github.com/wesnoth/wesnoth/commit/b59cd9bb8c81cefa0bcd2de5440acb59b7a5228d
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/lua/core/map.lua
    M src/scripting/game_lua_kernel.cpp
    M src/scripting/game_lua_kernel.hpp

  Log Message:
  -----------
  Move various functions into the map module

  Commit: 0a589e441fb7835fd000130c18f071799bef001d
      https://github.com/wesnoth/wesnoth/commit/0a589e441fb7835fd000130c18f071799bef001d
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/lua/core/map.lua
    M src/scripting/game_lua_kernel.cpp
    M src/scripting/lua_terrainfilter.cpp
    M src/scripting/lua_terrainmap.cpp
    M src/scripting/lua_terrainmap.hpp
    M src/scripting/mapgen_lua_kernel.cpp

  Log Message:
  -----------
  Rename existing map functions in the map generation kernel

  Commit: b580328b3a31037d4b94b1b85fa438e758a64e7d
      https://github.com/wesnoth/wesnoth/commit/b580328b3a31037d4b94b1b85fa438e758a64e7d
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/lua/core/map.lua
    M src/scripting/lua_kernel_base.cpp

  Log Message:
  -----------
  Rename two functions for consistency of terminology

  Commit: 61adb06b8f92d9c9b7fbbd09bfdf90b382d38f50
      https://github.com/wesnoth/wesnoth/commit/61adb06b8f92d9c9b7fbbd09bfdf90b382d38f50
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

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

  Log Message:
  -----------
  Add wesnoth.map.get_label

  Commit: 08d768db8fe3ca1b76d6e64cd0a4ed57ac47b9aa
      https://github.com/wesnoth/wesnoth/commit/08d768db8fe3ca1b76d6e64cd0a4ed57ac47b9aa
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M src/scripting/lua_kernel_base.cpp
    M src/scripting/lua_map_location_ops.cpp
    M src/scripting/lua_map_location_ops.hpp

  Log Message:
  -----------
  Add wesnoth.map.get_hexes_in_radius

  Commit: fb52d018cbbb5c8286401e6937b3f7d2e26fc446
      https://github.com/wesnoth/wesnoth/commit/fb52d018cbbb5c8286401e6937b3f7d2e26fc446
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M src/scripting/game_lua_kernel.cpp
    M src/scripting/lua_terrainmap.cpp
    M src/scripting/lua_terrainmap.hpp

  Log Message:
  -----------
  Add on_board and on_border map methods

  Commit: 24f57e3018e6a42a69923255ba1fcdba451f0cc6
      https://github.com/wesnoth/wesnoth/commit/24f57e3018e6a42a69923255ba1fcdba451f0cc6
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M src/scripting/game_lua_kernel.cpp
    M src/scripting/lua_terrainmap.cpp
    M src/scripting/lua_terrainmap.hpp

  Log Message:
  -----------
  Implement a map iterator

  Commit: 4068f52e74e2de60bf21bebd7c04857361ca3deb
      https://github.com/wesnoth/wesnoth/commit/4068f52e74e2de60bf21bebd7c04857361ca3deb
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/lua/core/map.lua

  Log Message:
  -----------
  Add a terrain hex reference API

This adds a metatable to all locations returned from wesnoth.map.find.

  Commit: c8386c7ffc1f0410b97c71645254abd2e519acfc
      https://github.com/wesnoth/wesnoth/commit/c8386c7ffc1f0410b97c71645254abd2e519acfc
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/lua/core/map.lua
    M src/scripting/game_lua_kernel.cpp
    M src/scripting/game_lua_kernel.hpp

  Log Message:
  -----------
  Add a wesnoth.terrain_types table

  Commit: 92ce2f29ee13fcd190f931a71ab464a58687b884
      https://github.com/wesnoth/wesnoth/commit/92ce2f29ee13fcd190f931a71ab464a58687b884
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/ai/lua/ai_helper.lua
    M data/ai/lua/battle_calcs.lua
    M data/ai/lua/ca_castle_switch.lua
    M data/ai/lua/ca_grab_villages.lua
    M data/ai/lua/ca_recruit_rushers.lua
    M data/ai/lua/ca_spread_poison.lua
    M data/ai/lua/ca_village_hunt.lua
    M data/ai/lua/generic_recruit_engine.lua
    M data/ai/lua/retreat.lua
    M data/ai/micro_ais/cas/ca_assassin_move.lua
    M data/ai/micro_ais/cas/ca_big_animals.lua
    M data/ai/micro_ais/cas/ca_bottleneck_move.lua
    M data/ai/micro_ais/cas/ca_fast_attack_utils.lua
    M data/ai/micro_ais/cas/ca_fast_move.lua
    M data/ai/micro_ais/cas/ca_forest_animals_new_rabbit.lua
    M data/ai/micro_ais/cas/ca_goto.lua
    M data/ai/micro_ais/cas/ca_healer_move.lua
    M data/ai/micro_ais/cas/ca_herding_dog_move.lua
    M data/ai/micro_ais/cas/ca_herding_f_herding_area.lua
    M data/ai/micro_ais/cas/ca_herding_herd_sheep.lua
    M data/ai/micro_ais/cas/ca_lurkers.lua
    M data/ai/micro_ais/cas/ca_protect_unit_move.lua
    M data/ai/micro_ais/cas/ca_recruit_random.lua
    M data/ai/micro_ais/cas/ca_stationed_guardian.lua
    M data/ai/micro_ais/cas/ca_wolves_move.lua
    M data/ai/micro_ais/cas/ca_zone_guardian.lua
    M data/campaigns/Descent_Into_Darkness/scenarios/04_Spring_of_Reprisal.cfg
    M data/campaigns/Eastern_Invasion/ai/ca_ogres_flee.lua
    M data/campaigns/Eastern_Invasion/lua/bandits.lua
    M data/campaigns/Heir_To_The_Throne/scenarios/13_The_Dwarven_Doors.cfg
    M data/campaigns/Northern_Rebirth/lua/respawn_utils.lua
    M data/campaigns/Son_Of_The_Black_Eye/ai/ca_transport_S6.lua
    M data/campaigns/The_Hammer_of_Thursagan/lua/spawns.lua
    M data/campaigns/The_Hammer_of_Thursagan/scenarios/05_Fear.cfg
    M data/campaigns/Two_Brothers/ai/ca_muff_toras_move.lua
    M data/campaigns/World_Conquest/lua/campaign/autorecall.lua
    M data/campaigns/World_Conquest/lua/campaign/enemy.lua
    M data/campaigns/World_Conquest/lua/campaign/enemy_themed.lua
    M data/campaigns/World_Conquest/lua/game_mechanics/bonus.lua
    M data/campaigns/World_Conquest/lua/game_mechanics/supply.lua
    M data/campaigns/World_Conquest/lua/map/generator/utilities.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/1A_Start.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/2A_Springs.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/2C_Glaciers.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/2D_Provinces.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/2E_Paradise.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/3C_Delta.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/3F_Wetland.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/4A_Thermal.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/4B_Volcanic.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/4C_Mines.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/6A_Rural.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/6B_Maritime.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/6C_Industrial.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration/6D_Feudal.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration_utils/engine.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration_utils/events.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration_utils/noise.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration_utils/utilities.lua
    M data/campaigns/World_Conquest/lua/map/postgeneration_utils/wild_zones.lua
    M data/campaigns/World_Conquest/lua/map/scenario_utils/bonus_points.lua
    M data/campaigns/World_Conquest/lua/map/wct_map_generator.lua
    M data/campaigns/World_Conquest/lua/optional_mechanics/destruction.lua
    M data/lua/helper.lua
    M data/lua/location_set.lua
    M data/lua/wml-tags.lua
    M data/lua/wml/animate_unit.lua
    M data/lua/wml/find_path.lua
    M data/lua/wml/items.lua
    M data/lua/wml/modify_side.lua
    M data/lua/wml/random_placement.lua
    M data/modifications/pick_advance/main.lua
    M data/multiplayer/scenarios/2p_Dark_Forecast.lua
    M data/scenario-test.cfg
    M src/scripting/lua_terrainmap.cpp

  Log Message:
  -----------
  Update everything to use the new wesnoth.map module

- get_terrain and set_terrain replaced with direct indexing operations
- get_map_size mostly replaced with either the iterator or an on_board call.
  Only a few cases really needed to know the size of the map for some other purpose.
- shroud and fog operations, village owner, time areas, and location filters
- get_terrain_info replaced with terrain_types table
- Map generation functions create_map and create_filter

  Commit: a86ff325177880d85934113f5965f9e5cef1502d
      https://github.com/wesnoth/wesnoth/commit/a86ff325177880d85934113f5965f9e5cef1502d
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M src/map/location.cpp

  Log Message:
  -----------
  No reason to use the less clear implementation, is there?

  Commit: 1a14a662ba2f0aa7218e5c43ef5dd376045c5e5a
      https://github.com/wesnoth/wesnoth/commit/1a14a662ba2f0aa7218e5c43ef5dd376045c5e5a
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/campaigns/World_Conquest/lua/map/postgeneration_utils/engine.lua
    M data/lua/core/map.lua

  Log Message:
  -----------
  Move WC's filter tag helpers into core

  Commit: 218bbb24d3d07dce9fa5e7f8061da7eaab00f7b3
      https://github.com/wesnoth/wesnoth/commit/218bbb24d3d07dce9fa5e7f8061da7eaab00f7b3
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M src/scripting/lua_terrainfilter.cpp

  Log Message:
  -----------
  Fix mapgen "notall" filter not working

  Commit: 1c73c9399102266a1ffeef89db232d82fecd4475
      https://github.com/wesnoth/wesnoth/commit/1c73c9399102266a1ffeef89db232d82fecd4475
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M src/scripting/lua_terrainfilter.cpp
    M src/scripting/lua_terrainfilter.hpp
    M src/scripting/mapgen_lua_kernel.cpp

  Log Message:
  -----------
  For consistency, mapgen should also expose a matches function

  Commit: a0d33172e97ca60e7f646587bc35cd28db5032df
      https://github.com/wesnoth/wesnoth/commit/a0d33172e97ca60e7f646587bc35cd28db5032df
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M src/scripting/game_lua_kernel.cpp

  Log Message:
  -----------
  Enable find() and matches() to be called as map methods

  Commit: edcc217ae4a55b4df70bf9fafedf5cc69c6df45f
      https://github.com/wesnoth/wesnoth/commit/edcc217ae4a55b4df70bf9fafedf5cc69c6df45f
  Author: Celtic Minstrel <celtic.minstrel.ca at some.place>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M data/lua/wml/test_condition.lua

  Log Message:
  -----------
  Update [test_condition] to be a little more verbose and make use of newer features

Compare: https://github.com/wesnoth/wesnoth/compare/21582496a282...edcc217ae4a5