Page MenuHomeWildfire Games

Remove CTerritoryManager pointer in CWorld, a relic from before simulation2
ClosedPublic

Authored by Sandarac on Jul 10 2017, 2:08 AM.

Details

Summary

As discussed in D642, this can be removed from CWorld, as it was missed in the old clean-up commits of the old simulation (rP7839 and similar).

Test Plan

See that CTerritoryManager doesn't exist anywhere else in the code.

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

Sandarac created this revision.Jul 10 2017, 2:08 AM
leper accepted this revision.Jul 10 2017, 2:19 AM

Thanks, I'll include this after the release. (Ping me in case I forget.)

This revision is now accepted and ready to land.Jul 10 2017, 2:19 AM
Vulcan added a subscriber: Vulcan.Jul 10 2017, 3:11 AM

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/1725/ for more details.

This revision was automatically updated to reflect the committed changes.