Skip to content

Conversation

@sprunk
Copy link
Collaborator

@sprunk sprunk commented Jan 8, 2026

The code is incorrect (path always leads to assertion), nothing actually sends this, and nothing should send it because the way to start by a local player is /forcestart rather than a packet.

The code is incorrect (path always leads to assertion), nothing actually sends this, and nothing should send it because the way to start by a local player is `/forcestart` rather than a packet.
@sprunk
Copy link
Collaborator Author

sprunk commented Jan 8, 2026

Always asserts due to gameHasStarted:

case NETMSG_STARTPLAYING: {
if (players[a].isLocal && gameHasStarted)
CheckForGameStart(true);

void CGameServer::CheckForGameStart(bool forced)
{
assert(!gameHasStarted);

Proper way to do this is already /forcestart:

case hashString("forcestart"): {
if (!gameHasStarted)
CheckForGameStart(true);

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.

2 participants