I like the proposed change. However, when starting the editor and clicking the "Players" button I get an exception:
widelands: ../src/logic/map.cc:411: const string& Widelands::Map::get_scenario_player_tribe(Widelands::PlayerNumber) const: Assertion `p <= get_nrplayers()' failed.
Problem is the change in diff line 81, where p can be greater than the number of active players.
Since the editor-user has the option to do so now anyway: Select the "random" tribe in line 42 of the diff?
« Back to merge proposal
I like the proposed change. However, when starting the editor and clicking the "Players" button I get an exception:
widelands: ../src/ logic/map. cc:411: const string& Widelands: :Map::get_ scenario_ player_ tribe(Widelands ::PlayerNumber) const: Assertion `p <= get_nrplayers()' failed.
Problem is the change in diff line 81, where p can be greater than the number of active players.
Since the editor-user has the option to do so now anyway: Select the "random" tribe in line 42 of the diff?