r1464 - trunk/code/sdl

[DONOTREPLY at icculus.org]

Author: tma
Date: 2008-08-30 15:29:34 -0400 (Sat, 30 Aug 2008)
New Revision: 1464

Modified:
   trunk/code/sdl/sdl_input.c
Log:
* Update IN_IsConsoleKey so that specifying ascii values/characters for the
  console key always results in the character being used as opposed to the key


Modified: trunk/code/sdl/sdl_input.c
===================================================================
--- trunk/code/sdl/sdl_input.c	2008-08-30 19:26:41 UTC (rev 1463)
+++ trunk/code/sdl/sdl_input.c	2008-08-30 19:29:34 UTC (rev 1464)
@@ -120,9 +120,24 @@
 IN_IsConsoleKey
 ===============
 */
-static qboolean IN_IsConsoleKey( keyNum_t key, const char *buf )
+static qboolean IN_IsConsoleKey( keyNum_t key, const char character )
 {
-	static int consoleKeys[ MAX_CONSOLE_KEYS ];
+	typedef struct consoleKey_s
+	{
+		enum
+		{
+			KEY,
+			CHARACTER
+		} type;
+
+		union
+		{
+			keyNum_t key;
+			char character;
+		} u;
+	} consoleKey_t;
+
+	static consoleKey_t consoleKeys[ MAX_CONSOLE_KEYS ];
 	static int numConsoleKeys = 0;
 	int i;
 
@@ -137,26 +152,53 @@
 
 		while( numConsoleKeys < MAX_CONSOLE_KEYS )
 		{
+			consoleKey_t *c = &consoleKeys[ numConsoleKeys ];
+			char *keyName;
+
 			token = COM_Parse( &text_p );
 			if( !token[ 0 ] )
 				break;
 
-			consoleKeys[ numConsoleKeys++ ] =
-				Key_StringToKeynum( token );
+			c->u.key = Key_StringToKeynum( token );
+
+			// 0 isn't a key
+			if( c->u.key == 0 )
+				continue;
+
+			keyName = Key_KeynumToString( c->u.key );
+
+			if( strlen( keyName ) == 1 )
+			{
+				c->type = CHARACTER;
+				c->u.character = *keyName;
+			}
+			else
+				c->type = KEY;
+
+			numConsoleKeys++;
 		}
 	}
 
 	// Use the character in preference to the key name
-	if( *buf )
+	if( character != '\0' )
 		key = 0;
 
 	for( i = 0; i < numConsoleKeys; i++ )
 	{
-		if( !consoleKeys[ i ] )
-			continue;
+		consoleKey_t *c = &consoleKeys[ i ];
 
-		if( consoleKeys[ i ] == key || consoleKeys[ i ] == *buf )
-			return qtrue;
+		switch( c->type )
+		{
+			case KEY:
+				if( key && c->u.key == key )
+					return qtrue;
+				break;
+
+			case CHARACTER:
+				if( c->u.character == character )
+					return qtrue;
+				break;
+		}
 	}
 
 	return qfalse;
@@ -291,7 +333,7 @@
 	if( in_keyboardDebug->integer )
 		IN_PrintKey( keysym, *key, down );
 
-	if( IN_IsConsoleKey( *key, buf ) )
+	if( IN_IsConsoleKey( *key, *buf ) )
 	{
 		// Console keys can't be bound or generate characters
 		*key = K_CONSOLE;