mirrors/0ad
1
0 Fork 2
mirror of https://gitea.wildfiregames.com/0ad/0ad.git synced 2026-11-08 10:09:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

UnitMotion: Fix ShouldAlternatePathfinder() using wrong constant

Browse Source 
Used MAX_FAILED_MOVEMENTS (a fixed constant) instead of
m_FailedMovements, making the "every N failures" clause always
false at compile time. Alternation only ever fired once per
failure streak instead of periodically as intended.

Introduced in 70ec781.
This commit is contained in:
Fabio Pedretti authored and Phosit committed 2026-10-15 16:56:34 +02:00
1 parent f10fe18d7a
commit 43d8446854
1 file changed
+1 -1
Download Patch File Download Diff File Expand all files Collapse all files
source/simulation2/components/CCmpUnitMotion.h
+1 -1
View File
@@ -746,7 +746,7 @@ private:
bool ShouldAlternatePathfinder() const
{
return (m_FailedMovements == ALTERNATE_PATH_TYPE_DELAY) || ((MAX_FAILED_MOVEMENTS - ALTERNATE_PATH_TYPE_DELAY) % ALTERNATE_PATH_TYPE_EVERY == 0);
return (m_FailedMovements == ALTERNATE_PATH_TYPE_DELAY) || ((m_FailedMovements - ALTERNATE_PATH_TYPE_DELAY) % ALTERNATE_PATH_TYPE_EVERY == 0);
}
bool InShortPathRange(const PathGoal& goal, const CFixedVector2D& pos) const