Page MenuHomeWildfire Games
Differential D1722

Test Entitymap helper
ClosedPublic
Actions

Authored by wraitii on Dec 30 2018, 6:53 PM.

Details

Reviewers
None
Commits
rP22027: Add tests for the Entitymap helper.
Summary

EntityMap is our "fast" version of std::map for storing entities, which has some specific performance concerns.

Before merging D8, I intend to merge the tests (so that it can be easily checked that D8 also passes the tests).

Test Plan

Compile, run tests.

Diff Detail

Event Timeline

wraitii created this revision.Dec 30 2018, 6:53 PM
Vulcan added a subscriber: Vulcan.Dec 30 2018, 6:54 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/differential/865/

Harbormaster failed remote builds in B6617: Diff 7155!Dec 30 2018, 6:54 PM
wraitii updated this revision to Diff 7256.Jan 5 2019, 11:58 AM

Make tests compatible with svn, comment out what will work with D8.

wraitii mentioned this in D8: Change internals of EntityMap to use an std:: vector.Jan 5 2019, 11:59 AM
wraitii added a child revision: D8: Change internals of EntityMap to use an std:: vector.
Vulcan added a comment.Jan 5 2019, 12:17 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/differential/915/

Harbormaster completed remote builds in B6713: Diff 7256.Jan 5 2019, 12:17 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jan 5 2019, 5:37 PM
Closed by commit rP22027: Add tests for the Entitymap helper. (authored by wraitii). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
source/
simulation2/
tests/
308 lines

Diff 7155

View Options

source/simulation2/tests/test_EntityMap.h

Loading...
Wildfire Games · Phabricator