Right now, you must call the MoveDelegate twice in each turn because certain things only happen during the "non-combat" version which runs the second time (clearing unit movement, etc.). To add support for 1914, it must be possible to have a single move phase in each turn.
(see patch #70 for a dependent change)
Move the unit and transport clean-up from the MoveDelegate to the EndTurnDelegate. These functions really make more sense here anyway, since unit movement restrictions, transport load restrictions, etc. are for the duration of the turn.
I don't believe there's anything in the PlaceDelegate that would depend on these having been cleared, but I may be wrong.
Rather than calling DelegateFinder to find the MoveDelegate and then use its TransportTracker instance, I thought it made more sense to make endOfRoundClearStateChange static, since it doesn't actually use any class members.
Log in to post a comment.