perf(ai): Avoid a few std::vector copies when passing ai paths to functions#1895

Merged

xezon merged 21 commits intoTheSuperHackers:mainfrom

bobtista:bobtista/move-vector-instead-of-copy

Jan 4, 2026

Commits on Dec 22, 2025

Use move semantics in aiFollowPath and aiFollowExitProductionPath
bobtista
committed
Use move semantics in AIStateMachine::setGoalPath
bobtista
committed
Update privateFollowPath to use move semantics
bobtista
committed
Use swap instead of move for VC6 compatibility
bobtista
committed
Use const_cast in aiDoCommand to pass non-const vector to privateFollowPath
bobtista
committed
Add conditional compilation for move vs swap and clear source vector
bobtista
committed
nit: make comments consistent
bobtista
committed
refactor(perf): centralize move semantics helper in CppMacros.h
bobtista
committed
refactor: remove unnecessary CppMacros.h includes
bobtista
committed
refactor(perf): rename move_assign_from_pointer to move_or_swap and change parameter to reference
bobtista
committed
refactor(perf): remove comment format and fix C++03 move semantics fallback
bobtista
committed
refactor(perf): make m_coords mutable to remove const_cast
bobtista
committed
remove include
bobtista
committed
refactor(perf): move coords into AICommandParmsStorage via const_cast
bobtista
committed
refactor(perf): remove mutable coords from AICommandParms
bobtista
committed
refactor(perf): keep AICommandParms const and copy coords where needed
bobtista
committed
Remove inline keyword from aiFollowPath functions
bobtista
committed
Use copy instead of const_cast for m_coords in AICommandParmsStorage::store
bobtista
committed
Clarify comment about copying coordinates in path following cases
bobtista
committed
Replicate AI path function changes from GeneralsMD
bobtista
committed

Commits on Jan 2, 2026

Move move_or_swap into stl namespace and clean up comments
bobtista
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

perf(ai): Avoid a few std::vector copies when passing ai paths to functions#1895

perf(ai): Avoid a few std::vector copies when passing ai paths to functions#1895
xezon merged 21 commits intoTheSuperHackers:mainfrom
bobtista:bobtista/move-vector-instead-of-copy

Commits on Dec 22, 2025

Use move semantics in aiFollowPath and aiFollowExitProductionPath

Use move semantics in AIStateMachine::setGoalPath

Update privateFollowPath to use move semantics

Use swap instead of move for VC6 compatibility

Use const_cast in aiDoCommand to pass non-const vector to privateFollowPath

Add conditional compilation for move vs swap and clear source vector

nit: make comments consistent

refactor(perf): centralize move semantics helper in CppMacros.h

refactor: remove unnecessary CppMacros.h includes

refactor(perf): rename move_assign_from_pointer to move_or_swap and change parameter to reference

refactor(perf): remove comment format and fix C++03 move semantics fallback

refactor(perf): make m_coords mutable to remove const_cast

remove include

refactor(perf): move coords into AICommandParmsStorage via const_cast

refactor(perf): remove mutable coords from AICommandParms

refactor(perf): keep AICommandParms const and copy coords where needed

Remove inline keyword from aiFollowPath functions

Use copy instead of const_cast for m_coords in AICommandParmsStorage::store

Clarify comment about copying coordinates in path following cases

Replicate AI path function changes from GeneralsMD

Commits on Jan 2, 2026

Move move_or_swap into stl namespace and clean up comments