← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-1815283-nil-value-territorial into lp:widelands

 

I looked also into this issue and stumbled over player.number and the description of players: https://wl.widelands.org/documentation/autogen_wl_bases/#wl.bases.EditorGameBase.players

So, assuming the second slot of a three players map is deactivated, the result will be:

for idx, player in ipairs(wl.Game().players) do
  print(idx, player.name, player.number)
end

Output:

1   Player 1   1
2   Player 3   3

So i am not sure if we should get rid of player.number at all? Don't know if it is feasible.

-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1815283-nil-value-territorial/+merge/363079
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1815283-nil-value-territorial into lp:widelands.


References