HomeWildfire Games

Add tests for the hierarchical pathfinder.

Description

Add tests for the hierarchical pathfinder.

These variously test some simple MakeGoalReachable and FindNearestNavcell calls, as well as checking edges and regions when updating the map.

Differential Revision: https://code.wildfiregames.com/D1833

Event Timeline

elexis added a subscriber: elexis.Apr 22 2019, 2:42 PM

@wraitii wasnt there supposed to be a new file committed here?

There was... Thanks for noticing.

vladislavbelov added inline comments.
/ps/trunk/source/simulation2/helpers/HierarchicalPathfinder.h
54

I think it's better to put private stuff under public sections. Because usually readers need a public API and only then private implementation.

wraitii added inline comments.Apr 22 2019, 11:44 PM
/ps/trunk/source/simulation2/helpers/HierarchicalPathfinder.h
54

I'd agree in general, but I tend to put friend declaration above everything. Not sure why. I don't think we have that many examples in the codebase to refer too but I can check.