petra: improve the use of garrisoning
ClosedPublic
Actions

Authored by mimo on Feb 12 2017, 10:22 PM.

Details

Reviewers

Commits

rP19222: petra: improve the use of garrisoning

Summary

be more inclined to garrison units to take advantage of arrow multipliers by easing the requirements on units (units affected to the defense can now be garrisoned, as well as those affected to the attack when they capture a structure). Futhermore, structures were garrisoned only when attacked, while now they can be garrisoned also when they are able to attack.

Test Plan

Check that the AI does the right thing.

Diff Detail

Repository

rP 0 A.D. Public Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

mimo created this revision.Feb 12 2017, 10:22 PM

Harbormaster completed remote builds in B512: Diff 517.Feb 12 2017, 11:43 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (302 tests)..............................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (302 tests)..............................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/353/ for more details.

It looks good to me.

This revision is now accepted and ready to land.Feb 13 2017, 9:01 AM

Closed by commit rP19222: petra: improve the use of garrisoning (authored by mimo). · Explain WhyFeb 13 2017, 5:04 PM

This revision was automatically updated to reflect the committed changes.