vilyaem/Xrasher
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert "common: cvar: make auto descriptions private to cvar subsystem, use them when Cvar_Get is called with NULL description"

Browse source 
This reverts commit ade560c601.
This commit is contained in:
Alibek Omarov 2025-02-10 12:47:39 +03:00
parent 3550bdb30d
commit 627444279f
5 changed files with 13 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
engine/client/cl_game.c
View file

@ -1766,7 +1766,7 @@ static cvar_t *GAME_EXPORT pfnCvar_RegisterClientVariable( const char *szName, c
|| !Q_stricmp( szName, "sensitivity" )) || !Q_stricmp( szName, "sensitivity" ))
flags |= FCVAR_PRIVILEGED; flags |= FCVAR_PRIVILEGED;
return (cvar_t *)Cvar_Get( szName, szValue, flags|FCVAR_CLIENTDLL, NULL ); return (cvar_t *)Cvar_Get( szName, szValue, flags|FCVAR_CLIENTDLL, Cvar_BuildAutoDescription( szName, flags|FCVAR_CLIENTDLL ));
} }
static int GAME_EXPORT Cmd_AddClientCommand( const char *cmd_name, xcommand_t function ) static int GAME_EXPORT Cmd_AddClientCommand( const char *cmd_name, xcommand_t function )

2
engine/client/cl_gameui.c
View file

@ -691,7 +691,7 @@ pfnCvar_RegisterVariable
*/ */
static cvar_t *GAME_EXPORT pfnCvar_RegisterGameUIVariable( const char *szName, const char *szValue, int flags ) static cvar_t *GAME_EXPORT pfnCvar_RegisterGameUIVariable( const char *szName, const char *szValue, int flags )
{ {
return (cvar_t *)Cvar_Get( szName, szValue, flags|FCVAR_GAMEUIDLL, NULL ); return (cvar_t *)Cvar_Get( szName, szValue, flags|FCVAR_GAMEUIDLL, Cvar_BuildAutoDescription( szName, flags|FCVAR_GAMEUIDLL ));
} }
static int GAME_EXPORT Cmd_AddGameUICommand( const char *cmd_name, xcommand_t function ) static int GAME_EXPORT Cmd_AddGameUICommand( const char *cmd_name, xcommand_t function )

16
engine/common/cvar.c
View file

@ -103,10 +103,16 @@ Cvar_BuildAutoDescription
build cvar auto description that based on the setup flags build cvar auto description that based on the setup flags
============ ============
*/ */
static const char *Cvar_BuildAutoDescription( int flags ) const char *Cvar_BuildAutoDescription( const char *szName, int flags )
{ {
static char desc[256]; static char desc[256];
if( FBitSet( flags, FCVAR_GLCONFIG ))
{
Q_snprintf( desc, sizeof( desc ), CVAR_GLCONFIG_DESCRIPTION, szName );
return desc;
}
desc[0] = '\0'; desc[0] = '\0';
if( FBitSet( flags, FCVAR_EXTDLL )) if( FBitSet( flags, FCVAR_EXTDLL ))
@ -441,7 +447,7 @@ convar_t *Cvar_Get( const char *name, const char *value, int flags, const char *
Cvar_DirectSet( var, value ); Cvar_DirectSet( var, value );
} }
if( FBitSet( var->flags, FCVAR_ALLOCATED ) && var_desc != NULL && Q_strcmp( var_desc, var->desc )) if( FBitSet( var->flags, FCVAR_ALLOCATED ) && Q_strcmp( var_desc, var->desc ))
{ {
if( !FBitSet( flags, FCVAR_GLCONFIG )) if( !FBitSet( flags, FCVAR_GLCONFIG ))
Con_Reportf( "%s change description from %s to %s\n", var->name, var->desc, var_desc ); Con_Reportf( "%s change description from %s to %s\n", var->name, var->desc, var_desc );
@ -454,10 +460,6 @@ convar_t *Cvar_Get( const char *name, const char *value, int flags, const char *
} }
// allocate a new cvar // allocate a new cvar
if( !var_desc )
var_desc = Cvar_BuildAutoDescription( flags );
var = Mem_Malloc( cvar_pool, sizeof( *var )); var = Mem_Malloc( cvar_pool, sizeof( *var ));
var->name = copystringpool( cvar_pool, name ); var->name = copystringpool( cvar_pool, name );
var->string = copystringpool( cvar_pool, value ); var->string = copystringpool( cvar_pool, value );
@ -1204,7 +1206,7 @@ static void Cvar_List_f( void )
if( FBitSet( var->flags, FCVAR_EXTENDED|FCVAR_ALLOCATED )) if( FBitSet( var->flags, FCVAR_EXTENDED|FCVAR_ALLOCATED ))
Con_Printf( " %-*s %s ^3%s^7\n", 32, var->name, value, var->desc ); Con_Printf( " %-*s %s ^3%s^7\n", 32, var->name, value, var->desc );
else Con_Printf( " %-*s %s ^3%s^7\n", 32, var->name, value, Cvar_BuildAutoDescription( var->flags )); else Con_Printf( " %-*s %s ^3%s^7\n", 32, var->name, value, Cvar_BuildAutoDescription( var->name, var->flags ));
count++; count++;
} }

1
engine/common/cvar.h
View file

@ -44,6 +44,7 @@ void Cvar_DirectSet( convar_t *var, const char *value );
void Cvar_DirectSetValue( convar_t *var, float value ); void Cvar_DirectSetValue( convar_t *var, float value );
void Cvar_Set( const char *var_name, const char *value ); void Cvar_Set( const char *var_name, const char *value );
void Cvar_SetValue( const char *var_name, float value ); void Cvar_SetValue( const char *var_name, float value );
const char *Cvar_BuildAutoDescription( const char *szName, int flags ) RETURNS_NONNULL;
float Cvar_VariableValue( const char *var_name ); float Cvar_VariableValue( const char *var_name );
int Cvar_VariableInteger( const char *var_name ); int Cvar_VariableInteger( const char *var_name );
const char *Cvar_VariableString( const char *var_name ) RETURNS_NONNULL; const char *Cvar_VariableString( const char *var_name ) RETURNS_NONNULL;

4
engine/common/filesystem_engine.c
View file

@ -258,8 +258,7 @@ static qboolean FS_DetermineReadOnlyRootDirectory( char *out, size_t size )
void FS_CheckConfig( void ) void FS_CheckConfig( void )
{ {
// only used to prevent rescan after reading config.cfg when user hasn't enabled any addon directories if( fs_mount_lv.value || fs_mount_hd.value || fs_mount_addon.value || fs_mount_l10n.value )
if( fs_mount_lv.value || fs_mount_hd.value || fs_mount_addon.value || fs_mount_l10n.value || Q_stricmp( ui_language.string, "english" ))
FS_Rescan_f(); FS_Rescan_f();
} }
@ -319,7 +318,6 @@ void FS_Init( const char *basedir )
Cvar_RegisterVariable( &fs_mount_lv ); Cvar_RegisterVariable( &fs_mount_lv );
Cvar_RegisterVariable( &fs_mount_addon ); Cvar_RegisterVariable( &fs_mount_addon );
Cvar_RegisterVariable( &fs_mount_l10n ); Cvar_RegisterVariable( &fs_mount_l10n );
Cvar_RegisterVariable( &ui_language );
if( !Sys_GetParmFromCmdLine( "-dll", host.gamedll )) if( !Sys_GetParmFromCmdLine( "-dll", host.gamedll ))
host.gamedll[0] = 0; host.gamedll[0] = 0;