Skip to content

Fix VTR viewed-map flush after reload#956

Open
MhaWay wants to merge 1 commit into
rwmt:devfrom
MhaWay:analysis/pr-865-projectile-desync
Open

Fix VTR viewed-map flush after reload#956
MhaWay wants to merge 1 commit into
rwmt:devfrom
MhaWay:analysis/pr-865-projectile-desync

Conversation

@MhaWay

@MhaWay MhaWay commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Fix VTR viewed-map flush after reload

This prevents viewed-map updates from being dropped during reload transitions, which left VTR/player-count state incoherent and caused the projectile slowdown symptom.

Validation:

  • Release build succeeded on the fix branch
  • Dev-port build succeeded and reproduced the issue/fix path

Copilot AI review requested due to automatic review settings June 28, 2026 15:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses a multiplayer reload edge case where viewed-map (PlayerCount) updates could be dropped during Multiplayer.reloading transitions, leaving player-count/VTR state inconsistent and contributing to the reported projectile slowdown symptom.

Changes:

  • Queue viewed-map updates during reload instead of dropping them.
  • Add a flush mechanism and invoke it from several frequently-hit patches to ensure pending updates are delivered once reload completes.
  • Reset pending viewed-map state alongside existing VTR sync state.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 87 to +92
if (Multiplayer.reloading)
{
pendingPreviousMapId = previous;
pendingCurrentMapId = current;
return;
}
@Zetrith

Zetrith commented Jun 28, 2026

Copy link
Copy Markdown
Member

This just brings back #862 which #865 was supposed to fix

@MhaWay

MhaWay commented Jun 28, 2026

Copy link
Copy Markdown
Contributor Author

This just brings back #862 which #865 was supposed to fix

Not yet tested that, I'll do ad soon as possible

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.

3 participants