Due to swapped order of player base flattening and height area calculation woods sometimes grew into players bases. This patch fixes that.
Details
- Reviewers
elexis - Group Reviewers
Restricted Owners Package (Owns No Changed Paths) - Commits
- rP19808: Prevent height based forests to grow into player start locations.
Create several maps with this patch with different combinations of map size, number of players and seed.
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 2346 Build 3906: Vulcan Build (Windows) Jenkins Build 3905: Vulcan Build Jenkins Build 3904: arc lint + arc unit
Event Timeline
Build has FAILED
Link to build: http://jw:8080/job/phabricator/1588/
See console output for more information: http://jw:8080/job/phabricator/1588/console
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/212/ for more details.
To reproduce the original bug:
Generate: Map Size: Normal, Number of Players: 4, Seed: 2537 (High ground trees in bases of player 1 and 4)
Build has FAILED
Link to build: http://jw:8080/job/phabricator/1589/
See console output for more information: http://jw:8080/job/phabricator/1589/console
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/213/ for more details.
Change reads correct, fixes the bug reproducible with the given seen and doesn't appear to influence anything else.
binaries/data/mods/public/maps/random/caledonian_meadows.js | ||
---|---|---|
634 | start locations |