Remove redundant forward declaration and server-side CS_SVDEMO set

Forward declaration of SVD_CleanupPlayback no longer needed (non-static,
declared in server.h). Server-side SV_SetConfigstring(CS_SVDEMO) is
redundant — G_InitGame sets it from the cvar which is now reliably
set before devmap.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
serge_shubin 2026-03-24 06:35:28 +08:00
parent 72d5df4ec9
commit 3d8291658f

View file

@ -107,8 +107,6 @@ typedef struct {
} svDemo_t; } svDemo_t;
static svDemo_t demo; static svDemo_t demo;
void SVD_CleanupPlayback( void );
// --------------------------------------------------------------- // ---------------------------------------------------------------
// Recording helpers // Recording helpers
// --------------------------------------------------------------- // ---------------------------------------------------------------
@ -831,11 +829,7 @@ void SVD_Play_f( void ) {
Cbuf_ExecuteText( EXEC_NOW, va("devmap %s\n", demo.playMapName) ); Cbuf_ExecuteText( EXEC_NOW, va("devmap %s\n", demo.playMapName) );
demo.starting = qfalse; demo.starting = qfalse;
// Also set configstring directly for cgame (must be after devmap // CS_SVDEMO configstring is set by G_InitGame from the cvar
// which creates the server and allocates configstrings)
if ( sv.state == SS_GAME ) {
SV_SetConfigstring( CS_SVDEMO, "1" );
}
// Reserve recorded player slots. Server is fresh (SV_Shutdown cleared // Reserve recorded player slots. Server is fresh (SV_Shutdown cleared
// old clients), local client hasn't connected yet. // old clients), local client hasn't connected yet.