Skip to content

[Vanilla Fix] Fixed the bug that low-air taking off / landing objects will receive twice damage#2146

Closed
NetsuNegi wants to merge 2 commits intoPhobos-developers:developfrom
NetsuNegi:bugfix/low_air_damage
Closed

[Vanilla Fix] Fixed the bug that low-air taking off / landing objects will receive twice damage#2146
NetsuNegi wants to merge 2 commits intoPhobos-developers:developfrom
NetsuNegi:bugfix/low_air_damage

Conversation

@NetsuNegi
Copy link
Copy Markdown
Contributor

No description provided.

@NetsuNegi NetsuNegi added ❓Vanilla bug Vanilla game bugs that are requested to be fixed Needs testing ⚙️T1 T1 maintainer review is sufficient labels Mar 23, 2026
@phoboscn-bot
Copy link
Copy Markdown

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/283/1

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 23, 2026

Nightly build for this pull request:

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.

Copy link
Copy Markdown
Contributor

@TaranDahl TaranDahl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should not be implemented like that.
WW has established two separate Trackers, namely the FirstObject linked list and ATC, to track the army and air force respectively. The essential reason for this issue is that the maintenance of these two Trackers is independent, and their maintenance standards are also irrelevant.
I believe the correct implementation method is to rectify their maintenance approaches. Units with a value greater than or equal to 208 will enter the ATC. Units with a value greater than or equal to 0 and less than 208 will enter the cell linked list.

@NetsuNegi NetsuNegi closed this Mar 23, 2026
@NetsuNegi NetsuNegi deleted the bugfix/low_air_damage branch March 23, 2026 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs testing ⚙️T1 T1 maintainer review is sufficient ❓Vanilla bug Vanilla game bugs that are requested to be fixed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants