Skip to content

Conversation

@Rainyan
Copy link
Collaborator

@Rainyan Rainyan commented Jul 10, 2025

Description

Add the neo_restart_this cvar for parity.

This cvar differs from mp_restartgame in that the latter will give a countdown, whereas neo_restart_this is instant.

As a side note, the cvar mp_restartgame_immediate seen below is CTeamplayRoundBasedRules related, which is not part of our gamerules inheritance chain, and its logic is not quite right, so that's why I'm rolling my own callback for the cvar.

Toolchain

  • Windows MSVC VS2022

Linked Issues

Rainyan added 2 commits July 17, 2025 14:21
Add the neo_restart_this cvar for parity.

This cvar differs from mp_restartgame in that the latter will give a
countdown whereas neo_restart_this is instant.
@Rainyan Rainyan force-pushed the bug/neo-restart-this branch from b4770cc to 9f3ae20 Compare July 17, 2025 11:21
@Rainyan Rainyan requested a review from a team July 17, 2025 11:22
@AdamTadeusz
Copy link
Contributor

Most definitely unrelated, but restarting a bunch before the round started gave me this error

image

@AdamTadeusz
Copy link
Contributor

Can't come up with a good way to replicate it it seems very random

Copy link
Contributor

@AdamTadeusz AdamTadeusz left a comment

Choose a reason for hiding this comment

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

We know this is an issue already that needs fixing so approving anyway

@Rainyan
Copy link
Collaborator Author

Rainyan commented Jul 23, 2025

Most definitely unrelated, but restarting a bunch before the round started gave me this error
image

Very interesting. I'll try and replicate too, later. Probably unrelated to this PR as you said but would be nice to fix at some point for good.

@Rainyan Rainyan requested a review from a team July 23, 2025 13:52
Copy link
Contributor

@sunzenshen sunzenshen left a comment

Choose a reason for hiding this comment

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

LGTM. If anything, the real issue needs to be debugged from RestartGame() if that was related to the bug, rather than from this wrapper.

@Rainyan Rainyan merged commit 1c47aa8 into NeotokyoRebuild:master Jul 25, 2025
13 of 14 checks passed
@Rainyan Rainyan deleted the bug/neo-restart-this branch July 25, 2025 08:42
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