HomeWildfire Games

Fix a couple of packing problems from rP21630

Description

Fix a couple of packing problems from rP21630

Packable units should pack in the attack order after the standground check, and they should pack before chasing.
Reported by: bb
Comments by: mimo at D1458

Details

Event Timeline

mimo added a subscriber: mimo.Apr 26 2018, 6:50 PM

This commit make things worse for packing units in attack-move (as explained several times in D1458, if they are now in standground stance, it is for a reason). But as it seems that now, the new rule is to commit patches without acceptation, i've done the same and committed my own fix in rP21786.

In rP21784#30460, @mimo wrote:

This commit make things worse for packing units in attack-move (as explained several times in D1458, if they are now in standground stance, it is for a reason).

It doesn't make them worse because in a22 every unit (except packable units) in standground would chase in attack-move, and now packable units will also chase (the point of your original commit was to remove the inconsistencies with packable units) rather than being bugged.

But as it seems that now, the new rule is to commit patches without acceptation,

You said

I agree with temple proposition in rP21630 (switching the order of the standground check and the pack)

and

That said, having the canPack at the place you mention seems to be useful to avoid units to be stuck.

which sounded to me like you agreed with the changes I made.
I didn't say "Reviewed by mimo" because you didn't official accept them, but still.

mimo added a comment.Apr 26 2018, 7:48 PM
In rP21784#30460, @mimo wrote:

This commit make things worse for packing units in attack-move (as explained several times in D1458, if they are now in standground stance, it is for a reason).

It doesn't make them worse because in a22 every unit (except packable units) in standground would chase in attack-move, and now packable units will also chase (the point of your original commit was to remove the inconsistencies with packable units) rather than being bugged.

But as it seems that now, the new rule is to commit patches without acceptation,

You said

I agree with temple proposition in rP21630 (switching the order of the standground check and the pack)

and

That said, having the canPack at the place you mention seems to be useful to avoid units to be stuck.

which sounded to me like you agreed with the changes I made.
I didn't say "Reviewed by mimo" because you didn't official accept them, but still.

Never extract some sentences out of context please! In all my answers, i also added that all that had to be completed by the change in attack-move.

It makes thing worse because if i set my pack unit to standground, it means i don't want it to chase units it has not chance to catch up, and will be drifted far from protective soldiers.