Page MenuHomeWildfire Games

Unit Motion - remove the m_Moving variable
Needs ReviewPublic

Authored by wraitii on Mon, May 13, 9:48 PM.

Details

Reviewers
None
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

m_Moving is supposed to tell us whether we are trying to move or not (NB: this is different from moving, which implies our position is changing). It was used by IsMoving(), which should be renamed "HasATarget" or something in a later diff.

The variable is not necessary since having a target is equivalent, and this reduces state, so remove it.

Test Plan

Compile.

Diff Detail

Event Timeline

wraitii created this revision.Mon, May 13, 9:48 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/differential/1399/display/redirect