Skip to content

fix bug where combat doesn't end on team retreat#499

Merged
majcosta merged 1 commit into
1dot13:masterfrom
majcosta:pr_fix_combat_not_ending_if_merc_is_inbound_retreated_sector
Sep 12, 2025
Merged

fix bug where combat doesn't end on team retreat#499
majcosta merged 1 commit into
1dot13:masterfrom
majcosta:pr_fix_combat_not_ending_if_merc_is_inbound_retreated_sector

Conversation

@majcosta
Copy link
Copy Markdown
Contributor

@majcosta majcosta commented Sep 7, 2025

fixes issue #498 where retreating from combat in a sector that has a merc inbound via the airdrop marker causes combat not to end

the reason this fails is that a the inbound soldier counts as being in the sector being retreated from, even though he isn't

I add a check for MercPtrs[i]->bInSector so this doesn't happen

that way the function doesn't leave early and CheckForEndOfCombatMode is reached

fixes issue 1dot13#498 where retreating from combat in a sector that has a merc
inbound via the airdrop marker causes combat not to end

the reason this fails is that a the inbound soldier counts as being in
the sector being retreated from, even though he isn't

I add a check for `MercPtrs[i]->bInSector` so this doesn't happen

that way the function doesn't leave early and CheckForEndOfCombatMode is
reached
@majcosta majcosta merged commit 4ec012c into 1dot13:master Sep 12, 2025
@majcosta majcosta deleted the pr_fix_combat_not_ending_if_merc_is_inbound_retreated_sector branch September 12, 2025 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant