Skip to content

Reapply #2095 and fix reconnection error cause by it#2134

Open
NetsuNegi wants to merge 3 commits intoPhobos-developers:developfrom
NetsuNegi:branch/reapply-2095
Open

Reapply #2095 and fix reconnection error cause by it#2134
NetsuNegi wants to merge 3 commits intoPhobos-developers:developfrom
NetsuNegi:branch/reapply-2095

Conversation

@NetsuNegi
Copy link
Contributor

No description provided.

@NetsuNegi NetsuNegi added Needs testing ⚙️T1 T1 maintainer review is sufficient Bugfix This is a bugfix that does not need documentation beyond mention in changelog Needs MP testing This PR needs to be tested for desync. No Documentation Needed No documentation needed whatsoever and removed Bugfix This is a bugfix that does not need documentation beyond mention in changelog labels Mar 12, 2026
@phoboscn-bot
Copy link

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

@github-actions
Copy link

github-actions bot commented Mar 13, 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.

}
}
else
{
Copy link
Member

Choose a reason for hiding this comment

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

bracket style

Copy link
Contributor Author

Choose a reason for hiding this comment

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

what ?

}
}
else
{
Copy link
Member

Choose a reason for hiding this comment

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

else
{
	pSource->SendToFirstLink(RadioCommand::NotifyUnlink);
}

to

else
	pSource->SendToFirstLink(RadioCommand::NotifyUnlink);

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it should be in line with the if statement, so the bracket could be kept

@Coronia
Copy link
Contributor

Coronia commented Mar 13, 2026

didn't desync in our test so far, and the previous issues are solved. More testing is appreciated tho

@Coronia Coronia added Tested and removed Needs testing Needs MP testing This PR needs to be tested for desync. labels Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Documentation Needed No documentation needed whatsoever ⚙️T1 T1 maintainer review is sufficient Tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants