X-Git-Url: http://de.git.xonotic.org/?p=xonotic%2Fdarkplaces.git;a=blobdiff_plain;f=keys.c;h=02a1a0d9d31f04d26db550e056dc031daff4900f;hp=91f4f55c52013dc52f974ad7b621c8173c510f20;hb=e93385631fb31028b8879b4e77b21d5ad95969c8;hpb=13e8c23d8ae30c2cece331a47e80176ea8a4f6cf

diff --git a/keys.c b/keys.c
index 91f4f55c..02a1a0d9 100644
--- a/keys.c
+++ b/keys.c
@@ -21,40 +21,32 @@
 */
 
 #include "quakedef.h"
+#include "cl_video.h"
 
+cvar_t con_closeontoggleconsole = {CVAR_SAVE, "con_closeontoggleconsole","1", "allows toggleconsole binds to close the console as well"};
 
 /*
 key up events are sent even if in console mode
 */
 
-#define MAX_INPUTLINE 256
-char		key_lines[32][MAX_INPUTLINE];
+int			edit_line = MAX_INPUTLINES-1;
+int			history_line = MAX_INPUTLINES-1;
+char		key_lines[MAX_INPUTLINES][MAX_INPUTLINE];
 int			key_linepos;
-static int	ctrl_down = false;
-static int	key_lastpress;
-int			key_insert;	// insert key toggle (for editing)
-
-int         edit_line = 0;
-int         history_line = 0;
-
+qboolean	key_insert = true;	// insert key toggle (for editing)
 keydest_t	key_dest;
 int			key_consoleactive;
+char		*keybindings[MAX_BINDMAPS][MAX_KEYS];
 
-static int	key_count;					// incremented every key event
 static int	key_bmap, key_bmap2;
+static unsigned char keydown[MAX_KEYS];	// 0 = up, 1 = down, 2 = repeating
 
-char				*keybindings[8][1024];
-static qboolean		consolekeys[1024];	// if true, can't be rebound while in
-										// console
-static qboolean		menubound[1024];		// if true, can't be rebound while in
-										// menu
-static unsigned int	key_repeats[1024];	// if > 1, it is autorepeating
-static qboolean		keydown[1024];
-
-typedef struct {
+typedef struct keyname_s
+{
 	const char	*name;
 	int			keynum;
-} keyname_t;
+}
+keyname_t;
 
 static const keyname_t   keynames[] = {
 	{"TAB", K_TAB},
@@ -197,11 +189,12 @@ static const keyname_t   keynames[] = {
 	{"AUX31", K_AUX31},
 	{"AUX32", K_AUX32},
 
-	{"SEMICOLON", ';'},			// because a raw semicolon seperates commands
+	{"SEMICOLON", ';'},			// because a raw semicolon separates commands
 	{"TILDE", '~'},
 	{"BACKQUOTE", '`'},
 	{"QUOTE", '"'},
 	{"APOSTROPHE", '\''},
+	{"BACKSLASH", '\\'},		// because a raw backslash is used for special characters
 
 	{NULL, 0}
 };
@@ -217,7 +210,7 @@ static const keyname_t   keynames[] = {
 void
 Key_ClearEditLine (int edit_line)
 {
-	memset (key_lines[edit_line], '\0', MAX_INPUTLINE);
+	memset (key_lines[edit_line], '\0', sizeof(key_lines[edit_line]));
 	key_lines[edit_line][0] = ']';
 	key_linepos = 1;
 }
@@ -284,16 +277,16 @@ Key_Console (int key, char ascii)
 		{
 			int i;
 			strtok(cbd, "\n\r\b");
-			i = strlen(cbd);
+			i = (int)strlen(cbd);
 			if (i + key_linepos >= MAX_INPUTLINE)
 				i= MAX_INPUTLINE - key_linepos;
 			if (i > 0)
 			{
 				cbd[i]=0;
-				strcat(key_lines[edit_line], cbd);
+				strlcat(key_lines[edit_line], cbd, sizeof(key_lines[edit_line]));
 				key_linepos += i;
 			}
-			free(cbd);
+			Z_Free(cbd);
 		}
 		return;
 	}
@@ -312,8 +305,12 @@ Key_Console (int key, char ascii)
 		Cbuf_AddText (key_lines[edit_line]+1);	// skip the ]
 		Cbuf_AddText ("\n");
 		Con_Printf("%s\n",key_lines[edit_line]);
-		edit_line = (edit_line + 1) & 31;
+		if(key_lines[edit_line][1] == 0) // empty line (just a ])?
+			return; // no, no, you can't submit empty lines to the history...
+		// LordHavoc: redesigned edit_line/history_line
+		edit_line = 31;
 		history_line = edit_line;
+		memmove(key_lines[0], key_lines[1], sizeof(key_lines[0]) * edit_line);
 		key_lines[edit_line][0] = ']';
 		key_lines[edit_line][1] = 0;	// EvilTypeGuy: null terminate
 		key_linepos = 1;
@@ -334,12 +331,29 @@ Key_Console (int key, char ascii)
 
 	// Advanced Console Editing by Radix radix@planetquake.com
 	// Added/Modified by EvilTypeGuy eviltypeguy@qeradiant.com
+	// Enhanced by [515]
 
 	// left arrow will just move left one without erasing, backspace will
-	// actually erase charcter
+	// actually erase character
 	if (key == K_LEFTARROW || key == K_KP_LEFTARROW)
 	{
-		if (key_linepos > 1)
+		if (key_linepos < 2)
+			return;
+		if(keydown[K_CTRL])
+		{
+			int		pos;
+			char	k;
+			pos = key_linepos-1;
+			if(pos)
+				while(--pos)
+				{
+					k = key_lines[edit_line][pos];
+					if(k == '\"' || k == ';' || k == ' ' || k == '\'')
+						break;
+				}
+			key_linepos = pos + 1;
+		}
+		else
 			key_linepos--;
 		return;
 	}
@@ -349,7 +363,7 @@ Key_Console (int key, char ascii)
 	{
 		if (key_linepos > 1)
 		{
-			strcpy(key_lines[edit_line] + key_linepos - 1, key_lines[edit_line] + key_linepos);
+			strlcpy(key_lines[edit_line] + key_linepos - 1, key_lines[edit_line] + key_linepos, sizeof(key_lines[edit_line]) + 1 - key_linepos);
 			key_linepos--;
 		}
 		return;
@@ -358,8 +372,10 @@ Key_Console (int key, char ascii)
 	// delete char on cursor
 	if (key == K_DEL || key == K_KP_DEL)
 	{
-		if ((size_t)key_linepos < strlen(key_lines[edit_line]))
-			strcpy(key_lines[edit_line] + key_linepos, key_lines[edit_line] + key_linepos + 1);
+		size_t linelen;
+		linelen = strlen(key_lines[edit_line]);
+		if (key_linepos < (int)linelen)
+			memmove(key_lines[edit_line] + key_linepos, key_lines[edit_line] + key_linepos + 1, linelen - key_linepos);
 		return;
 	}
 
@@ -368,18 +384,24 @@ Key_Console (int key, char ascii)
 	// otherwise just go right one
 	if (key == K_RIGHTARROW || key == K_KP_RIGHTARROW)
 	{
-		if (strlen(key_lines[edit_line]) == (size_t)key_linepos)
+		if (key_linepos >= (int)strlen(key_lines[edit_line]))
+			return;
+		if(keydown[K_CTRL])
 		{
-			if (strlen(key_lines[(edit_line + 31) & 31]) <= (size_t)key_linepos)
-				return; // no character to get
-
-			key_lines[edit_line][key_linepos] = key_lines[(edit_line + 31) & 31][key_linepos];
-			key_linepos++;
-			key_lines[edit_line][key_linepos] = 0;
+			int		pos, len;
+			char	k;
+			len = (int)strlen(key_lines[edit_line]);
+			pos = key_linepos;
+			while(++pos < len)
+			{
+				k = key_lines[edit_line][pos];
+				if(k == '\"' || k == ';' || k == ' ' || k == '\'')
+					break;
+			}
+			key_linepos = pos;
 		}
 		else
 			key_linepos++;
-
 		return;
 	}
 
@@ -393,51 +415,48 @@ Key_Console (int key, char ascii)
 
 	if (key == K_UPARROW || key == K_KP_UPARROW || (key == 'p' && keydown[K_CTRL]))
 	{
-		do
+		if (history_line > 0 && key_lines[history_line-1][1])
 		{
-			history_line = (history_line - 1) & 31;
-		} while (history_line != edit_line
-				&& !key_lines[history_line][1]);
-		if (history_line == edit_line)
-			history_line = (edit_line+1)&31;
-		strcpy(key_lines[edit_line], key_lines[history_line]);
-		key_linepos = strlen(key_lines[edit_line]);
+			size_t linelen;
+			history_line--;
+			linelen = strlen(key_lines[history_line]);
+			memcpy(key_lines[edit_line], key_lines[history_line], linelen + 1);
+			key_linepos = (int)linelen;
+		}
 		return;
 	}
 
 	if (key == K_DOWNARROW || key == K_KP_DOWNARROW || (key == 'n' && keydown[K_CTRL]))
 	{
-		if (history_line == edit_line) return;
-		do
-		{
-			history_line = (history_line + 1) & 31;
-		}
-		while (history_line != edit_line
-			&& !key_lines[history_line][1]);
-		if (history_line == edit_line)
+		history_line++;
+		if (history_line >= edit_line)
 		{
+			history_line = edit_line;
 			key_lines[edit_line][0] = ']';
+			key_lines[edit_line][1] = 0;
 			key_linepos = 1;
 		}
 		else
 		{
-			strcpy(key_lines[edit_line], key_lines[history_line]);
-			key_linepos = strlen(key_lines[edit_line]);
+			size_t linelen;
+			linelen = strlen(key_lines[history_line]);
+			memcpy(key_lines[edit_line], key_lines[history_line], linelen + 1);
+			key_linepos = (int)linelen;
 		}
 		return;
 	}
 
 	if (key == K_PGUP || key == K_KP_PGUP || key == K_MWHEELUP)
 	{
-		con_backscroll += ((int) vid.conheight >> 5);
-		if (con_backscroll > con_totallines - (vid.conheight>>3) - 1)
-			con_backscroll = con_totallines - (vid.conheight>>3) - 1;
+		con_backscroll += ((int) vid_conheight.integer >> 5);
+		if (con_backscroll > con_totallines - (vid_conheight.integer>>3) - 1)
+			con_backscroll = con_totallines - (vid_conheight.integer>>3) - 1;
 		return;
 	}
 
 	if (key == K_PGDN || key == K_KP_PGDN || key == K_MWHEELDOWN)
 	{
-		con_backscroll -= ((int) vid.conheight >> 5);
+		con_backscroll -= ((int) vid_conheight.integer >> 5);
 		if (con_backscroll < 0)
 			con_backscroll = 0;
 		return;
@@ -445,43 +464,39 @@ Key_Console (int key, char ascii)
 
 	if (key == K_HOME || key == K_KP_HOME)
 	{
-		con_backscroll = con_totallines - (vid.conheight>>3) - 1;
+		if (keydown[K_CTRL])
+			con_backscroll = con_totallines - (vid_conheight.integer>>3) - 1;
+		else
+			key_linepos = 1;
 		return;
 	}
 
 	if (key == K_END || key == K_KP_END)
 	{
-		con_backscroll = 0;
+		if (keydown[K_CTRL])
+			con_backscroll = 0;
+		else
+			key_linepos = (int)strlen(key_lines[edit_line]);
 		return;
 	}
 
 	// non printable
-	if (ascii < 32 || ascii > 126)
+	if (ascii < 32)
 		return;
 
 	if (key_linepos < MAX_INPUTLINE-1)
 	{
-		int i;
-
-		if (key_insert)	// check insert mode
+		int len;
+		len = (int)strlen(&key_lines[edit_line][key_linepos]);
+		// check insert mode, or always insert if at end of line
+		if (key_insert || len == 0)
 		{
-			// can't do strcpy to move string to right
-			i = strlen(key_lines[edit_line]) - 1;
-
-			if (i == 254)
-				i--;
-
-			for (; i >= key_linepos; i--)
-				key_lines[edit_line][i + 1] = key_lines[edit_line][i];
+			// can't use strcpy to move string to right
+			len++;
+			memmove(&key_lines[edit_line][key_linepos + 1], &key_lines[edit_line][key_linepos], len);
 		}
-
-		// only null terminate if at the end
-		i = key_lines[edit_line][key_linepos];
 		key_lines[edit_line][key_linepos] = ascii;
 		key_linepos++;
-
-		if (!i)
-			key_lines[edit_line][key_linepos] = 0;
 	}
 }
 
@@ -495,7 +510,7 @@ static void
 Key_Message (int key, char ascii)
 {
 
-	if (key == K_ENTER)
+	if (key == K_ENTER || ascii == 10 || ascii == 13)
 	{
 		Cmd_ForwardStringToServer(va("%s %s", chat_team ? "say_team" : "say ", chat_buffer));
 
@@ -570,18 +585,24 @@ Key_KeynumToString (int keynum)
 	const keyname_t  *kn;
 	static char tinystr[2];
 
-	if (keynum == -1)
+	// -1 is an invalid code
+	if (keynum < 0)
 		return "<KEY NOT FOUND>";
-	if (keynum > 32 && keynum < 127) {	// printable ascii
-		tinystr[0] = keynum;
-		tinystr[1] = 0;
-		return tinystr;
-	}
 
+	// search overrides first, because some characters are special
 	for (kn = keynames; kn->name; kn++)
 		if (keynum == kn->keynum)
 			return kn->name;
 
+	// if it is printable, output it as a single character
+	if (keynum > 32 && keynum < 256)
+	{
+		tinystr[0] = keynum;
+		tinystr[1] = 0;
+		return tinystr;
+	}
+
+	// if it is not overridden and not printable, we don't know what to do with it
 	return "<UNKNOWN KEYNUM>";
 }
 
@@ -589,10 +610,10 @@ Key_KeynumToString (int keynum)
 void
 Key_SetBinding (int keynum, int bindmap, const char *binding)
 {
-	char       *new;
-	int         l;
+	char *newbinding;
+	size_t l;
 
-	if (keynum == -1)
+	if (keynum == -1 || keynum >= MAX_KEYS)
 		return;
 
 // free old bindings
@@ -602,10 +623,10 @@ Key_SetBinding (int keynum, int bindmap, const char *binding)
 	}
 // allocate memory for new binding
 	l = strlen (binding);
-	new = Z_Malloc (l + 1);
-	strcpy (new, binding);
-	new[l] = 0;
-	keybindings[bindmap][keynum] = new;
+	newbinding = (char *)Z_Malloc (l + 1);
+	memcpy (newbinding, binding, l + 1);
+	newbinding[l] = 0;
+	keybindings[bindmap][keynum] = newbinding;
 }
 
 static void
@@ -637,7 +658,7 @@ static void
 Key_In_Bind_f (void)
 {
 	int         i, c, b, m;
-	char        cmd[1024];
+	char        cmd[MAX_INPUTLINE];
 
 	c = Cmd_Argc ();
 
@@ -653,7 +674,7 @@ Key_In_Bind_f (void)
 	}
 
 	b = Key_StringToKeynum (Cmd_Argv (2));
-	if (b == -1) {
+	if (b == -1 || b >= MAX_KEYS) {
 		Con_Printf("\"%s\" isn't a valid key\n", Cmd_Argv (2));
 		return;
 	}
@@ -739,7 +760,7 @@ static void
 Key_Bind_f (void)
 {
 	int         i, c, b;
-	char        cmd[1024];
+	char        cmd[MAX_INPUTLINE];
 
 	c = Cmd_Argc ();
 
@@ -748,7 +769,7 @@ Key_Bind_f (void)
 		return;
 	}
 	b = Key_StringToKeynum (Cmd_Argv (1));
-	if (b == -1) {
+	if (b == -1 || b >= MAX_KEYS) {
 		Con_Printf("\"%s\" isn't a valid key\n", Cmd_Argv (1));
 		return;
 	}
@@ -780,16 +801,24 @@ void
 Key_WriteBindings (qfile_t *f)
 {
 	int         i, j;
+	char bindbuf[MAX_INPUTLINE];
+	const char *p;
 
-	for (i = 0; i < (int)(sizeof(keybindings[0])/sizeof(keybindings[0][0])); i++)
-		if (keybindings[0][i])
-			FS_Printf(f, "bind \"%s\" \"%s\"\n",
-					Key_KeynumToString (i), keybindings[0][i]);
-	for (j = 1; j < 8; j++)
+	for (j = 0; j < MAX_BINDMAPS; j++)
+	{
 		for (i = 0; i < (int)(sizeof(keybindings[0])/sizeof(keybindings[0][0])); i++)
-			if (keybindings[j][i])
-				FS_Printf(f, "in_bind %d \"%s\" \"%s\"\n",
-						j, Key_KeynumToString (i), keybindings[j][i]);
+		{
+			p = keybindings[j][i];
+			if (p)
+			{
+				Cmd_QuoteString(bindbuf, sizeof(bindbuf), p, "\"\\");
+				if (j == 0)
+					FS_Printf(f, "bind %s \"%s\"\n", Key_KeynumToString (i), bindbuf);
+				else
+					FS_Printf(f, "in_bind %d %s \"%s\"\n", j, Key_KeynumToString (i), bindbuf);
+			}
+		}
+	}
 }
 
 
@@ -804,50 +833,32 @@ Key_Init (void)
 	}
 	key_linepos = 1;
 
-//
-// init ascii characters in console mode
-//
-	for (i = 32; i < 128; i++)
-		consolekeys[i] = true;
-	consolekeys[K_ENTER] = true; consolekeys[K_KP_ENTER] = true;
-	consolekeys[K_TAB] = true;
-	consolekeys[K_LEFTARROW] = true; consolekeys[K_KP_LEFTARROW] = true;
-	consolekeys[K_RIGHTARROW] = true; consolekeys[K_KP_RIGHTARROW] = true;
-	consolekeys[K_UPARROW] = true; consolekeys[K_KP_UPARROW] = true;
-	consolekeys[K_DOWNARROW] = true; consolekeys[K_KP_DOWNARROW] = true;
-	consolekeys[K_BACKSPACE] = true;
-	consolekeys[K_DEL] = true; consolekeys[K_KP_DEL] = true;
-	consolekeys[K_INS] = true; consolekeys[K_KP_INS] = true;
-	consolekeys[K_HOME] = true; consolekeys[K_KP_HOME] = true;
-	consolekeys[K_END] = true; consolekeys[K_KP_END] = true;
-	consolekeys[K_PGUP] = true; consolekeys[K_KP_PGUP] = true;
-	consolekeys[K_PGDN] = true; consolekeys[K_KP_PGDN] = true;
-	consolekeys[K_SHIFT] = true;
-	consolekeys[K_MWHEELUP] = true;
-	consolekeys[K_MWHEELDOWN] = true;
-	consolekeys[K_KP_PLUS] = true;
-	consolekeys[K_KP_MINUS] = true;
-	consolekeys[K_KP_DIVIDE] = true;
-	consolekeys[K_KP_MULTIPLY] = true;
-	consolekeys['`'] = false;
-	consolekeys['~'] = false;
-
-	menubound[K_ESCAPE] = true;
-	for (i = 0; i < 12; i++)
-		menubound[K_F1 + i] = true;
-
 //
 // register our functions
 //
-	Cmd_AddCommand ("in_bind", Key_In_Bind_f);
-	Cmd_AddCommand ("in_unbind", Key_In_Unbind_f);
-	Cmd_AddCommand ("in_bindmap", Key_In_Bindmap_f);
+	Cmd_AddCommand ("in_bind", Key_In_Bind_f, "binds a command to the specified key in the selected bindmap");
+	Cmd_AddCommand ("in_unbind", Key_In_Unbind_f, "removes command on the specified key in the selected bindmap");
+	Cmd_AddCommand ("in_bindmap", Key_In_Bindmap_f, "selects active foreground and background (used only if a key is not bound in the foreground) bindmaps for typing");
+
+	Cmd_AddCommand ("bind", Key_Bind_f, "binds a command to the specified key in bindmap 0");
+	Cmd_AddCommand ("unbind", Key_Unbind_f, "removes a command on the specified key in bindmap 0");
+	Cmd_AddCommand ("unbindall", Key_Unbindall_f, "removes all commands from all keys in all bindmaps (leaving only shift-escape and escape)");
+
+	Cvar_RegisterVariable (&con_closeontoggleconsole);
+}
 
-	Cmd_AddCommand ("bind", Key_Bind_f);
-	Cmd_AddCommand ("unbind", Key_Unbind_f);
-	Cmd_AddCommand ("unbindall", Key_Unbindall_f);
+const char *Key_GetBind (int key)
+{
+	const char *bind;
+	if (key < 0 || key >= MAX_KEYS)
+		return NULL;
+	bind = keybindings[key_bmap][key];
+	if (!bind)
+		bind = keybindings[key_bmap2][key];
+	return bind;
 }
 
+qboolean CL_VM_InputEvent (qboolean pressed, int key);
 
 /*
 ===================
@@ -858,142 +869,164 @@ Should NOT be called during an interrupt!
 void
 Key_Event (int key, char ascii, qboolean down)
 {
-	const char	*kb;
-	char		cmd[1024];
+	const char *bind;
+	qboolean q;
 
-	keydown[key] = down;
+	if (key < 0 || key >= MAX_KEYS)
+		return;
 
-	if (!down)
-		key_repeats[key] = 0;
+	// get key binding
+	bind = keybindings[key_bmap][key];
+	if (!bind)
+		bind = keybindings[key_bmap2][key];
 
-	key_lastpress = key;
-	key_count++;
-	if (key_count <= 0) {
-		return;							// just catching keys for Con_NotifyBox
-	}
+	if (developer.integer >= 1000)
+		Con_Printf("Key_Event(%i, '%c', %s) keydown %i bind \"%s\"\n", key, ascii, down ? "down" : "up", keydown[key], bind ? bind : "");
 
-	// update auto-repeat status
-	if (down) {
-		key_repeats[key]++;
-		if (key_repeats[key] > 1) {
-			if ((key_consoleactive && !consolekeys[key]) ||
-					(!key_consoleactive && key_dest == key_game &&
-					 (cls.state == ca_connected && cls.signon == SIGNONS)))
-				return;						// ignore most autorepeats
+	if(key_dest == key_game)
+	{
+		q = CL_VM_InputEvent(!down, key);
+		if(q)
+		{
+			if (down)
+				keydown[key] = min(keydown[key] + 1, 2);
+			else
+				keydown[key] = 0;
+			return;
 		}
 	}
 
-	if (key == K_CTRL)
-		ctrl_down = down;
+	if (down)
+	{
+		// increment key repeat count each time a down is received so that things
+		// which want to ignore key repeat can ignore it
+		keydown[key] = min(keydown[key] + 1, 2);
+	}
+	else
+	{
+		// clear repeat count now that the key is released
+		keydown[key] = 0;
+		// key up events only generate commands if the game key binding is a button
+		// command (leading + sign).  These will occur even in console mode, to
+		// keep the character from continuing an action started before a console
+		// switch.  Button commands include the kenum as a parameter, so multiple
+		// downs can be matched with ups
+		if (bind && bind[0] == '+')
+			Cbuf_AddText(va("-%s %i\n", bind + 1, key));
+	}
 
+	// key_consoleactive is a flag not a key_dest because the console is a
+	// high priority overlay ontop of the normal screen (designed as a safety
+	// feature so that developers and users can rescue themselves from a bad
+	// situation).
 	//
-	// handle escape specially, so the user can never unbind it
-	//
-	if (key == K_ESCAPE) {
-		if (!down)
+	// this also means that toggling the console on/off does not lose the old
+	// key_dest state
+
+	// specially handle escape (togglemenu) and shift-escape (toggleconsole)
+	// engine bindings, these are not handled as normal binds so that the user
+	// can recover from a completely empty bindmap
+	if (key == K_ESCAPE)
+	{
+		// ignore key repeats on escape
+		if (keydown[key] > 1)
+			return;
+		// escape does these things:
+		// key_consoleactive - close console
+		// key_message - abort messagemode
+		// key_menu - go to parent menu (or key_game)
+		// key_game - open menu
+		// in all modes shift-escape toggles console
+		if (((key_consoleactive & KEY_CONSOLEACTIVE_USER) || keydown[K_SHIFT]) && down)
+		{
+			Con_ToggleConsole_f ();
 			return;
-		switch (key_dest) {
+		}
+		switch (key_dest)
+		{
 			case key_message:
-				Key_Message (key, ascii);
+				if (down)
+					Key_Message (key, ascii);
 				break;
 			case key_menu:
-				MR_Keydown (key, ascii);
+				MR_KeyEvent (key, ascii, down);
 				break;
 			case key_game:
-				MR_ToggleMenu_f ();
+				if (down)
+					MR_ToggleMenu_f ();
 				break;
 			default:
-				if(UI_Callback_IsSlotUsed(key_dest - 3))
-					UI_Callback_KeyDown (key, ascii);
-				else
-					Sys_Error ("Bad key_dest");
+				Con_Printf ("Key_Event: Bad key_dest\n");
 		}
 		return;
 	}
 
-	if (down)
+	// send function keydowns to interpreter no matter what mode is
+	if (key >= K_F1 && key <= K_F12 && down)
 	{
-		if (!(kb = keybindings[key_bmap][key]))
-			kb = keybindings[key_bmap2][key];
-		if (kb && !strncmp(kb, "toggleconsole", strlen("toggleconsole")))
-		{
-			Cbuf_AddText (kb);
-			Cbuf_AddText ("\n");
+		// ignore key repeats on F1-F12 binds
+		if (keydown[key] > 1)
 			return;
+		if (bind)
+		{
+			// button commands add keynum as a parm
+			if (bind[0] == '+')
+				Cbuf_AddText (va("%s %i\n", bind, key));
+			else
+			{
+				Cbuf_AddText (bind);
+				Cbuf_AddText ("\n");
+			}
 		}
+		return;
 	}
 
-	if (key_consoleactive && consolekeys[key] && down)
-		Key_Console (key, ascii);
-	else
+#if 0
+	// ignore binds (other than the above escape/F1-F12 keys) while in console
+	if (key_consoleactive && down)
+#else
+	// respond to toggleconsole binds while in console unless the pressed key
+	// happens to be the color prefix character (such as on German keyboards)
+	if (key_consoleactive && down && (!con_closeontoggleconsole.integer || !bind || strncmp(bind, "toggleconsole", strlen("toggleconsole")) || ascii == STRING_COLOR_TAG))
+#endif
 	{
-		//
-		// key up events only generate commands if the game key binding is a button
-		// command (leading + sign).  These will occur even in console mode, to
-		// keep the character from continuing an action started before a console
-		// switch.  Button commands include the kenum as a parameter, so multiple
-		// downs can be matched with ups
-		//
-		if (!down) {
-			if (!(kb = keybindings[key_bmap][key]))
-				kb = keybindings[key_bmap2][key];
-
-			if (kb && kb[0] == '+') {
-				dpsnprintf (cmd, sizeof(cmd), "-%s %i\n", kb + 1, key);
-				Cbuf_AddText (cmd);
-			}
-			return;
-		}
+		Key_Console (key, ascii);
+		return;
+	}
 
-		//
-		// during demo playback, most keys bring up the main menu
-		//
-		if (cls.demoplayback && down && consolekeys[key] && key_dest == key_game) {
-			MR_ToggleMenu_f ();
-			return;
-		}
+	// ignore binds while a video is played, let the video system handle the key event
+	if (cl_videoplaying)
+	{
+		CL_Video_KeyEvent (key, ascii, keydown[key] != 0);
+		return;
+	}
 
-		//
-		// if not a consolekey, send to the interpreter no matter what mode is
-		//
-		if ((key_dest == key_menu && menubound[key])
-				|| (key_consoleactive && !consolekeys[key])
-				|| (key_dest == key_game &&
-					((cls.state == ca_connected) || !consolekeys[key]))) {
-			if (!(kb = keybindings[key_bmap][key]))
-				kb = keybindings[key_bmap2][key];
-			if (kb) {
-				if (kb[0] == '+') {			// button commands add keynum as a parm
-					dpsnprintf (cmd, sizeof(cmd), "%s %i\n", kb, key);
-					Cbuf_AddText (cmd);
-				} else {
-					Cbuf_AddText (kb);
+	// anything else is a key press into the game, chat line, or menu
+	switch (key_dest)
+	{
+		case key_message:
+			if (down)
+				Key_Message (key, ascii);
+			break;
+		case key_menu:
+			MR_KeyEvent (key, ascii, down);
+			break;
+		case key_game:
+			// ignore key repeats on binds
+			if (bind && keydown[key] == 1 && down)
+			{
+				// button commands add keynum as a parm
+				if (bind[0] == '+')
+					Cbuf_AddText (va("%s %i\n", bind, key));
+				else
+				{
+					Cbuf_AddText (bind);
 					Cbuf_AddText ("\n");
 				}
 			}
-			return;
-		}
-
-		if (!down)
-			return;							// other systems only care about key
-		// down events
-
-		switch (key_dest) {
-			case key_message:
-				Key_Message (key, ascii);
-				break;
-			case key_menu:
-				MR_Keydown (key, ascii);
-				break;
-			case key_game:
-				Key_Console (key, ascii);
-				break;
-			default:
-				if(UI_Callback_IsSlotUsed(key_dest - 3))
-					UI_Callback_KeyDown (key, ascii);
-				else
-					Sys_Error ("Bad key_dest");
-		}
+			break;
+		default:
+			Con_Printf ("Key_Event: Bad key_dest\n");
 	}
 }
 
@@ -1005,11 +1038,5 @@ Key_ClearStates
 void
 Key_ClearStates (void)
 {
-	int i;
-
-	for (i = 0; i < (int)(sizeof(keydown)/sizeof(keydown[0])); i++)
-	{
-		keydown[i] = false;
-		key_repeats[i] = 0;
-	}
+	memset(keydown, 0, sizeof(keydown));
 }