Page MenuHomeWildfire Games

UnitMotion - Split Move() into several functions
Needs ReviewPublic

Authored by wraitii on Wed, May 15, 10:44 PM.

Details

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

Move() is generally 4 parts:

  • Moving
  • Updating our state
  • Handling obstructed moves
  • Checking if we are at destination.

These can all be put in their own functions, clarifying logic and making it harder to make mistakes. D1893 and D1896 make it trivial.

Test Plan

Compile.

Diff Detail