]> de.git.xonotic.org Git - xonotic/darkplaces.git/blobdiff - cmd.c
build: minor adjustments
[xonotic/darkplaces.git] / cmd.c
diff --git a/cmd.c b/cmd.c
index f3d0a1cb1a5bbefeb92db6535b33eba2d03e4f3d..3e35b336b9c3d3914385336289152e111895a95d 100644 (file)
--- a/cmd.c
+++ b/cmd.c
@@ -22,9 +22,8 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 #include "quakedef.h"
 #include "thread.h"
 
-cmd_state_t cmd_client;
-cmd_state_t cmd_server;
-cmd_state_t cmd_serverfromclient;
+cmd_state_t *cmd_local;
+cmd_state_t *cmd_serverfromclient;
 
 cmd_userdefined_t cmd_userdefined_all;
 cmd_userdefined_t cmd_userdefined_null;
@@ -34,27 +33,23 @@ typedef struct cmd_iter_s {
 }
 cmd_iter_t;
 
-static cmd_iter_t cmd_iter_all[] = {
-       {&cmd_client},
-       {&cmd_server},
-       {&cmd_serverfromclient},
-       {NULL},
-};
+static cmd_iter_t *cmd_iter_all;
 
+mempool_t *cbuf_mempool;
 
 // we only run the +whatever commandline arguments once
-qboolean host_stuffcmdsrun = false;
+qbool host_stuffcmdsrun = false;
 
 //=============================================================================
 
-void Cbuf_Lock(cmd_state_t *cmd)
+void Cbuf_Lock(cmd_buf_t *cbuf)
 {
-       Thread_AtomicLock(&cmd->text_lock);
+       Thread_LockMutex(cbuf->lock);
 }
 
-void Cbuf_Unlock(cmd_state_t *cmd)
+void Cbuf_Unlock(cmd_buf_t *cbuf)
 {
-       Thread_AtomicUnlock(&cmd->text_lock);
+       Thread_UnlockMutex(cbuf->lock);
 }
 
 
@@ -69,7 +64,7 @@ bind g "impulse 5 ; +attack ; wait ; -attack ; impulse 2"
 */
 static void Cmd_Wait_f (cmd_state_t *cmd)
 {
-       cmd->wait = true;
+       cmd->cbuf->wait = true;
 }
 
 /*
@@ -79,106 +74,185 @@ Cmd_Defer_f
 Cause a command to be executed after a delay.
 ============
 */
+static void Cbuf_ParseText(cmd_state_t *cmd, llist_t *head, cmd_input_t *existing, const char *text, qbool allowpending);
+static void Cbuf_LinkString(cmd_state_t *cmd, llist_t *head, cmd_input_t *existing, const char *text, qbool leavepending, unsigned int cmdsize);
 static void Cmd_Defer_f (cmd_state_t *cmd)
 {
+       cmd_input_t *current;
+       cmd_buf_t *cbuf = cmd->cbuf;
+       unsigned int cmdsize;
+
        if(Cmd_Argc(cmd) == 1)
        {
-               cmddeferred_t *next = cmd->deferred_list;
-               if(!next)
+               if(List_Is_Empty(&cbuf->deferred))
                        Con_Printf("No commands are pending.\n");
-               while(next)
+               else
                {
-                       Con_Printf("-> In %9.2f: %s\n", next->delay, next->value);
-                       next = next->next;
+                       List_For_Each_Entry(current, &cbuf->deferred, cmd_input_t, list)
+                               Con_Printf("-> In %9.2f: %s\n", current->delay, current->text);
                }
-       } else if(Cmd_Argc(cmd) == 2 && !strcasecmp("clear", Cmd_Argv(cmd, 1)))
+       }
+       else if(Cmd_Argc(cmd) == 2 && !strcasecmp("clear", Cmd_Argv(cmd, 1)))
        {
-               while(cmd->deferred_list)
+               while(!List_Is_Empty(&cbuf->deferred))
                {
-                       cmddeferred_t *defcmd = cmd->deferred_list;
-                       cmd->deferred_list = defcmd->next;
-                       Mem_Free(defcmd->value);
-                       Mem_Free(defcmd);
+                       cbuf->size -= List_Entry(cbuf->deferred.next, cmd_input_t, list)->length;
+                       List_Move_Tail(cbuf->deferred.next, &cbuf->free);
                }
-       } else if(Cmd_Argc(cmd) == 3)
+       }
+       else if(Cmd_Argc(cmd) == 3 && (cmdsize = strlen(Cmd_Argv(cmd, 2))) )
        {
-               const char *value = Cmd_Argv(cmd, 2);
-               cmddeferred_t *defcmd = (cmddeferred_t*)Mem_Alloc(tempmempool, sizeof(*defcmd));
-               size_t len = strlen(value);
+               Cbuf_Lock(cbuf);
 
-               defcmd->delay = atof(Cmd_Argv(cmd, 1));
-               defcmd->value = (char*)Mem_Alloc(tempmempool, len+1);
-               memcpy(defcmd->value, value, len+1);
-               defcmd->next = NULL;
+               Cbuf_LinkString(cmd, &cbuf->deferred, NULL, Cmd_Argv(cmd, 2), false, cmdsize);
+               List_Entry(cbuf->deferred.prev, cmd_input_t, list)->delay = atof(Cmd_Argv(cmd, 1));
 
-               if(cmd->deferred_list)
-               {
-                       cmddeferred_t *next = cmd->deferred_list;
-                       while(next->next)
-                               next = next->next;
-                       next->next = defcmd;
-               } else
-                       cmd->deferred_list = defcmd;
-               /* Stupid me... this changes the order... so commands with the same delay go blub :S
-                 defcmd->next = cmd_deferred_list;
-                 cmd_deferred_list = defcmd;*/
-       } else {
+               Cbuf_Unlock(cbuf);
+       }
+       else
+       {
                Con_Printf("usage: defer <seconds> <command>\n"
                           "       defer clear\n");
                return;
        }
 }
 
+/*
+=============================================================================
+
+                                               COMMAND BUFFER
+
+ * The Quake command-line is super basic. It can be entered in the console
+ * or in config files. A semicolon is used to terminate a command and chain
+ * them together. Otherwise, a newline delineates command input.
+ *
+ * In most engines, the Quake command-line is a simple linear text buffer that
+ * is parsed when it executes. In Darkplaces, we use a linked list of command
+ * input and parse the input on the spot.
+ *
+ * This was done because Darkplaces allows multiple command interpreters on the
+ * same thread. Previously, each interpreter maintained its own buffer and this
+ * caused problems related to execution order, and maintaining a single simple
+ * buffer for all interpreters makes it non-trivial to keep track of which
+ * command should execute on which interpreter.
+
+=============================================================================
+*/
+
 /*
 ============
-Cmd_Centerprint_f
+Cbuf_NodeGet
 
-Print something to the center of the screen using SCR_Centerprint
+Returns an existing buffer node for appending or reuse, or allocates a new one
 ============
 */
-static void Cmd_Centerprint_f (cmd_state_t *cmd)
+static cmd_input_t *Cbuf_NodeGet(cmd_buf_t *cbuf, cmd_input_t *existing)
 {
-       char msg[MAX_INPUTLINE];
-       unsigned int i, c, p;
-       c = Cmd_Argc(cmd);
-       if(c >= 2)
+       cmd_input_t *node;
+       if(existing && existing->pending)
+               node = existing;
+       else if(!List_Is_Empty(&cbuf->free))
        {
-               strlcpy(msg, Cmd_Argv(cmd,1), sizeof(msg));
-               for(i = 2; i < c; ++i)
-               {
-                       strlcat(msg, " ", sizeof(msg));
-                       strlcat(msg, Cmd_Argv(cmd, i), sizeof(msg));
-               }
-               c = (unsigned int)strlen(msg);
-               for(p = 0, i = 0; i < c; ++i)
-               {
-                       if(msg[i] == '\\')
-                       {
-                               if(msg[i+1] == 'n')
-                                       msg[p++] = '\n';
-                               else if(msg[i+1] == '\\')
-                                       msg[p++] = '\\';
-                               else {
-                                       msg[p++] = '\\';
-                                       msg[p++] = msg[i+1];
-                               }
-                               ++i;
-                       } else {
-                               msg[p++] = msg[i];
-                       }
-               }
-               msg[p] = '\0';
-               SCR_CenterPrint(msg);
+               node = List_Entry(cbuf->free.next, cmd_input_t, list);
+               node->length = node->pending = 0;
        }
+       else
+       {
+               node = (cmd_input_t *)Mem_Alloc(cbuf_mempool, sizeof(cmd_input_t));
+               node->list.prev = node->list.next = &node->list;
+               node->size = node->length = node->pending = 0;
+       }
+       return node;
 }
 
 /*
-=============================================================================
+============
+Cbuf_LinkString
 
-                                               COMMAND BUFFER
+Copies a command string into a buffer node.
+The input should not be null-terminated, the output will be.
+============
+*/
+static void Cbuf_LinkString(cmd_state_t *cmd, llist_t *head, cmd_input_t *existing, const char *text, qbool leavepending, unsigned int cmdsize)
+{
+       cmd_buf_t *cbuf = cmd->cbuf;
+       cmd_input_t *node = Cbuf_NodeGet(cbuf, existing);
+       unsigned int offset = node->length; // > 0 if(pending)
 
-=============================================================================
+       // node will match existing if its text was pending continuation
+       if(node != existing)
+       {
+               node->source = cmd;
+               List_Move_Tail(&node->list, head);
+       }
+
+       node->length += cmdsize;
+       if(node->size < node->length)
+       {
+               node->text = (char *)Mem_Realloc(cbuf_mempool, node->text, node->length + 1);
+               node->size = node->length;
+       }
+       cbuf->size += cmdsize;
+
+       dp_ustr2stp(&node->text[offset], node->length + 1, text, cmdsize);
+       //Con_Printf("^5Cbuf_LinkString(): %s `^7%s^5`\n", node->pending ? "append" : "new", &node->text[offset]);
+       node->pending = leavepending;
+}
+
+/*
+============
+Cbuf_ParseText
+
+Parses text to isolate command strings for linking into the buffer
+separators: \n \r or unquoted and uncommented ';'
+============
 */
+static void Cbuf_ParseText(cmd_state_t *cmd, llist_t *head, cmd_input_t *existing, const char *text, qbool allowpending)
+{
+       unsigned int cmdsize = 0, start = 0, pos;
+       qbool quotes = false, comment = false;
+
+       for (pos = 0; text[pos]; ++pos)
+       {
+               switch(text[pos])
+               {
+                       case ';':
+                               if (comment || quotes)
+                                       break;
+                       case '\r':
+                       case '\n':
+                               comment = false;
+                               quotes = false; // matches div0-stable
+                               if (cmdsize)
+                               {
+                                       Cbuf_LinkString(cmd, head, existing, &text[start], false, cmdsize);
+                                       cmdsize = 0;
+                               }
+                               else if (existing && existing->pending) // all I got was this lousy \n
+                                       existing->pending = false;
+                               continue; // don't increment cmdsize
+
+                       case '/':
+                               if (!quotes && text[pos + 1] == '/' && (pos == 0 || ISWHITESPACE(text[pos - 1])))
+                                       comment = true;
+                               break;
+                       case '"':
+                               if (!comment && (pos == 0 || text[pos - 1] != '\\'))
+                                       quotes = !quotes;
+                               break;
+               }
+
+               if (!comment)
+               {
+                       if (!cmdsize)
+                               start = pos;
+                       ++cmdsize;
+               }
+       }
+
+       if (cmdsize) // the line didn't end yet but we do have a string
+               Cbuf_LinkString(cmd, head, existing, &text[start], allowpending, cmdsize);
+}
 
 /*
 ============
@@ -189,43 +263,53 @@ Adds command text at the end of the buffer
 */
 void Cbuf_AddText (cmd_state_t *cmd, const char *text)
 {
-       int             l;
+       size_t l = strlen(text);
+       cmd_buf_t *cbuf = cmd->cbuf;
+       llist_t llist = {&llist, &llist};
 
-       l = (int)strlen(text);
+       if (cbuf->size + l > cbuf->maxsize)
+       {
+               Con_Printf(CON_WARN "Cbuf_AddText: input too large, %luKB ought to be enough for anybody.\n", (unsigned long)(cbuf->maxsize / 1024));
+               return;
+       }
 
-       Cbuf_Lock(cmd);
-       if (cmd->text.maxsize - cmd->text.cursize <= l)
-               Con_Print("Cbuf_AddText: overflow\n");
-       else
-               SZ_Write(&cmd->text, (const unsigned char *)text, l);
-       Cbuf_Unlock(cmd);
-}
+       Cbuf_Lock(cbuf);
+
+       // If the string terminates but the (last) line doesn't, the node will be left in the pending state (to be continued).
+       Cbuf_ParseText(cmd, &llist, (List_Is_Empty(&cbuf->start) ? NULL : List_Entry(cbuf->start.prev, cmd_input_t, list)), text, true);
+       List_Splice_Tail(&llist, &cbuf->start);
 
+       Cbuf_Unlock(cbuf);
+}
 
 /*
 ============
 Cbuf_InsertText
 
 Adds command text immediately after the current command
-Adds a \n to the text
-FIXME: actually change the command buffer to do less copying
 ============
 */
 void Cbuf_InsertText (cmd_state_t *cmd, const char *text)
 {
+       cmd_buf_t *cbuf = cmd->cbuf;
+       llist_t llist = {&llist, &llist};
        size_t l = strlen(text);
-       Cbuf_Lock(cmd);
-       // we need to memmove the existing text and stuff this in before it...
-       if (cmd->text.cursize + l >= (size_t)cmd->text.maxsize)
-               Con_Print("Cbuf_InsertText: overflow\n");
-       else
+
+       if (cbuf->size + l > cbuf->maxsize)
        {
-               // we don't have a SZ_Prepend, so...
-               memmove(cmd->text.data + l, cmd->text.data, cmd->text.cursize);
-               cmd->text.cursize += (int)l;
-               memcpy(cmd->text.data, text, l);
+               Con_Printf(CON_WARN "Cbuf_InsertText: input too large, %luKB ought to be enough for anybody.\n", (unsigned long)(cbuf->maxsize / 1024));
+               return;
        }
-       Cbuf_Unlock(cmd);
+
+       Cbuf_Lock(cbuf);
+
+       // bones_was_here assertion: when prepending to the buffer it never makes sense to leave node(s) in the `pending` state,
+       // it would have been impossible to append to such text later in the old raw text buffer,
+       // and allowing it causes bugs when .cfg files lack \n at EOF (see: https://gitlab.com/xonotic/darkplaces/-/issues/378).
+       Cbuf_ParseText(cmd, &llist, (List_Is_Empty(&cbuf->start) ? NULL : List_Entry(cbuf->start.next, cmd_input_t, list)), text, false);
+       List_Splice(&llist, &cbuf->start);
+
+       Cbuf_Unlock(cbuf);
 }
 
 /*
@@ -233,39 +317,28 @@ void Cbuf_InsertText (cmd_state_t *cmd, const char *text)
 Cbuf_Execute_Deferred --blub
 ============
 */
-static void Cbuf_Execute_Deferred (cmd_state_t *cmd)
+static void Cbuf_Execute_Deferred (cmd_buf_t *cbuf)
 {
-       cmddeferred_t *defcmd, *prev;
-       double eat;
-       if (host.realtime - cmd->deferred_oldrealtime < 0 || host.realtime - cmd->deferred_oldrealtime > 1800) cmd->deferred_oldrealtime = host.realtime;
-       eat = host.realtime - cmd->deferred_oldrealtime;
-       if (eat < (1.0 / 120.0))
+       cmd_input_t *current, *n;
+       vec_t eat;
+
+       if (host.realtime - cbuf->deferred_oldtime < 0 || host.realtime - cbuf->deferred_oldtime > 1800)
+               cbuf->deferred_oldtime = host.realtime;
+       eat = host.realtime - cbuf->deferred_oldtime;
+       if (eat < 1.0/128.0)
                return;
-       cmd->deferred_oldrealtime = host.realtime;
-       prev = NULL;
-       defcmd = cmd->deferred_list;
-       while(defcmd)
+       cbuf->deferred_oldtime = host.realtime;
+
+       List_For_Each_Entry_Safe(current, n, &cbuf->deferred, cmd_input_t, list)
        {
-               defcmd->delay -= eat;
-               if(defcmd->delay <= 0)
+               current->delay -= eat;
+               if(current->delay <= 0)
                {
-                       Cbuf_AddText(cmd, defcmd->value);
-                       Cbuf_AddText(cmd, ";\n");
-                       Mem_Free(defcmd->value);
-
-                       if(prev) {
-                               prev->next = defcmd->next;
-                               Mem_Free(defcmd);
-                               defcmd = prev->next;
-                       } else {
-                               cmd->deferred_list = defcmd->next;
-                               Mem_Free(defcmd);
-                               defcmd = cmd->deferred_list;
-                       }
-                       continue;
+                       Cbuf_AddText(current->source, current->text); // parse deferred string and append its cmdstring(s)
+                       List_Entry(cbuf->start.prev, cmd_input_t, list)->pending = false; // faster than div0-stable's Cbuf_AddText(";\n");
+                       List_Move_Tail(&current->list, &cbuf->free); // make deferred string memory available for reuse
+                       cbuf->size -= current->length;
                }
-               prev = defcmd;
-               defcmd = defcmd->next;
        }
 }
 
@@ -274,117 +347,99 @@ static void Cbuf_Execute_Deferred (cmd_state_t *cmd)
 Cbuf_Execute
 ============
 */
-static qboolean Cmd_PreprocessString(cmd_state_t *cmd, const char *intext, char *outtext, unsigned maxoutlen, cmdalias_t *alias );
-void Cbuf_Execute (cmd_state_t *cmd)
+extern qbool prvm_runawaycheck;
+void Cbuf_Execute (cmd_buf_t *cbuf)
 {
-       int i;
-       char *text;
-       char line[MAX_INPUTLINE];
-       char preprocessed[MAX_INPUTLINE];
-       char *firstchar;
-       qboolean quotes;
-       char *comment;
+       cmd_input_t *current;
+       unsigned int i = 0;
 
        // LadyHavoc: making sure the tokenizebuffer doesn't get filled up by repeated crashes
-       cmd->tokenizebufferpos = 0;
+       cbuf->tokenizebufferpos = 0;
 
-       while (cmd->text.cursize)
+       while (!List_Is_Empty(&cbuf->start))
        {
-// find a \n or ; line break
-               text = (char *)cmd->text.data;
-
-               quotes = false;
-               comment = NULL;
-               for (i=0 ; i < cmd->text.cursize ; i++)
+               /*
+                * Delete the text from the command buffer and move remaining
+                * commands down. This is necessary because commands (exec, alias)
+                * can insert data at the beginning of the text buffer
+                */
+               current = List_Entry(cbuf->start.next, cmd_input_t, list);
+
+               /*
+                * Assume we're rolling with the current command-line and
+                * always set this false because alias expansion or cbuf insertion
+                * without a newline may set this true, and cause weirdness.
+                */
+               current->pending = false;
+
+               Cmd_PreprocessAndExecuteString(current->source, current->text, current->length, src_local, false);
+               cbuf->size -= current->length;
+               // Recycle memory so using WASD doesn't cause a malloc and free
+               List_Move_Tail(&current->list, &cbuf->free);
+
+               if (cbuf->wait)
                {
-                       if(!comment)
-                       {
-                               if (text[i] == '"')
-                                       quotes = !quotes;
-
-                               if(quotes)
-                               {
-                                       // make sure i doesn't get > cursize which causes a negative
-                                       // size in memmove, which is fatal --blub
-                                       if (i < (cmd->text.cursize-1) && (text[i] == '\\' && (text[i+1] == '"' || text[i+1] == '\\')))
-                                               i++;
-                               }
-                               else
-                               {
-                                       if(text[i] == '/' && text[i + 1] == '/' && (i == 0 || ISWHITESPACE(text[i-1])))
-                                               comment = &text[i];
-                                       if(text[i] == ';')
-                                               break;  // don't break if inside a quoted string or comment
-                               }
-                       }
-
-                       if (text[i] == '\r' || text[i] == '\n')
-                               break;
+                       /*
+                        * Skip out while text still remains in
+                        * buffer, leaving it for next frame
+                        */
+                       cbuf->wait = false;
+                       break;
                }
 
-               // better than CRASHING on overlong input lines that may SOMEHOW enter the buffer
-               if(i >= MAX_INPUTLINE)
+               if (++i == 1000000 && prvm_runawaycheck)
                {
-                       Con_Warnf("Warning: console input buffer had an overlong line. Ignored.\n");
-                       line[0] = 0;
+                       Con_Printf(CON_WARN "Cbuf_Execute: runaway loop counter hit limit of %d commands, clearing command buffers!\n", i);
+                       Cbuf_Clear(cbuf);
                }
-               else
-               {
-                       memcpy (line, text, comment ? (comment - text) : i);
-                       line[comment ? (comment - text) : i] = 0;
-               }
-
-// delete the text from the command buffer and move remaining commands down
-// this is necessary because commands (exec, alias) can insert data at the
-// beginning of the text buffer
+       }
+}
 
-               if (i == cmd->text.cursize)
-                       cmd->text.cursize = 0;
-               else
-               {
-                       i++;
-                       cmd->text.cursize -= i;
-                       memmove (cmd->text.data, text+i, cmd->text.cursize);
-               }
+/*
+===================
+Cbuf_Frame_Input
 
-// execute the command line
-               firstchar = line;
-               while(*firstchar && ISWHITESPACE(*firstchar))
-                       ++firstchar;
-               if(
-                       (strncmp(firstchar, "alias", 5) || !ISWHITESPACE(firstchar[5]))
-                       &&
-                       (strncmp(firstchar, "bind", 4) || !ISWHITESPACE(firstchar[4]))
-                       &&
-                       (strncmp(firstchar, "in_bind", 7) || !ISWHITESPACE(firstchar[7]))
-               )
-               {
-                       if(Cmd_PreprocessString( cmd, line, preprocessed, sizeof(preprocessed), NULL ))
-                               Cmd_ExecuteString (cmd, preprocessed, src_command, false);
-               }
-               else
-               {
-                       Cmd_ExecuteString (cmd, line, src_command, false);
-               }
+Add them exactly as if they had been typed at the console
+===================
+*/
+static void Cbuf_Frame_Input(void)
+{
+       char *line;
 
-               if (cmd->wait)
-               {       // skip out while text still remains in buffer, leaving it
-                       // for next frame
-                       cmd->wait = false;
-                       break;
-               }
+       if ((line = Sys_ConsoleInput()))
+       {
+               // bones_was_here: prepending allows a loop such as `alias foo "bar; wait; foo"; foo`
+               // to be broken with an alias or unalias command
+               Cbuf_InsertText(cmd_local, line);
        }
 }
 
-void Cbuf_Frame(cmd_state_t *cmd)
+void Cbuf_Frame(cmd_buf_t *cbuf)
 {
-       Cbuf_Execute_Deferred(cmd);
-       if (cmd->text.cursize)
+       // check for commands typed to the host
+       Cbuf_Frame_Input();
+
+//     R_TimeReport("preconsole");
+
+       // execute commands queued with the defer command
+       Cbuf_Execute_Deferred(cbuf);
+       if (cbuf->size)
        {
                SV_LockThreadMutex();
-               Cbuf_Execute(cmd);
+               Cbuf_Execute(cbuf);
                SV_UnlockThreadMutex();
        }
+
+//     R_TimeReport("console");
+}
+
+void Cbuf_Clear(cmd_buf_t *cbuf)
+{
+       while (!List_Is_Empty(&cbuf->start))
+               List_Move_Tail(cbuf->start.next, &cbuf->free);
+       while (!List_Is_Empty(&cbuf->deferred))
+               List_Move_Tail(cbuf->deferred.next, &cbuf->free);
+       cbuf->size = 0;
 }
 
 /*
@@ -462,21 +517,21 @@ static void Cmd_Exec(cmd_state_t *cmd, const char *filename)
 {
        char *f;
        size_t filenameLen = strlen(filename);
-       qboolean isdefaultcfg =
+       qbool isdefaultcfg =
                !strcmp(filename, "default.cfg") ||
                (filenameLen >= 12 && !strcmp(filename + filenameLen - 12, "/default.cfg"));
 
        if (!strcmp(filename, "config.cfg"))
        {
                filename = CONFIGFILENAME;
-               if (COM_CheckParm("-noconfig"))
+               if (Sys_CheckParm("-noconfig"))
                        return; // don't execute config.cfg
        }
 
        f = (char *)FS_LoadFile (filename, tempmempool, false, NULL);
        if (!f)
        {
-               Con_Printf("couldn't exec %s\n",filename);
+               Con_Printf(CON_WARN "couldn't exec %s\n",filename);
                return;
        }
        Con_Printf("execing %s\n",filename);
@@ -487,9 +542,6 @@ static void Cmd_Exec(cmd_state_t *cmd, const char *filename)
        if (isdefaultcfg)
                Cbuf_InsertText(cmd, "\ncvar_lockdefaults\n");
 
-       // insert newline after the text to make sure the last line is terminated (some text editors omit the trailing newline)
-       // (note: insertion order here is backwards from execution order, so this adds it after the text, by calling it before...)
-       Cbuf_InsertText (cmd, "\n");
        Cbuf_InsertText (cmd, f);
        Mem_Free(f);
 
@@ -662,7 +714,8 @@ static void Cmd_Exec(cmd_state_t *cmd, const char *filename)
                        Cbuf_InsertText(cmd, "\n"
 "csqc_polygons_defaultmaterial_nocullface 1\n"
 "con_chatsound_team_mask 13\n"
-"sv_gameplayfix_customstats 1\n"
+"sv_qcstats 1\n"
+"mod_q1bsp_zero_hullsize_cutoff 8.03125\n"
                                );
                        break;
                // Steel Storm: Burning Retribution csqc misinterprets CSQC_InputEvent if type is a value other than 0 or 1
@@ -733,10 +786,10 @@ static void Cmd_Exec_f (cmd_state_t *cmd)
                return;
        }
 
-       s = FS_Search(Cmd_Argv(cmd, 1), true, true);
+       s = FS_Search(Cmd_Argv(cmd, 1), true, true, NULL);
        if(!s || !s->numfilenames)
        {
-               Con_Printf("couldn't exec %s\n",Cmd_Argv(cmd, 1));
+               Con_Printf(CON_WARN "couldn't exec %s\n",Cmd_Argv(cmd, 1));
                return;
        }
 
@@ -838,7 +891,7 @@ static void Cmd_Toggle_f(cmd_state_t *cmd)
                }
                else
                { // Invalid CVar
-                       Con_Printf("ERROR : CVar '%s' not found\n", Cmd_Argv(cmd, 1) );
+                       Con_Printf(CON_WARN "ERROR : CVar '%s' not found\n", Cmd_Argv(cmd, 1) );
                }
        }
 }
@@ -852,7 +905,7 @@ Creates a new command that executes a command string (possibly ; seperated)
 */
 static void Cmd_Alias_f (cmd_state_t *cmd)
 {
-       cmdalias_t      *a;
+       cmd_alias_t     *a;
        char            line[MAX_INPUTLINE];
        int                     i, c;
        const char              *s;
@@ -869,7 +922,7 @@ static void Cmd_Alias_f (cmd_state_t *cmd)
        s = Cmd_Argv(cmd, 1);
        if (strlen(s) >= MAX_ALIAS_NAME)
        {
-               Con_Print("Alias name is too long\n");
+               Con_Print(CON_WARN "Alias name is too long\n");
                return;
        }
 
@@ -885,10 +938,10 @@ static void Cmd_Alias_f (cmd_state_t *cmd)
 
        if (!a)
        {
-               cmdalias_t *prev, *current;
+               cmd_alias_t *prev, *current;
 
-               a = (cmdalias_t *)Z_Malloc (sizeof(cmdalias_t));
-               strlcpy (a->name, s, sizeof (a->name));
+               a = (cmd_alias_t *)Z_Malloc (sizeof(cmd_alias_t));
+               dp_strlcpy (a->name, s, sizeof (a->name));
                // insert it at the right alphanumeric position
                for( prev = NULL, current = cmd->userdefined->alias ; current && strcmp( current->name, a->name ) < 0 ; prev = current, current = current->next )
                        ;
@@ -907,10 +960,10 @@ static void Cmd_Alias_f (cmd_state_t *cmd)
        for (i=2 ; i < c ; i++)
        {
                if (i != 2)
-                       strlcat (line, " ", sizeof (line));
-               strlcat (line, Cmd_Argv(cmd, i), sizeof (line));
+                       dp_strlcat (line, " ", sizeof (line));
+               dp_strlcat (line, Cmd_Argv(cmd, i), sizeof (line));
        }
-       strlcat (line, "\n", sizeof (line));
+       dp_strlcat (line, "\n", sizeof (line));
 
        alloclen = strlen (line) + 1;
        if(alloclen >= 2)
@@ -928,7 +981,7 @@ Remove existing aliases.
 */
 static void Cmd_UnAlias_f (cmd_state_t *cmd)
 {
-       cmdalias_t      *a, *p;
+       cmd_alias_t     *a, *p;
        int i;
        const char *s;
 
@@ -970,7 +1023,7 @@ static void Cmd_UnAlias_f (cmd_state_t *cmd)
 =============================================================================
 */
 
-static const char *Cmd_GetDirectCvarValue(cmd_state_t *cmd, const char *varname, cmdalias_t *alias, qboolean *is_multiple)
+static const char *Cmd_GetDirectCvarValue(cmd_state_t *cmd, const char *varname, cmd_alias_t *alias, qbool *is_multiple)
 {
        cvar_t *cvar;
        long argno;
@@ -1031,17 +1084,17 @@ static const char *Cmd_GetDirectCvarValue(cmd_state_t *cmd, const char *varname,
                }
        }
 
-       if((cvar = Cvar_FindVar(cmd->cvars, varname, cmd->cvars_flagsmask)) && !(cvar->flags & CVAR_PRIVATE))
+       if((cvar = Cvar_FindVar(cmd->cvars, varname, cmd->cvars_flagsmask)) && !(cvar->flags & CF_PRIVATE))
                return cvar->string;
 
        return NULL;
 }
 
-qboolean Cmd_QuoteString(char *out, size_t outlen, const char *in, const char *quoteset, qboolean putquotes)
+qbool Cmd_QuoteString(char *out, size_t outlen, const char *in, const char *quoteset, qbool putquotes)
 {
-       qboolean quote_quot = !!strchr(quoteset, '"');
-       qboolean quote_backslash = !!strchr(quoteset, '\\');
-       qboolean quote_dollar = !!strchr(quoteset, '$');
+       qbool quote_quot = !!strchr(quoteset, '"');
+       qbool quote_backslash = !!strchr(quoteset, '\\');
+       qbool quote_dollar = !!strchr(quoteset, '$');
 
        if(putquotes)
        {
@@ -1096,14 +1149,14 @@ fail:
        return false;
 }
 
-static const char *Cmd_GetCvarValue(cmd_state_t *cmd, const char *var, size_t varlen, cmdalias_t *alias)
+static const char *Cmd_GetCvarValue(cmd_state_t *cmd, const char *var, size_t varlen, cmd_alias_t *alias)
 {
        static char varname[MAX_INPUTLINE]; // cmd_mutex
        static char varval[MAX_INPUTLINE]; // cmd_mutex
        const char *varstr = NULL;
        char *varfunc;
-       qboolean required = false;
-       qboolean optional = false;
+       qbool required = false;
+       qbool optional = false;
        static char asis[] = "asis"; // just to suppress const char warnings
 
        if(varlen >= MAX_INPUTLINE)
@@ -1122,9 +1175,9 @@ static const char *Cmd_GetCvarValue(cmd_state_t *cmd, const char *var, size_t va
        {
                // empty cvar name?
                if(alias)
-                       Con_Warnf("Warning: Could not expand $ in alias %s\n", alias->name);
+                       Con_Printf(CON_WARN "Warning: Could not expand $ in alias %s\n", alias->name);
                else
-                       Con_Warnf("Warning: Could not expand $\n");
+                       Con_Printf(CON_WARN "Warning: Could not expand $\n");
                return "$";
        }
 
@@ -1157,7 +1210,7 @@ static const char *Cmd_GetCvarValue(cmd_state_t *cmd, const char *var, size_t va
                varstr = Cmd_GetDirectCvarValue(cmd, Cmd_GetDirectCvarValue(cmd, varname + 1, alias, NULL), alias, NULL);
        else
        {
-               qboolean is_multiple = false;
+               qbool is_multiple = false;
                // Exception: $* and $n- don't use the quoted form by default
                varstr = Cmd_GetDirectCvarValue(cmd, varname, alias, &is_multiple);
                if(is_multiple)
@@ -1170,9 +1223,9 @@ static const char *Cmd_GetCvarValue(cmd_state_t *cmd, const char *var, size_t va
                if(required)
                {
                        if(alias)
-                               Con_Errorf("Error: Could not expand $%s in alias %s\n", varname, alias->name);
+                               Con_Printf(CON_ERROR "Error: Could not expand $%s in alias %s\n", varname, alias->name);
                        else
-                               Con_Errorf("Error: Could not expand $%s\n", varname);
+                               Con_Printf(CON_ERROR "Error: Could not expand $%s\n", varname);
                        return NULL;
                }
                else if(optional)
@@ -1182,9 +1235,9 @@ static const char *Cmd_GetCvarValue(cmd_state_t *cmd, const char *var, size_t va
                else
                {
                        if(alias)
-                               Con_Warnf("Warning: Could not expand $%s in alias %s\n", varname, alias->name);
+                               Con_Printf(CON_WARN "Warning: Could not expand $%s in alias %s\n", varname, alias->name);
                        else
-                               Con_Warnf("Warning: Could not expand $%s\n", varname);
+                               Con_Printf(CON_WARN "Warning: Could not expand $%s\n", varname);
                        dpsnprintf(varval, sizeof(varval), "$%s", varname);
                        return varval;
                }
@@ -1207,12 +1260,14 @@ static const char *Cmd_GetCvarValue(cmd_state_t *cmd, const char *var, size_t va
        return varstr;
 }
 
-/*
+/**
 Cmd_PreprocessString
 
 Preprocesses strings and replaces $*, $param#, $cvar accordingly. Also strips comments.
+Returns the number of bytes written to *outtext excluding the \0 terminator.
 */
-static qboolean Cmd_PreprocessString(cmd_state_t *cmd, const char *intext, char *outtext, unsigned maxoutlen, cmdalias_t *alias ) {
+static size_t Cmd_PreprocessString(cmd_state_t *cmd, const char *intext, char *outtext, unsigned maxoutlen, cmd_alias_t *alias)
+{
        const char *in;
        size_t eat, varlen;
        unsigned outlen;
@@ -1220,7 +1275,7 @@ static qboolean Cmd_PreprocessString(cmd_state_t *cmd, const char *intext, char
 
        // don't crash if there's no room in the outtext buffer
        if( maxoutlen == 0 ) {
-               return false;
+               return 0;
        }
        maxoutlen--; // because of \0
 
@@ -1293,7 +1348,7 @@ static qboolean Cmd_PreprocessString(cmd_state_t *cmd, const char *intext, char
                                {
                                        val = Cmd_GetCvarValue(cmd, in + 1, varlen, alias);
                                        if(!val)
-                                               return false;
+                                               return 0;
                                        eat = varlen + 2;
                                }
                                else
@@ -1306,7 +1361,7 @@ static qboolean Cmd_PreprocessString(cmd_state_t *cmd, const char *intext, char
                                varlen = strspn(in, "#*0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ_-");
                                val = Cmd_GetCvarValue(cmd, in, varlen, alias);
                                if(!val)
-                                       return false;
+                                       return 0;
                                eat = varlen;
                        }
                        if(val)
@@ -1330,8 +1385,8 @@ static qboolean Cmd_PreprocessString(cmd_state_t *cmd, const char *intext, char
                else 
                        outtext[outlen++] = *in++;
        }
-       outtext[outlen] = 0;
-       return true;
+       outtext[outlen] = '\0';
+       return outlen;
 }
 
 /*
@@ -1341,11 +1396,11 @@ Cmd_ExecuteAlias
 Called for aliases and fills in the alias into the cbuffer
 ============
 */
-static void Cmd_ExecuteAlias (cmd_state_t *cmd, cmdalias_t *alias)
+static void Cmd_ExecuteAlias (cmd_state_t *cmd, cmd_alias_t *alias)
 {
        static char buffer[ MAX_INPUTLINE ]; // cmd_mutex
        static char buffer2[ MAX_INPUTLINE ]; // cmd_mutex
-       qboolean ret = Cmd_PreprocessString( cmd, alias->value, buffer, sizeof(buffer) - 2, alias );
+       qbool ret = Cmd_PreprocessString( cmd, alias->value, buffer, sizeof(buffer) - 2, alias );
        if(!ret)
                return;
        // insert at start of command buffer, so that aliases execute in order
@@ -1358,6 +1413,26 @@ static void Cmd_ExecuteAlias (cmd_state_t *cmd, cmdalias_t *alias)
        Cbuf_InsertText(cmd, buffer2);
 }
 
+void Cmd_PreprocessAndExecuteString(cmd_state_t *cmd, const char *text, size_t textlen, cmd_source_t src, qbool lockmutex)
+{
+       char preprocessed[MAX_INPUTLINE];
+       size_t preprocessed_len;
+       const char *firstchar;
+
+       firstchar = text;
+       while(*firstchar && ISWHITESPACE(*firstchar))
+               ++firstchar;
+       if((strncmp(firstchar, "alias", 5)   || !ISWHITESPACE(firstchar[5]))
+       && (strncmp(firstchar, "bind", 4)    || !ISWHITESPACE(firstchar[4]))
+       && (strncmp(firstchar, "in_bind", 7) || !ISWHITESPACE(firstchar[7])))
+       {
+               if((preprocessed_len = Cmd_PreprocessString(cmd, text, preprocessed, sizeof(preprocessed), NULL)))
+                       Cmd_ExecuteString(cmd, preprocessed, preprocessed_len, src, lockmutex);
+       }
+       else
+               Cmd_ExecuteString(cmd, text, textlen, src, lockmutex);
+}
+
 /*
 ========
 Cmd_List
@@ -1373,7 +1448,7 @@ static void Cmd_List_f (cmd_state_t *cmd)
        const char *partial;
        size_t len;
        int count;
-       qboolean ispattern;
+       qbool ispattern;
 
        if (Cmd_Argc(cmd) > 1)
        {
@@ -1389,7 +1464,7 @@ static void Cmd_List_f (cmd_state_t *cmd)
        }
 
        count = 0;
-       for (func = cmd->userdefined->csqc_functions; func; func = func->next)
+       for (func = cmd->userdefined->qc_functions; func; func = func->next)
        {
                if (partial && (ispattern ? !matchpattern_with_separator(func->name, partial, false, "", false) : strncmp(partial, func->name, len)))
                        continue;
@@ -1419,10 +1494,10 @@ static void Cmd_Apropos_f(cmd_state_t *cmd)
 {
        cmd_function_t *func;
        cvar_t *cvar;
-       cmdalias_t *alias;
+       cmd_alias_t *alias;
        const char *partial;
        int count;
-       qboolean ispattern;
+       qbool ispattern;
        char vabuf[1024];
 
        if (Cmd_Argc(cmd) > 1)
@@ -1447,17 +1522,17 @@ static void Cmd_Apropos_f(cmd_state_t *cmd)
                        Cvar_PrintHelp(cvar, cvar->name, true);
                        count++;
                }
-               for (int i = 0; i < cvar->aliasindex; i++)
+               for (char **cvar_alias = cvar->aliases; cvar_alias && *cvar_alias; cvar_alias++)
                {
-                       if (matchpattern_with_separator(cvar->aliases[i], partial, true, "", false))
+                       if (matchpattern_with_separator(*cvar_alias, partial, true, "", false))
                        {
                                Con_Printf ("cvar ");
-                               Cvar_PrintHelp(cvar, cvar->aliases[i], true);
+                               Cvar_PrintHelp(cvar, *cvar_alias, true);
                                count++;
                        }
                }
        }
-       for (func = cmd->userdefined->csqc_functions; func; func = func->next)
+       for (func = cmd->userdefined->qc_functions; func; func = func->next)
        {
                if (!matchpattern_with_separator(func->name, partial, true, "", false))
                        if (!matchpattern_with_separator(func->description, partial, true, "", false))
@@ -1485,6 +1560,23 @@ static void Cmd_Apropos_f(cmd_state_t *cmd)
        Con_Printf("%i result%s\n\n", count, (count > 1) ? "s" : "");
 }
 
+static cmd_state_t *Cmd_AddInterpreter(cmd_buf_t *cbuf, cvar_state_t *cvars, unsigned cvars_flagsmask, unsigned cmds_flagsmask, cmd_userdefined_t *userdefined)
+{
+       cmd_state_t *cmd = (cmd_state_t *)Mem_Alloc(tempmempool, sizeof(cmd_state_t));
+       
+       cmd->mempool = Mem_AllocPool("commands", 0, NULL);
+       // space for commands and script files
+       cmd->cbuf = cbuf;
+       cmd->null_string = "";
+
+       cmd->cvars = cvars;
+       cmd->cvars_flagsmask = cvars_flagsmask;
+       cmd->cmd_flagsmask = cmds_flagsmask;
+       cmd->userdefined = userdefined;
+
+       return cmd;
+}
+
 /*
 ============
 Cmd_Init
@@ -1492,76 +1584,80 @@ Cmd_Init
 */
 void Cmd_Init(void)
 {
-       cmd_iter_t *cmd_iter;
-       for (cmd_iter = cmd_iter_all; cmd_iter->cmd; cmd_iter++)
+       cmd_buf_t *cbuf;
+       unsigned cvars_flagsmask, cmds_flagsmask;
+
+       cbuf_mempool = Mem_AllocPool("Command buffer", 0, NULL);
+       cbuf = (cmd_buf_t *)Mem_Alloc(cbuf_mempool, sizeof(cmd_buf_t));
+       cbuf->maxsize = CMDBUFSIZE;
+       cbuf->lock = Thread_CreateMutex();
+       cbuf->wait = false;
+       host.cbuf = cbuf;
+
+       cbuf->start.prev = cbuf->start.next = &(cbuf->start);
+       cbuf->deferred.prev = cbuf->deferred.next = &(cbuf->deferred);
+       cbuf->free.prev = cbuf->free.next = &(cbuf->free);
+
+       // FIXME: Get rid of cmd_iter_all eventually. This is just a hack to reduce the amount of work to make the interpreters dynamic.
+       cmd_iter_all = (cmd_iter_t *)Mem_Alloc(tempmempool, sizeof(cmd_iter_t) * 3);
+
+       // local console
+       if (cls.state == ca_dedicated)
        {
-               cmd_state_t *cmd = cmd_iter->cmd;
-               cmd->mempool = Mem_AllocPool("commands", 0, NULL);
-               // space for commands and script files
-               cmd->text.data = cmd->text_buf;
-               cmd->text.maxsize = sizeof(cmd->text_buf);
-               cmd->text.cursize = 0;
-               cmd->null_string = "";
-       }
-       // client console can see server cvars because the user may start a server
-       cmd_client.cvars = &cvars_all;
-       cmd_client.cvars_flagsmask = CVAR_CLIENT | CVAR_SERVER;
-       cmd_client.cmd_flags = CMD_CLIENT | CMD_CLIENT_FROM_SERVER | CMD_SERVER_FROM_CLIENT;
-       cmd_client.userdefined = &cmd_userdefined_all;
-       // dedicated server console can only see server cvars, there is no client
-       cmd_server.cvars = &cvars_all;
-       cmd_server.cvars_flagsmask = CVAR_SERVER;
-       cmd_server.cmd_flags = CMD_SERVER;
-       cmd_server.userdefined = &cmd_userdefined_all;
+               cvars_flagsmask = CF_SERVER;
+               cmds_flagsmask = CF_SERVER | CF_SERVER_FROM_CLIENT;
+       }
+       else
+       {
+               cvars_flagsmask = CF_CLIENT | CF_SERVER;
+               cmds_flagsmask = CF_CLIENT | CF_SERVER | CF_CLIENT_FROM_SERVER | CF_SERVER_FROM_CLIENT;
+       }
+       cmd_iter_all[0].cmd = cmd_local = Cmd_AddInterpreter(cbuf, &cvars_all, cvars_flagsmask, cmds_flagsmask, &cmd_userdefined_all);
+       cmd_local->Handle = Cmd_CL_Callback;
+
        // server commands received from clients have no reason to access cvars, cvar expansion seems perilous.
-       cmd_serverfromclient.cvars = &cvars_null;
-       cmd_serverfromclient.cvars_flagsmask = 0;
-       cmd_serverfromclient.cmd_flags = CMD_SERVER_FROM_CLIENT;
-       cmd_serverfromclient.userdefined = &cmd_userdefined_null;
-}
+       cmd_iter_all[1].cmd = cmd_serverfromclient = Cmd_AddInterpreter(cbuf, &cvars_null, 0, CF_SERVER_FROM_CLIENT | CF_USERINFO, &cmd_userdefined_null);
+       cmd_serverfromclient->Handle = Cmd_SV_Callback;
 
-void Cmd_Init_Commands(qboolean dedicated_server)
-{
+       cmd_iter_all[2].cmd = NULL;
 //
 // register our commands
 //
        // client-only commands
-       Cmd_AddCommand(CMD_CLIENT | CMD_CLIENT_FROM_SERVER, "cmd", Cmd_ForwardToServer_f, "send a console commandline to the server (used by some mods)");
-       Cmd_AddCommand(CMD_SHARED, "wait", Cmd_Wait_f, "make script execution wait for next rendered frame");
-       Cmd_AddCommand(CMD_CLIENT, "cprint", Cmd_Centerprint_f, "print something at the screen center");
+       Cmd_AddCommand(CF_SHARED, "wait", Cmd_Wait_f, "make script execution wait for next rendered frame");
 
        // maintenance commands used for upkeep of cvars and saved configs
-       Cmd_AddCommand(CMD_SHARED, "stuffcmds", Cmd_StuffCmds_f, "execute commandline parameters (must be present in quake.rc script)");
-       Cmd_AddCommand(CMD_SHARED, "cvar_lockdefaults", Cvar_LockDefaults_f, "stores the current values of all cvars into their default values, only used once during startup after parsing default.cfg");
-       Cmd_AddCommand(CMD_SHARED, "cvar_resettodefaults_all", Cvar_ResetToDefaults_All_f, "sets all cvars to their locked default values");
-       Cmd_AddCommand(CMD_SHARED, "cvar_resettodefaults_nosaveonly", Cvar_ResetToDefaults_NoSaveOnly_f, "sets all non-saved cvars to their locked default values (variables that will not be saved to config.cfg)");
-       Cmd_AddCommand(CMD_SHARED, "cvar_resettodefaults_saveonly", Cvar_ResetToDefaults_SaveOnly_f, "sets all saved cvars to their locked default values (variables that will be saved to config.cfg)");
+       Cmd_AddCommand(CF_SHARED, "stuffcmds", Cmd_StuffCmds_f, "execute commandline parameters (must be present in quake.rc script)");
+       Cmd_AddCommand(CF_SHARED, "cvar_lockdefaults", Cvar_LockDefaults_f, "stores the current values of all cvars into their default values, only used once during startup after parsing default.cfg");
+       Cmd_AddCommand(CF_SHARED, "cvar_resettodefaults_all", Cvar_ResetToDefaults_All_f, "sets all cvars to their locked default values");
+       Cmd_AddCommand(CF_SHARED, "cvar_resettodefaults_nosaveonly", Cvar_ResetToDefaults_NoSaveOnly_f, "sets all non-saved cvars to their locked default values (variables that will not be saved to config.cfg)");
+       Cmd_AddCommand(CF_SHARED, "cvar_resettodefaults_saveonly", Cvar_ResetToDefaults_SaveOnly_f, "sets all saved cvars to their locked default values (variables that will be saved to config.cfg)");
 
        // general console commands used in multiple environments
-       Cmd_AddCommand(CMD_SHARED, "exec", Cmd_Exec_f, "execute a script file");
-       Cmd_AddCommand(CMD_SHARED, "echo",Cmd_Echo_f, "print a message to the console (useful in scripts)");
-       Cmd_AddCommand(CMD_SHARED, "alias",Cmd_Alias_f, "create a script function (parameters are passed in as $X (being X a number), $* for all parameters, $X- for all parameters starting from $X). Without arguments show the list of all alias");
-       Cmd_AddCommand(CMD_SHARED, "unalias",Cmd_UnAlias_f, "remove an alias");
-       Cmd_AddCommand(CMD_SHARED, "set", Cvar_Set_f, "create or change the value of a console variable");
-       Cmd_AddCommand(CMD_SHARED, "seta", Cvar_SetA_f, "create or change the value of a console variable that will be saved to config.cfg");
-       Cmd_AddCommand(CMD_SHARED, "unset", Cvar_Del_f, "delete a cvar (does not work for static ones like _cl_name, or read-only ones)");
+       Cmd_AddCommand(CF_SHARED, "exec", Cmd_Exec_f, "execute a script file");
+       Cmd_AddCommand(CF_SHARED, "echo",Cmd_Echo_f, "print a message to the console (useful in scripts)");
+       Cmd_AddCommand(CF_SHARED, "alias",Cmd_Alias_f, "create a script function (parameters are passed in as $X (being X a number), $* for all parameters, $X- for all parameters starting from $X). Without arguments show the list of all alias");
+       Cmd_AddCommand(CF_SHARED, "unalias",Cmd_UnAlias_f, "remove an alias");
+       Cmd_AddCommand(CF_SHARED, "set", Cvar_Set_f, "create or change the value of a console variable");
+       Cmd_AddCommand(CF_SHARED, "seta", Cvar_SetA_f, "create or change the value of a console variable that will be saved to config.cfg");
+       Cmd_AddCommand(CF_SHARED, "unset", Cvar_Del_f, "delete a cvar (does not work for static ones like _cl_name, or read-only ones)");
 
 #ifdef FILLALLCVARSWITHRUBBISH
-       Cmd_AddCommand(CMD_SHARED, "fillallcvarswithrubbish", Cvar_FillAll_f, "fill all cvars with a specified number of characters to provoke buffer overruns");
+       Cmd_AddCommand(CF_SHARED, "fillallcvarswithrubbish", Cvar_FillAll_f, "fill all cvars with a specified number of characters to provoke buffer overruns");
 #endif /* FILLALLCVARSWITHRUBBISH */
 
        // 2000-01-09 CmdList, CvarList commands By Matthias "Maddes" Buecher
        // Added/Modified by EvilTypeGuy eviltypeguy@qeradiant.com
-       Cmd_AddCommand(CMD_SHARED, "cmdlist", Cmd_List_f, "lists all console commands beginning with the specified prefix or matching the specified wildcard pattern");
-       Cmd_AddCommand(CMD_SHARED, "cvarlist", Cvar_List_f, "lists all console variables beginning with the specified prefix or matching the specified wildcard pattern");
-       Cmd_AddCommand(CMD_SHARED, "apropos", Cmd_Apropos_f, "lists all console variables/commands/aliases containing the specified string in the name or description");
-       Cmd_AddCommand(CMD_SHARED, "find", Cmd_Apropos_f, "lists all console variables/commands/aliases containing the specified string in the name or description");
+       Cmd_AddCommand(CF_SHARED, "cmdlist", Cmd_List_f, "lists all console commands beginning with the specified prefix or matching the specified wildcard pattern");
+       Cmd_AddCommand(CF_SHARED, "cvarlist", Cvar_List_f, "lists all console variables beginning with the specified prefix or matching the specified wildcard pattern");
+       Cmd_AddCommand(CF_SHARED, "apropos", Cmd_Apropos_f, "lists all console variables/commands/aliases containing the specified string in the name or description");
+       Cmd_AddCommand(CF_SHARED, "find", Cmd_Apropos_f, "lists all console variables/commands/aliases containing the specified string in the name or description");
 
-       Cmd_AddCommand(CMD_SHARED, "defer", Cmd_Defer_f, "execute a command in the future");
+       Cmd_AddCommand(CF_SHARED, "defer", Cmd_Defer_f, "execute a command in the future");
 
        // DRESK - 5/14/06
        // Support Doom3-style Toggle Command
-       Cmd_AddCommand(CMD_SHARED | CMD_CLIENT_FROM_SERVER, "toggle", Cmd_Toggle_f, "toggles a console variable's values (use for more info)");
+       Cmd_AddCommand(CF_SHARED | CF_CLIENT_FROM_SERVER, "toggle", Cmd_Toggle_f, "toggles a console variable's values (use for more info)");
 }
 
 /*
@@ -1576,63 +1672,32 @@ void Cmd_Shutdown(void)
        {
                cmd_state_t *cmd = cmd_iter->cmd;
 
-               if (cmd->text_lock)
+               if (cmd->cbuf->lock)
                {
                        // we usually have this locked when we get here from Host_Quit_f
-                       Cbuf_Unlock(cmd);
+                       Cbuf_Unlock(cmd->cbuf);
                }
 
                Mem_FreePool(&cmd->mempool);
        }
 }
 
-/*
-============
-Cmd_Argc
-============
-*/
-int            Cmd_Argc (cmd_state_t *cmd)
-{
-       return cmd->argc;
-}
-
-/*
-============
-Cmd_Argv
-============
-*/
-const char *Cmd_Argv(cmd_state_t *cmd, int arg)
-{
-       if (arg >= cmd->argc )
-               return cmd->null_string;
-       return cmd->argv[arg];
-}
-
-/*
-============
-Cmd_Args
-============
-*/
-const char *Cmd_Args (cmd_state_t *cmd)
-{
-       return cmd->args;
-}
-
-
 /*
 ============
 Cmd_TokenizeString
 
 Parses the given string into command line tokens.
+Takes a null terminated string.  Does not need to be /n terminated.
 ============
 */
-// AK: This function should only be called from ExcuteString because the current design is a bit of an hack
+// AK: This function should only be called from ExecuteString because the current design is a bit of an hack
 static void Cmd_TokenizeString (cmd_state_t *cmd, const char *text)
 {
        int l;
 
        cmd->argc = 0;
        cmd->args = NULL;
+       cmd->cmdline = NULL;
 
        while (1)
        {
@@ -1656,6 +1721,8 @@ static void Cmd_TokenizeString (cmd_state_t *cmd, const char *text)
                if (!*text)
                        return;
 
+               if(!cmd->argc)
+                       cmd->cmdline = text;
                if (cmd->argc == 1)
                        cmd->args = text;
 
@@ -1665,14 +1732,14 @@ static void Cmd_TokenizeString (cmd_state_t *cmd, const char *text)
                if (cmd->argc < MAX_ARGS)
                {
                        l = (int)strlen(com_token) + 1;
-                       if (cmd->tokenizebufferpos + l > CMD_TOKENIZELENGTH)
+                       if (cmd->cbuf->tokenizebufferpos + l > CMD_TOKENIZELENGTH)
                        {
-                               Con_Printf("Cmd_TokenizeString: ran out of %i character buffer space for command arguments\n", CMD_TOKENIZELENGTH);
+                               Con_Printf(CON_WARN "Cmd_TokenizeString: ran out of %i character buffer space for command arguments\n", CMD_TOKENIZELENGTH);
                                break;
                        }
-                       memcpy (cmd->tokenizebuffer + cmd->tokenizebufferpos, com_token, l);
-                       cmd->argv[cmd->argc] = cmd->tokenizebuffer + cmd->tokenizebufferpos;
-                       cmd->tokenizebufferpos += l;
+                       memcpy (cmd->cbuf->tokenizebuffer + cmd->cbuf->tokenizebufferpos, com_token, l);
+                       cmd->argv[cmd->argc] = cmd->cbuf->tokenizebuffer + cmd->cbuf->tokenizebufferpos;
+                       cmd->cbuf->tokenizebufferpos += l;
                        cmd->argc++;
                }
        }
@@ -1684,28 +1751,22 @@ static void Cmd_TokenizeString (cmd_state_t *cmd, const char *text)
 Cmd_AddCommand
 ============
 */
-void Cmd_AddCommand(int flags, const char *cmd_name, xcommand_t function, const char *description)
+void Cmd_AddCommand(unsigned flags, const char *cmd_name, xcommand_t function, const char *description)
 {
        cmd_function_t *func;
        cmd_function_t *prev, *current;
        cmd_state_t *cmd;
-       xcommand_t save = NULL;
        int i;
 
-       for (i = 0; i < 3; i++)
+       for (i = 0; i < 2; i++)
        {
                cmd = cmd_iter_all[i].cmd;
-               if (flags & cmd->cmd_flags)
+               if (flags & cmd->cmd_flagsmask)
                {
-                       if(cmd == &cmd_client && (flags & CMD_SERVER_FROM_CLIENT) && !(flags & CMD_CLIENT))
-                       {
-                               save = function;
-                               function = Cmd_ForwardToServer_f;
-                       }
                        // fail if the command is a variable name
                        if (Cvar_FindVar(cmd->cvars, cmd_name, ~0))
                        {
-                               Con_Printf("Cmd_AddCommand: %s already defined as a var\n", cmd_name);
+                               Con_Printf(CON_WARN "Cmd_AddCommand: %s already defined as a var\n", cmd_name);
                                return;
                        }
 
@@ -1716,8 +1777,8 @@ void Cmd_AddCommand(int flags, const char *cmd_name, xcommand_t function, const
                                {
                                        if (!strcmp(cmd_name, func->name))
                                        {
-                                               Con_Printf("Cmd_AddCommand: %s already defined\n", cmd_name);
-                                               goto next;
+                                               Con_Printf(CON_WARN "Cmd_AddCommand: %s already defined\n", cmd_name);
+                                               continue;
                                        }
                                }
 
@@ -1741,40 +1802,48 @@ void Cmd_AddCommand(int flags, const char *cmd_name, xcommand_t function, const
                        }
                        else
                        {
-                               // mark csqcfunc if the function already exists in the csqc_functions list
-                               for (func = cmd->userdefined->csqc_functions; func; func = func->next)
+                               // mark qcfunc if the function already exists in the qc_functions list
+                               for (func = cmd->userdefined->qc_functions; func; func = func->next)
                                {
                                        if (!strcmp(cmd_name, func->name))
                                        {
-                                               func->csqcfunc = true; //[515]: csqc
+                                               func->qcfunc = true; //[515]: csqc
                                                continue;
                                        }
                                }
 
-
                                func = (cmd_function_t *)Mem_Alloc(cmd->mempool, sizeof(cmd_function_t));
+                               func->flags = flags;
                                func->name = cmd_name;
                                func->function = function;
                                func->description = description;
-                               func->csqcfunc = true; //[515]: csqc
-                               func->next = cmd->userdefined->csqc_functions;
+                               func->qcfunc = true; //[515]: csqc
+                               func->next = cmd->userdefined->qc_functions;
+
+                               // bones_was_here: if this QC command overrides an engine command, store its pointer
+                               // to avoid doing this search at invocation if QC declines to handle this command.
+                               for (cmd_function_t *f = cmd->engine_functions; f; f = f->next)
+                               {
+                                       if (!strcmp(cmd_name, f->name))
+                                       {
+                                               Con_DPrintf("Adding QC override of engine command %s\n", cmd_name);
+                                               func->overridden = f;
+                                               break;
+                                       }
+                               }
 
                                // insert it at the right alphanumeric position
-                               for (prev = NULL, current = cmd->userdefined->csqc_functions; current && strcmp(current->name, func->name) < 0; prev = current, current = current->next)
+                               for (prev = NULL, current = cmd->userdefined->qc_functions; current && strcmp(current->name, func->name) < 0; prev = current, current = current->next)
                                        ;
                                if (prev) {
                                        prev->next = func;
                                }
                                else {
-                                       cmd->userdefined->csqc_functions = func;
+                                       cmd->userdefined->qc_functions = func;
                                }
                                func->next = current;
                        }
-                       if (save)
-                               function = save;
                }
-next:
-               continue;
        }
 }
 
@@ -1783,11 +1852,11 @@ next:
 Cmd_Exists
 ============
 */
-qboolean Cmd_Exists (cmd_state_t *cmd, const char *cmd_name)
+qbool Cmd_Exists (cmd_state_t *cmd, const char *cmd_name)
 {
        cmd_function_t  *func;
 
-       for (func = cmd->userdefined->csqc_functions; func; func = func->next)
+       for (func = cmd->userdefined->qc_functions; func; func = func->next)
                if (!strcmp(cmd_name, func->name))
                        return true;
 
@@ -1815,7 +1884,7 @@ const char *Cmd_CompleteCommand (cmd_state_t *cmd, const char *partial)
                return NULL;
 
 // check functions
-       for (func = cmd->userdefined->csqc_functions; func; func = func->next)
+       for (func = cmd->userdefined->qc_functions; func; func = func->next)
                if (!strncasecmp(partial, func->name, len))
                        return func->name;
 
@@ -1848,7 +1917,7 @@ int Cmd_CompleteCountPossible (cmd_state_t *cmd, const char *partial)
                return 0;
 
        // Loop through the command list and count all partial matches
-       for (func = cmd->userdefined->csqc_functions; func; func = func->next)
+       for (func = cmd->userdefined->qc_functions; func; func = func->next)
                if (!strncasecmp(partial, func->name, len))
                        h++;
 
@@ -1879,7 +1948,7 @@ const char **Cmd_CompleteBuildList (cmd_state_t *cmd, const char *partial)
        len = strlen(partial);
        buf = (const char **)Mem_Alloc(tempmempool, sizeofbuf + sizeof (const char *));
        // Loop through the functions lists and print all matches
-       for (func = cmd->userdefined->csqc_functions; func; func = func->next)
+       for (func = cmd->userdefined->qc_functions; func; func = func->next)
                if (!strncasecmp(partial, func->name, len))
                        buf[bpos++] = func->name;
        for (func = cmd->engine_functions; func; func = func->next)
@@ -1896,7 +1965,7 @@ void Cmd_CompleteCommandPrint (cmd_state_t *cmd, const char *partial)
        cmd_function_t *func;
        size_t len = strlen(partial);
        // Loop through the command list and print all matches
-       for (func = cmd->userdefined->csqc_functions; func; func = func->next)
+       for (func = cmd->userdefined->qc_functions; func; func = func->next)
                if (!strncasecmp(partial, func->name, len))
                        Con_Printf("^2%s^7: %s\n", func->name, func->description);
        for (func = cmd->engine_functions; func; func = func->next)
@@ -1915,7 +1984,7 @@ void Cmd_CompleteCommandPrint (cmd_state_t *cmd, const char *partial)
 */
 const char *Cmd_CompleteAlias (cmd_state_t *cmd, const char *partial)
 {
-       cmdalias_t *alias;
+       cmd_alias_t *alias;
        size_t len;
 
        len = strlen(partial);
@@ -1934,7 +2003,7 @@ const char *Cmd_CompleteAlias (cmd_state_t *cmd, const char *partial)
 // written by LadyHavoc
 void Cmd_CompleteAliasPrint (cmd_state_t *cmd, const char *partial)
 {
-       cmdalias_t *alias;
+       cmd_alias_t *alias;
        size_t len = strlen(partial);
        // Loop through the alias list and print all matches
        for (alias = cmd->userdefined->alias; alias; alias = alias->next)
@@ -1954,7 +2023,7 @@ void Cmd_CompleteAliasPrint (cmd_state_t *cmd, const char *partial)
 */
 int Cmd_CompleteAliasCountPossible (cmd_state_t *cmd, const char *partial)
 {
-       cmdalias_t      *alias;
+       cmd_alias_t     *alias;
        size_t          len;
        int                     h;
 
@@ -1984,7 +2053,7 @@ int Cmd_CompleteAliasCountPossible (cmd_state_t *cmd, const char *partial)
 */
 const char **Cmd_CompleteAliasBuildList (cmd_state_t *cmd, const char *partial)
 {
-       cmdalias_t *alias;
+       cmd_alias_t *alias;
        size_t len = 0;
        size_t bpos = 0;
        size_t sizeofbuf = (Cmd_CompleteAliasCountPossible (cmd, partial) + 1) * sizeof (const char *);
@@ -2005,7 +2074,7 @@ const char **Cmd_CompleteAliasBuildList (cmd_state_t *cmd, const char *partial)
 void Cmd_ClearCSQCCommands (cmd_state_t *cmd)
 {
        cmd_function_t *func;
-       cmd_function_t **next = &cmd->userdefined->csqc_functions;
+       cmd_function_t **next = &cmd->userdefined->qc_functions;
        
        while(*next)
        {
@@ -2017,6 +2086,70 @@ void Cmd_ClearCSQCCommands (cmd_state_t *cmd)
 
 extern cvar_t sv_cheats;
 
+/*
+ * Cloudwalk FIXME: This idea sounded great in my head but...
+ * How do we handle commands that can be received by the client,
+ * but which the server can also execute locally?
+ * 
+ * If we create a callback where the engine will forward to server
+ * but try to execute the command locally if it's dedicated,
+ * we're back to intermixing client and server code which I'm
+ * trying to avoid. There's no other way I can think of to
+ * implement that behavior that doesn't involve an #ifdef, or
+ * making a mess of hooks.
+ */
+qbool Cmd_Callback(cmd_state_t *cmd, cmd_function_t *func)
+{
+       if (func->function)
+               func->function(cmd);
+       else
+               Con_Printf(CON_WARN "Command \"%s\" can not be executed\n", Cmd_Argv(cmd, 0));
+       return true;
+}
+
+qbool Cmd_CL_Callback(cmd_state_t *cmd, cmd_function_t *func, const char *text, size_t textlen, cmd_source_t src)
+{
+       // TODO: Assign these functions to QC commands directly?
+       if(func->qcfunc)
+       {
+               if(((func->flags & CF_CLIENT) && CL_VM_ConsoleCommand(text, textlen)) ||
+                  ((func->flags & CF_SERVER) && SV_VM_ConsoleCommand(text, textlen)))
+                       return true;
+
+               if (func->overridden) // If this QC command overrides an engine command,
+                       func = func->overridden; // fall back to that command.
+       }
+       if (func->flags & CF_SERVER_FROM_CLIENT)
+       {
+               if(host_isclient.integer)
+               {
+                       CL_ForwardToServer_f(cmd);
+                       return true;
+               }
+               else if(!(func->flags & CF_SERVER))
+               {
+                       Con_Printf(CON_WARN "Cannot execute client commands from a dedicated server console.\n");
+                       return true;
+               }
+       }
+       return Cmd_Callback(cmd, func);
+}
+
+qbool Cmd_SV_Callback(cmd_state_t *cmd, cmd_function_t *func, const char *text, size_t textlen, cmd_source_t src)
+{
+       if(func->qcfunc && (func->flags & CF_SERVER))
+               return SV_VM_ConsoleCommand(text, textlen);
+       else if (src == src_client)
+       {
+               if((func->flags & CF_CHEAT) && !sv_cheats.integer)
+                       SV_ClientPrintf(CON_WARN "No cheats allowed. The server must have sv_cheats set to 1\n");
+               else
+                       func->function(cmd);
+               return true;
+       }
+       return false;
+}
+
 /*
 ============
 Cmd_ExecuteString
@@ -2025,14 +2158,15 @@ A complete command line has been parsed, so try to execute it
 FIXME: lookupnoadd the token to speed search?
 ============
 */
-void Cmd_ExecuteString (cmd_state_t *cmd, const char *text, cmd_source_t src, qboolean lockmutex)
+void Cmd_ExecuteString(cmd_state_t *cmd, const char *text, size_t textlen, cmd_source_t src, qbool lockmutex)
 {
        int oldpos;
        cmd_function_t *func;
-       cmdalias_t *a;
+       cmd_alias_t *a;
+
        if (lockmutex)
-               Cbuf_Lock(cmd);
-       oldpos = cmd->tokenizebufferpos;
+               Cbuf_Lock(cmd->cbuf);
+       oldpos = cmd->cbuf->tokenizebufferpos;
        cmd->source = src;
 
        Cmd_TokenizeString (cmd, text);
@@ -2042,50 +2176,28 @@ void Cmd_ExecuteString (cmd_state_t *cmd, const char *text, cmd_source_t src, qb
                goto done; // no tokens
 
 // check functions
-       for (func = cmd->userdefined->csqc_functions; func; func = func->next)
-       {
+       for (func = cmd->userdefined->qc_functions; func; func = func->next)
                if (!strcasecmp(cmd->argv[0], func->name))
-               {
-                       if (func->csqcfunc && CL_VM_ConsoleCommand(text))       //[515]: csqc
-                               goto done;
-                       break;
-               }
-       }
+                       if(cmd->Handle(cmd, func, text, textlen, src))
+                               goto functions_done;
 
        for (func = cmd->engine_functions; func; func=func->next)
-       {
                if (!strcasecmp (cmd->argv[0], func->name))
-               {
-                       switch (src)
-                       {
-                       case src_command:
-                               if (func->function)
-                                       func->function(cmd);
-                               else
-                                       Con_Printf("Command \"%s\" can not be executed\n", Cmd_Argv(cmd, 0));
-                               goto done;
-                       case src_client:
-                               if (func->function)
-                               {
-                                       if((func->flags & CMD_CHEAT) && !sv_cheats.integer)
-                                               SV_ClientPrintf("No cheats allowed. The server must have sv_cheats set to 1\n");
-                                       else
-                                               func->function(cmd);
-                                       goto done;
-                               }
-                       }
-                       break;
-               }
-       }
+                       if(cmd->Handle(cmd, func, text, textlen, src))
+                               goto functions_done;
 
-       // if it's a client command and no command was found, say so.
+functions_done:
+       // If it's a client command and wasn't found and handled, say so.
+       // Also don't let clients call server aliases.
        if (cmd->source == src_client)
        {
-               Con_Printf("Client \"%s\" tried to execute \"%s\"\n", host_client->name, text);
+               if (!func)
+                       Con_Printf("Client \"%s\" tried to execute \"%s\"\n", host_client->name, text);
                goto done;
        }
 
 // check alias
+       // Execute any alias with the same name as a command after the command.
        for (a=cmd->userdefined->alias ; a ; a=a->next)
        {
                if (!strcasecmp (cmd->argv[0], a->name))
@@ -2095,171 +2207,20 @@ void Cmd_ExecuteString (cmd_state_t *cmd, const char *text, cmd_source_t src, qb
                }
        }
 
+       // If the command was found and handled don't try to handle it as a cvar.
+       if (func)
+               goto done;
+
 // check cvars
-       if (!Cvar_Command(cmd) && host.framecount > 0)
-               Con_Printf("Unknown command \"%s\"\n", Cmd_Argv(cmd, 0));
+       // Xonotic is still maintained so we don't want to hide problems from getting fixed
+       if (!Cvar_Command(cmd) && (host.framecount > 0 || gamemode == GAME_XONOTIC))
+               Con_Printf(CON_WARN "Unknown command \"%s\"\n", Cmd_Argv(cmd, 0));
 done:
-       cmd->tokenizebufferpos = oldpos;
+       cmd->cbuf->tokenizebufferpos = oldpos;
        if (lockmutex)
-               Cbuf_Unlock(cmd);
+               Cbuf_Unlock(cmd->cbuf);
 }
 
-
-/*
-===================
-Cmd_ForwardStringToServer
-
-Sends an entire command string over to the server, unprocessed
-===================
-*/
-void Cmd_ForwardStringToServer (const char *s)
-{
-       char temp[128];
-       if (cls.state != ca_connected)
-       {
-               Con_Printf("Can't \"%s\", not connected\n", s);
-               return;
-       }
-
-       if (!cls.netcon)
-               return;
-
-       // LadyHavoc: thanks to Fuh for bringing the pure evil of SZ_Print to my
-       // attention, it has been eradicated from here, its only (former) use in
-       // all of darkplaces.
-       if (cls.protocol == PROTOCOL_QUAKEWORLD)
-               MSG_WriteByte(&cls.netcon->message, qw_clc_stringcmd);
-       else
-               MSG_WriteByte(&cls.netcon->message, clc_stringcmd);
-       if ((!strncmp(s, "say ", 4) || !strncmp(s, "say_team ", 9)) && cl_locs_enable.integer)
-       {
-               // say/say_team commands can replace % character codes with status info
-               while (*s)
-               {
-                       if (*s == '%' && s[1])
-                       {
-                               // handle proquake message macros
-                               temp[0] = 0;
-                               switch (s[1])
-                               {
-                               case 'l': // current location
-                                       CL_Locs_FindLocationName(temp, sizeof(temp), cl.movement_origin);
-                                       break;
-                               case 'h': // current health
-                                       dpsnprintf(temp, sizeof(temp), "%i", cl.stats[STAT_HEALTH]);
-                                       break;
-                               case 'a': // current armor
-                                       dpsnprintf(temp, sizeof(temp), "%i", cl.stats[STAT_ARMOR]);
-                                       break;
-                               case 'x': // current rockets
-                                       dpsnprintf(temp, sizeof(temp), "%i", cl.stats[STAT_ROCKETS]);
-                                       break;
-                               case 'c': // current cells
-                                       dpsnprintf(temp, sizeof(temp), "%i", cl.stats[STAT_CELLS]);
-                                       break;
-                               // silly proquake macros
-                               case 'd': // loc at last death
-                                       CL_Locs_FindLocationName(temp, sizeof(temp), cl.lastdeathorigin);
-                                       break;
-                               case 't': // current time
-                                       dpsnprintf(temp, sizeof(temp), "%.0f:%.0f", floor(cl.time / 60), cl.time - floor(cl.time / 60) * 60);
-                                       break;
-                               case 'r': // rocket launcher status ("I have RL", "I need rockets", "I need RL")
-                                       if (!(cl.stats[STAT_ITEMS] & IT_ROCKET_LAUNCHER))
-                                               dpsnprintf(temp, sizeof(temp), "I need RL");
-                                       else if (!cl.stats[STAT_ROCKETS])
-                                               dpsnprintf(temp, sizeof(temp), "I need rockets");
-                                       else
-                                               dpsnprintf(temp, sizeof(temp), "I have RL");
-                                       break;
-                               case 'p': // powerup status (outputs "quad" "pent" and "eyes" according to status)
-                                       if (cl.stats[STAT_ITEMS] & IT_QUAD)
-                                       {
-                                               if (temp[0])
-                                                       strlcat(temp, " ", sizeof(temp));
-                                               strlcat(temp, "quad", sizeof(temp));
-                                       }
-                                       if (cl.stats[STAT_ITEMS] & IT_INVULNERABILITY)
-                                       {
-                                               if (temp[0])
-                                                       strlcat(temp, " ", sizeof(temp));
-                                               strlcat(temp, "pent", sizeof(temp));
-                                       }
-                                       if (cl.stats[STAT_ITEMS] & IT_INVISIBILITY)
-                                       {
-                                               if (temp[0])
-                                                       strlcat(temp, " ", sizeof(temp));
-                                               strlcat(temp, "eyes", sizeof(temp));
-                                       }
-                                       break;
-                               case 'w': // weapon status (outputs "SSG:NG:SNG:GL:RL:LG" with the text between : characters omitted if you lack the weapon)
-                                       if (cl.stats[STAT_ITEMS] & IT_SUPER_SHOTGUN)
-                                               strlcat(temp, "SSG", sizeof(temp));
-                                       strlcat(temp, ":", sizeof(temp));
-                                       if (cl.stats[STAT_ITEMS] & IT_NAILGUN)
-                                               strlcat(temp, "NG", sizeof(temp));
-                                       strlcat(temp, ":", sizeof(temp));
-                                       if (cl.stats[STAT_ITEMS] & IT_SUPER_NAILGUN)
-                                               strlcat(temp, "SNG", sizeof(temp));
-                                       strlcat(temp, ":", sizeof(temp));
-                                       if (cl.stats[STAT_ITEMS] & IT_GRENADE_LAUNCHER)
-                                               strlcat(temp, "GL", sizeof(temp));
-                                       strlcat(temp, ":", sizeof(temp));
-                                       if (cl.stats[STAT_ITEMS] & IT_ROCKET_LAUNCHER)
-                                               strlcat(temp, "RL", sizeof(temp));
-                                       strlcat(temp, ":", sizeof(temp));
-                                       if (cl.stats[STAT_ITEMS] & IT_LIGHTNING)
-                                               strlcat(temp, "LG", sizeof(temp));
-                                       break;
-                               default:
-                                       // not a recognized macro, print it as-is...
-                                       temp[0] = s[0];
-                                       temp[1] = s[1];
-                                       temp[2] = 0;
-                                       break;
-                               }
-                               // write the resulting text
-                               SZ_Write(&cls.netcon->message, (unsigned char *)temp, (int)strlen(temp));
-                               s += 2;
-                               continue;
-                       }
-                       MSG_WriteByte(&cls.netcon->message, *s);
-                       s++;
-               }
-               MSG_WriteByte(&cls.netcon->message, 0);
-       }
-       else // any other command is passed on as-is
-               SZ_Write(&cls.netcon->message, (const unsigned char *)s, (int)strlen(s) + 1);
-}
-
-/*
-===================
-Cmd_ForwardToServer
-
-Sends the entire command line over to the server
-===================
-*/
-void Cmd_ForwardToServer_f (cmd_state_t *cmd)
-{
-       const char *s;
-       char vabuf[1024];
-       if (!strcasecmp(Cmd_Argv(cmd, 0), "cmd"))
-       {
-               // we want to strip off "cmd", so just send the args
-               s = Cmd_Argc(cmd) > 1 ? Cmd_Args(cmd) : "";
-       }
-       else
-       {
-               // we need to keep the command name, so send Cmd_Argv(cmd, 0), a space and then Cmd_Args(cmd)
-               s = va(vabuf, sizeof(vabuf), "%s %s", Cmd_Argv(cmd, 0), Cmd_Argc(cmd) > 1 ? Cmd_Args(cmd) : "");
-       }
-       // don't send an empty forward message if the user tries "cmd" by itself
-       if (!s || !*s)
-               return;
-       Cmd_ForwardStringToServer(s);
-}
-
-
 /*
 ================
 Cmd_CheckParm
@@ -2275,7 +2236,7 @@ int Cmd_CheckParm (cmd_state_t *cmd, const char *parm)
 
        if (!parm)
        {
-               Con_Printf ("Cmd_CheckParm: NULL");
+               Con_Printf(CON_WARN "Cmd_CheckParm: NULL");
                return 0;
        }
 
@@ -2295,8 +2256,8 @@ void Cmd_SaveInitState(void)
        {
                cmd_state_t *cmd = cmd_iter->cmd;
                cmd_function_t *f;
-               cmdalias_t *a;
-               for (f = cmd->userdefined->csqc_functions; f; f = f->next)
+               cmd_alias_t *a;
+               for (f = cmd->userdefined->qc_functions; f; f = f->next)
                        f->initstate = true;
                for (f = cmd->engine_functions; f; f = f->next)
                        f->initstate = true;
@@ -2316,8 +2277,8 @@ void Cmd_RestoreInitState(void)
        {
                cmd_state_t *cmd = cmd_iter->cmd;
                cmd_function_t *f, **fp;
-               cmdalias_t *a, **ap;
-               for (fp = &cmd->userdefined->csqc_functions; (f = *fp);)
+               cmd_alias_t *a, **ap;
+               for (fp = &cmd->userdefined->qc_functions; (f = *fp);)
                {
                        if (f->initstate)
                                fp = &f->next;
@@ -2368,3 +2329,7 @@ void Cmd_RestoreInitState(void)
        }
        Cvar_RestoreInitState(&cvars_all);
 }
+
+void Cmd_NoOperation_f(cmd_state_t *cmd)
+{
+}