Consider moving std::vector instead of copying the data

[Caball009]

There are a couple of functions where std::vector<T> containers are copied instead of moved:
aiFollowExitProductionPath
aiFollowPath
AIStateMachine::setGoalPath

Maybe more could be found with queries such as: \(.*std::vector<.*>\s*\*

Beware that moving from const std::vector<T>* is meaningless, so this would require removing const from the function parameter at all call sites.