petra: take restricted classes into account
ClosedPublic
Actions

Authored by mimo on May 10 2017, 8:32 PM.

Details

Reviewers: None
Commits: rP19572: petra: take restricted classes into account

Summary

petra fixes to take restricted classes into accound.

This patch implements some changes needed to take restricted classes into account for example for D450, but it does not claim to be complete. This will have to be fully tested when these restrictions are really used.

Test Plan

Test that everything works as expected.

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

mimo created this revision.May 10 2017, 8:32 PM

mimo mentioned this in D450: Let wall being captured by territory but not by capture attack..May 10 2017, 8:40 PM

fatherbushido added a subscriber: fatherbushido.May 10 2017, 9:03 PM

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!

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

Harbormaster completed remote builds in B1627: Diff 1830.May 11 2017, 3:28 AM

2017-05-11_0001.zip36 KBDownload

I just ran a game tested with D473 + D450 and noticed nothing wrong.