Ares DriverKilled Fix

[Fly-Star-him]

What kind of change is this?

• New feature, vanilla bugfix or enhancement of a released feature - changelog, docs and credits entries are needed.
• Improvement to a new (unreleased) feature - docs and credits entries are needed; no changelog entry (Skip Changelog).
• Bugfix to a new (unreleased) feature - credits entry is needed; no changelog or docs entries (Skip Changelog, Skip Docs).
• Bugfix to an old (released) feature - changelog and credits entries are needed; no docs entry (Skip Docs).
• Completely minor change (e.g. a typo fix) - no entries are needed (Skip Changelog, Skip Docs, Skip Credits).

Description

Fixed an issue in Ares where vehicles would occasionally continue to fire and move even after the driver had been killed.

修复Ares下载具驾驶员被击杀后仍有可能继续开火、移动的问题。

[phoboscn-bot]

To Chinese users:
This pull request has been mentioned on Phobos CN. There might be relevant details there:

致中文用户：
此拉取请求已在 Phobos CN 上被提及。那里可能有相关详细信息：

https://www.phoboscn.top/t/topic/798/1

[github-actions]

Nightly build for this pull request:

• compiled-dll-cc3adafb316297d27af0b4885825778dd13bb345.zip
  These artifacts will expire in 90 days and will not be available for download after that time.

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

[TaranDahl]

What is the root cause of this issue

[Fly-Star-him]

This PR needs to be tested for performance.

Some players have reported that vehicles whose drivers have been killed may still launch attacks; here is one such case:

Since I am unable to reproduce this issue myself, I can only implement some potential fixes.

[TaranDahl]

I think your GIF clearly demonstrates the correlation between this issue and the miner being attacked. Combined with the Rescue mission, it most likely points to Base_Is_Attacked.
I suspect the check at 70849D causes the rescue behavior to trigger incorrectly for these units in non-campaign modes.

[NetsuNegi]

I think your GIF clearly demonstrates the correlation between this issue and the miner being attacked. Combined with the Rescue mission, it most likely points to Base_Is_Attacked. I suspect the check at 70849D causes the rescue behavior to trigger incorrectly for these units in non-campaign modes.

Not only in this function, also at 0x7027E9 that called Base_Is_Attacked, this call won't check DriverKilled. This will cause a driver killed vehicle call for reinforcement when receive damage.

[NetsuNegi]

I think your GIF clearly demonstrates the correlation between this issue and the miner being attacked. Combined with the Rescue mission, it most likely points to Base_Is_Attacked. I suspect the check at 70849D causes the rescue behavior to trigger incorrectly for these units in non-campaign modes.

https://www.phoboscn.top/t/topic/800