Yet more redundant state.
With all the other changes merged in, the only point of m_PathState was to know if our last request was a long or short path. So just store that instead, and remove another instance of useless state.
UnitMotion is now pretty clean. There's still a need for a little bit of cleanup around M_CurSpeed and m_FacePointAfterMove, but that'll come later.