]> de.git.xonotic.org Git - xonotic/darkplaces.git/blobdiff - cvar.c
Cleaned up alot more memory leaks. (still get 720 leaks just running demo1.dem)
[xonotic/darkplaces.git] / cvar.c
diff --git a/cvar.c b/cvar.c
index b51e87aa276195e821a382557c9ab6a810fa2d47..cdd4def5929690c360274cd7c68309d7ea3fe25e 100644 (file)
--- a/cvar.c
+++ b/cvar.c
@@ -8,7 +8,7 @@ of the License, or (at your option) any later version.
 
 This program is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 
 See the GNU General Public License for more details.
 
@@ -21,33 +21,57 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 
 #include "quakedef.h"
 
-cvar_t *cvar_vars;
-char   *cvar_null_string = "";
+cvar_t *cvar_vars = NULL;
+char *cvar_null_string = "";
 
 /*
 ============
 Cvar_FindVar
 ============
 */
-cvar_t *Cvar_FindVar (char *var_name)
+cvar_t *Cvar_FindVar (const char *var_name)
 {
-       cvar_t  *var;
+       cvar_t *var;
 
-       for (var=cvar_vars ; var ; var=var->next)
-               if (!strcmp (var_name, var->name))
+       for (var = cvar_vars;var;var = var->next)
+               if (!strcasecmp (var_name, var->name))
                        return var;
 
        return NULL;
 }
 
+cvar_t *Cvar_FindVarAfter (const char *prev_var_name, int neededflags)
+{
+       cvar_t *var;
+
+       if (*prev_var_name)
+       {
+               var = Cvar_FindVar (prev_var_name);
+               if (!var)
+                       return NULL;
+               var = var->next;
+       }
+       else
+               var = cvar_vars;
+
+       // search for the next cvar matching the needed flags
+       while (var)
+       {
+               if ((var->flags & neededflags) || !neededflags)
+                       break;
+               var = var->next;
+       }
+       return var;
+}
+
 /*
 ============
 Cvar_VariableValue
 ============
 */
-float  Cvar_VariableValue (char *var_name)
+float Cvar_VariableValue (const char *var_name)
 {
-       cvar_t  *var;
+       cvar_t *var;
 
        var = Cvar_FindVar (var_name);
        if (!var)
@@ -61,7 +85,7 @@ float Cvar_VariableValue (char *var_name)
 Cvar_VariableString
 ============
 */
-char *Cvar_VariableString (char *var_name)
+const char *Cvar_VariableString (const char *var_name)
 {
        cvar_t *var;
 
@@ -77,7 +101,7 @@ char *Cvar_VariableString (char *var_name)
 Cvar_CompleteVariable
 ============
 */
-char *Cvar_CompleteVariable (char *partial)
+const char *Cvar_CompleteVariable (const char *partial)
 {
        cvar_t          *cvar;
        int                     len;
@@ -89,7 +113,7 @@ char *Cvar_CompleteVariable (char *partial)
 
 // check functions
        for (cvar=cvar_vars ; cvar ; cvar=cvar->next)
-               if (!strncmp (partial,cvar->name, len))
+               if (!strncasecmp (partial,cvar->name, len))
                        return cvar->name;
 
        return NULL;
@@ -104,8 +128,7 @@ char *Cvar_CompleteVariable (char *partial)
        Thanks to Fett erich@heintz.com
 
 */
-int
-Cvar_CompleteCountPossible (char *partial)
+int Cvar_CompleteCountPossible (const char *partial)
 {
        cvar_t  *cvar;
        int             len;
@@ -134,17 +157,16 @@ Cvar_CompleteCountPossible (char *partial)
        Thanks to taniwha
 
 */
-char   **
-Cvar_CompleteBuildList (char *partial)
+const char **Cvar_CompleteBuildList (const char *partial)
 {
-       cvar_t  *cvar;
-       int             len = 0;
-       int             bpos = 0;
-       int             sizeofbuf = (Cvar_CompleteCountPossible (partial) + 1) * sizeof (char *);
-       char    **buf;
+       const cvar_t *cvar;
+       int len = 0;
+       int bpos = 0;
+       int sizeofbuf = (Cvar_CompleteCountPossible (partial) + 1) * sizeof (const char *);
+       const char **buf;
 
        len = strlen(partial);
-       buf = Mem_Alloc(tempmempool, sizeofbuf + sizeof (char *));
+       buf = Mem_Alloc(tempmempool, sizeofbuf + sizeof (const char *));
        // Loop through the alias list and print all matches
        for (cvar = cvar_vars; cvar; cvar = cvar->next)
                if (!strncasecmp(partial, cvar->name, len))
@@ -159,16 +181,10 @@ Cvar_CompleteBuildList (char *partial)
 Cvar_Set
 ============
 */
-void Cvar_SetQuick (cvar_t *var, char *value)
+void Cvar_SetQuick_Internal (cvar_t *var, const char *value)
 {
        qboolean changed;
 
-       if (var == NULL)
-       {
-               Con_Printf("Cvar_SetQuick: var == NULL\n");
-               return;
-       }
-
        changed = strcmp(var->string, value);
        // LordHavoc: don't reallocate when there is no change
        if (!changed)
@@ -184,24 +200,33 @@ void Cvar_SetQuick (cvar_t *var, char *value)
        strcpy (var->string, value);
        var->value = atof (var->string);
        var->integer = (int) var->value;
-       if ((var->flags & CVAR_NOTIFY) && changed)
+       if ((var->flags & CVAR_NOTIFY) && changed && sv.active)
+               SV_BroadcastPrintf("\"%s\" changed to \"%s\"\n", var->name, var->string);
+}
+
+void Cvar_SetQuick (cvar_t *var, const char *value)
+{
+       if (var == NULL)
        {
-               if (sv.active)
-                       SV_BroadcastPrintf ("\"%s\" changed to \"%s\"\n", var->name, var->string);
+               Con_Print("Cvar_SetQuick: var == NULL\n");
+               return;
        }
+
+       if (developer.integer)
+               Con_Printf("Cvar_SetQuick({\"%s\", \"%s\", %i}, \"%s\");\n", var->name, var->string, var->flags, value);
+
+       Cvar_SetQuick_Internal(var, value);
 }
 
-void Cvar_Set (char *var_name, char *value)
+void Cvar_Set (const char *var_name, const char *value)
 {
        cvar_t *var;
        var = Cvar_FindVar (var_name);
        if (var == NULL)
        {
-               // there is an error in C code if this happens
-               Con_Printf ("Cvar_Set: variable %s not found\n", var_name);
+               Con_Printf("Cvar_Set: variable %s not found\n", var_name);
                return;
        }
-
        Cvar_SetQuick(var, value);
 }
 
@@ -210,22 +235,26 @@ void Cvar_Set (char *var_name, char *value)
 Cvar_SetValue
 ============
 */
-void Cvar_SetValueQuick (cvar_t *var, float value)
+void Cvar_SetValueQuick(cvar_t *var, float value)
 {
-       char    val[32];
+       char val[256];
 
-       // LordHavoc: changed from %f to %g to use shortest representation
-       sprintf (val, "%g",value);
-       Cvar_SetQuick (var, val);
+       if ((float)((int)value) == value)
+               sprintf(val, "%i", (int)value);
+       else
+               sprintf(val, "%f", value);
+       Cvar_SetQuick(var, val);
 }
 
-void Cvar_SetValue (char *var_name, float value)
+void Cvar_SetValue(const char *var_name, float value)
 {
-       char    val[32];
+       char val[256];
 
-       // LordHavoc: changed from %f to %g to use shortest representation
-       sprintf (val, "%g",value);
-       Cvar_Set (var_name, val);
+       if ((float)((int)value) == value)
+               sprintf(val, "%i", (int)value);
+       else
+               sprintf(val, "%f", value);
+       Cvar_Set(var_name, val);
 }
 
 /*
@@ -237,19 +266,58 @@ Adds a freestanding variable to the variable list.
 */
 void Cvar_RegisterVariable (cvar_t *variable)
 {
-       char    *oldstr;
+       cvar_t *cvar, *cvar2;
+       char *oldstr;
+
+       if (developer.integer)
+               Con_Printf("Cvar_RegisterVariable({\"%s\", \"%s\", %i});\n", variable->name, variable->string, variable->flags);
 
 // first check to see if it has already been defined
-       if (Cvar_FindVar (variable->name))
+       cvar = Cvar_FindVar (variable->name);
+       if (cvar)
        {
-               Con_Printf ("Can't register variable %s, already defined\n", variable->name);
+               if (cvar->flags & CVAR_ALLOCATED)
+               {
+                       if (developer.integer)
+                               Con_Printf("...  replacing existing allocated cvar {\"%s\", \"%s\", %i}", cvar->name, cvar->string, cvar->flags);
+                       // fixed variables replace allocated ones
+                       // (because the engine directly accesses fixed variables)
+                       // NOTE: this isn't actually used currently
+                       // (all cvars are registered before config parsing)
+                       variable->flags |= (cvar->flags & ~CVAR_ALLOCATED);
+                       // cvar->string is now owned by variable instead
+                       variable->string = cvar->string;
+                       variable->value = atof (variable->string);
+                       variable->integer = (int) variable->value;
+                       // replace cvar with this one...
+                       variable->next = cvar->next;
+                       if (cvar_vars == cvar)
+                       {
+                               // head of the list is easy to change
+                               cvar_vars = variable;
+                       }
+                       else
+                       {
+                               // otherwise find it somewhere in the list
+                               for (cvar2 = cvar_vars;cvar2->next != cvar;cvar2 = cvar2->next);
+                               if (cvar2->next == cvar)
+                                       cvar2->next = variable;
+                       }
+
+                       // get rid of old allocated cvar
+                       // (but not the cvar->string, because we kept that)
+                       Z_Free(cvar->name);
+                       Z_Free(cvar);
+               }
+               else
+                       Con_Printf("Can't register variable %s, already defined\n", variable->name);
                return;
        }
 
 // check for overlap with a command
        if (Cmd_Exists (variable->name))
        {
-               Con_Printf ("Cvar_RegisterVariable: %s is a command\n", variable->name);
+               Con_Printf("Cvar_RegisterVariable: %s is a command\n", variable->name);
                return;
        }
 
@@ -265,6 +333,54 @@ void Cvar_RegisterVariable (cvar_t *variable)
        cvar_vars = variable;
 }
 
+/*
+============
+Cvar_Get
+
+Adds a newly allocated variable to the variable list or sets its value.
+============
+*/
+cvar_t *Cvar_Get (const char *name, const char *value, int flags)
+{
+       cvar_t *cvar;
+
+       if (developer.integer)
+               Con_Printf("Cvar_Get(\"%s\", \"%s\", %i);\n", name, value, flags);
+       
+// first check to see if it has already been defined
+       cvar = Cvar_FindVar (name);
+       if (cvar)
+       {
+               cvar->flags |= flags;
+               Cvar_SetQuick_Internal (cvar, value);
+               return cvar;
+       }
+
+// check for overlap with a command
+       if (Cmd_Exists (name))
+       {
+               Con_Printf("Cvar_Get: %s is a command\n", name);
+               return NULL;
+       }
+
+// allocate a new cvar, cvar name, and cvar string
+// FIXME: these never get Z_Free'd
+       cvar = Z_Malloc(sizeof(cvar_t));
+       cvar->flags = flags | CVAR_ALLOCATED;
+       cvar->name = Z_Malloc(strlen(name)+1);
+       strcpy(cvar->name, name);
+       cvar->string = Z_Malloc(strlen(value)+1);
+       strcpy(cvar->string, value);
+       cvar->value = atof (cvar->string);
+       cvar->integer = (int) cvar->value;
+
+// link the variable in
+       cvar->next = cvar_vars;
+       cvar_vars = cvar;
+       return cvar;
+}
+
+
 /*
 ============
 Cvar_Command
@@ -284,10 +400,20 @@ qboolean  Cvar_Command (void)
 // perform a variable print or set
        if (Cmd_Argc() == 1)
        {
-               Con_Printf ("\"%s\" is \"%s\"\n", v->name, v->string);
+               // only print if host_initialized (otherwise it could print twice if this is in a script)
+               if (host_initialized)
+                       Con_Printf("\"%s\" is \"%s\"\n", v->name, v->string);
                return true;
        }
 
+       if (developer.integer)
+               Con_Print("Cvar_Command: ");
+
+       if (v->flags & CVAR_READONLY)
+       {
+               Con_Printf("%s is read-only\n", v->name);
+               return true;
+       }
        Cvar_Set (v->name, Cmd_Argv(1));
        return true;
 }
@@ -301,13 +427,13 @@ Writes lines containing "set variable value" for all variables
 with the archive flag set to true.
 ============
 */
-void Cvar_WriteVariables (QFile *f)
+void Cvar_WriteVariables (qfile_t *f)
 {
        cvar_t  *var;
 
        for (var = cvar_vars ; var ; var = var->next)
                if (var->flags & CVAR_SAVE)
-                       Qprintf (f, "%s \"%s\"\n", var->name, var->string);
+                       FS_Printf(f, "%s%s \"%s\"\n", var->flags & CVAR_ALLOCATED ? "seta " : "", var->name, var->string);
 }
 
 
@@ -320,31 +446,88 @@ Cvar_List
 */
 void Cvar_List_f (void)
 {
-       cvar_t  *cvar;
-       char    *partial;
-       int             len;
-       int             count;
+       cvar_t *cvar;
+       const char *partial;
+       int len, count;
 
-       if (Cmd_Argc() > 1) {
+       if (Cmd_Argc() > 1)
+       {
                partial = Cmd_Argv (1);
                len = strlen(partial);
-       } else {
+       }
+       else
+       {
                partial = NULL;
                len = 0;
        }
 
        count = 0;
-       for (cvar = cvar_vars; cvar; cvar = cvar->next) {
-               if (partial && strncmp (partial,cvar->name,len))
+       for (cvar = cvar_vars; cvar; cvar = cvar->next)
+       {
+               if (partial && strncasecmp (partial,cvar->name,len))
                        continue;
 
-               Con_Printf ("%s is \"%s\"\n", cvar->name, cvar->string);
+               Con_Printf("%s is \"%s\"\n", cvar->name, cvar->string);
                count++;
        }
 
-       Con_Printf ("%i cvar(s)", count);
+       Con_Printf("%i cvar(s)", count);
        if (partial)
-               Con_Printf (" beginning with \"%s\"", partial);
-       Con_Print("\n");
+               Con_Printf(" beginning with \"%s\"", partial);
+       Con_Print("\n");
 }
 // 2000-01-09 CvarList command by Maddes
+
+void Cvar_Set_f (void)
+{
+       cvar_t *cvar;
+
+       // make sure it's the right number of parameters
+       if (Cmd_Argc() < 3)
+       {
+               Con_Printf("Set: wrong number of parameters, usage: set <variablename> <value>\n");
+               return;
+       }
+
+       // check if it's read-only
+       cvar = Cvar_FindVar(Cmd_Argv(1));
+       if (cvar && cvar->flags & CVAR_READONLY)
+       {
+               Con_Printf("Set: %s is read-only\n", cvar->name);
+               return;
+       }
+
+       if (developer.integer)
+               Con_Print("Set: ");
+
+       // all looks ok, create/modify the cvar
+       Cvar_Get(Cmd_Argv(1), Cmd_Argv(2), 0);
+}
+
+void Cvar_SetA_f (void)
+{
+       cvar_t *cvar;
+
+       // make sure it's the right number of parameters
+       if (Cmd_Argc() < 3)
+       {
+               Con_Printf("SetA: wrong number of parameters, usage: seta <variablename> <value>\n");
+               return;
+       }
+
+       // check if it's read-only
+       cvar = Cvar_FindVar(Cmd_Argv(1));
+       if (cvar && cvar->flags & CVAR_READONLY)
+       {
+               Con_Printf("SetA: %s is read-only\n", cvar->name);
+               return;
+       }
+
+       if (developer.integer)
+               Con_Print("SetA: ");
+
+       // all looks ok, create/modify the cvar
+       Cvar_Get(Cmd_Argv(1), Cmd_Argv(2), CVAR_SAVE);
+}
+
+