Decouple execversion from MODVERSION; add scheme for MINORMODVERSION

This commit is contained in:
mazmazz 2018-12-17 13:38:23 -05:00
parent 1029463741
commit 657e7321f6
3 changed files with 18 additions and 8 deletions

View File

@ -63,7 +63,7 @@ CV_PossibleValue_t CV_YesNo[] = {{0, "No"}, {1, "Yes"}, {0, NULL}};
CV_PossibleValue_t CV_Unsigned[] = {{0, "MIN"}, {999999999, "MAX"}, {0, NULL}};
CV_PossibleValue_t CV_Natural[] = {{1, "MIN"}, {999999999, "MAX"}, {0, NULL}};
// Filter consvars by MODVERSION
// Filter consvars by EXECVERSION
// First implementation is 26 (2.1.21), so earlier configs default at 25 (2.1.20)
// Also set CV_HIDEN during runtime, after config is loaded
consvar_t cv_execversion = {"execversion","25",0,CV_Unsigned, NULL, 0, NULL, NULL, 0, 0, NULL};
@ -1589,7 +1589,7 @@ void CV_InitFilterVar(void)
static boolean CV_FilterJoyAxisVars(consvar_t *v, const char *valstr)
{
// If ALL axis settings are previous defaults, set them to the new defaults
// MODVERSION < 26 (2.1.21)
// EXECVERSION < 26 (2.1.21)
if (joyaxis_default)
{
@ -1730,6 +1730,11 @@ static boolean CV_FilterJoyAxisVars(consvar_t *v, const char *valstr)
static boolean CV_FilterVarByVersion(consvar_t *v, const char *valstr)
{
INT32 majorexecversion = abs(cv_execversion.value) & 0xFFFF;
#if 0 // unused for now
INT32 minorexecversion = abs(cv_execversion.value) >> 16;
#endif
// True means allow the CV change, False means block it
// We only care about CV_SAVE because this filters the user's config files
@ -1737,8 +1742,7 @@ static boolean CV_FilterVarByVersion(consvar_t *v, const char *valstr)
if (!(v->flags & CV_SAVE))
return true;
// We go by MODVERSION here
if (cv_execversion.value < 26) // 26 = 2.1.21
if (majorexecversion < 26) // 26 = 2.1.21
{
// MOUSE SETTINGS
// alwaysfreelook split between first and third person (chasefreelook)

View File

@ -216,6 +216,12 @@ extern FILE *logstream;
// Note that we use this to help keep internal testing in check; this is why v2.1.0 is not version "1".
#define MODVERSION 26
// To version config.cfg, EXECVERSION was originally equal to MODVERSION
// So keep it equal to MODVERSION, and increment MINOREXECVERSION whenever a config change
// is needed that does not correspond to an increment in MODVERSION
#define EXECVERSION 26
#define MINOREXECVERSION 1
// =========================================================================
// The maximum number of players, multiplayer/networking.

View File

@ -451,7 +451,7 @@ void Command_LoadConfig_f(void)
COM_BufInsertText(va("exec \"%s\"\n", configfile));
// don't filter anymore vars and don't let this convsvar be changed
COM_BufInsertText(va("%s \"%d\"\n", cv_execversion.name, MODVERSION));
COM_BufInsertText(va("%s \"%d\"\n", cv_execversion.name, EXECVERSION));
cv_execversion.flags |= CV_HIDEN;
}
@ -503,7 +503,7 @@ void M_FirstLoadConfig(void)
// no COM_BufExecute() needed; that does it right away
// don't filter anymore vars and don't let this convsvar be changed
COM_BufInsertText(va("%s \"%d\"\n", cv_execversion.name, MODVERSION));
COM_BufInsertText(va("%s \"%d\"\n", cv_execversion.name, EXECVERSION));
cv_execversion.flags |= CV_HIDEN;
// make sure I_Quit() will write back the correct config
@ -562,8 +562,8 @@ void M_SaveConfig(const char *filename)
fprintf(f, "// SRB2 configuration file.\n");
// print execversion FIRST, because subsequent consvars need to be filtered
// always print current MODVERSION
fprintf(f, "%s \"%d\"\n", cv_execversion.name, MODVERSION);
// always print current EXECVERSION
fprintf(f, "%s \"%d\"\n", cv_execversion.name, (EXECVERSION + (MINOREXECVERSION << 16)));
// FIXME: save key aliases if ever implemented..