As covered with rP18604, the simulation should always be initialized with a Seed number (otherwise we might always get the same Regicide hero on maps with very few entities for example).
The commit didn't copy that logic change to Gamesetup.cpp which will be addressed in this patch.
Details
Details
- Reviewers
- None
- Commits
- rP19749: Always set the Seed property from autostarted games, because it is required and…
- Trac Tickets
- #3965
Without this patch, the autostart on a non-random map (like pyrogenesis -autostart="scenarios/Arcadia") complains about the not specified seed and uses the default 0. With this patch, it uses the default 0 without the complaint on all maptypes.
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Checking XML files...
http://jw:8080/job/phabricator/1445/ for more details.
Comment Actions
Executing section Default... Executing section Source... Executing section JS... Executing section XML GUI... Executing section Python... Executing section Perl...
http://jw:8080/job/phabricator_lint/106/ for more details.