[Scummvm-cvs-logs] SF.net SVN: scummvm: [29924] scummvm/trunk/engines/agi/agi.cpp

[thebluegr at users.sourceforge.net]

Revision: 29924
          http://scummvm.svn.sourceforge.net/scummvm/?rev=29924&view=rev
Author:   thebluegr
Date:     2007-12-20 09:04:15 -0800 (Thu, 20 Dec 2007)

Log Message:
-----------
Fix for isalpha() assertions

Modified Paths:
--------------
    scummvm/trunk/engines/agi/agi.cpp

Modified: scummvm/trunk/engines/agi/agi.cpp
===================================================================
--- scummvm/trunk/engines/agi/agi.cpp	2007-12-20 15:38:22 UTC (rev 29923)
+++ scummvm/trunk/engines/agi/agi.cpp	2007-12-20 17:04:15 UTC (rev 29924)
@@ -219,6 +219,11 @@
 				key = KEY_BACKSPACE;
 				break;
 			default:
+				// FIXME: This fixes assertions with isalpha below, but it essentially filters
+				// out all function keys (control, alt and shift)
+				if (key > 255)
+					break;
+
 				// FIXME: We let lots of keys slip through here unchanged, passing our internal
 				// keycode values directly to the AGI core. Do we really want that???
 				if (isalpha(key)) {


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.