[ros-diffs] [rharabien] 53659: [WIN32K] - Don't use magic type values for IntMapVirtualKeyEx calls - Fix a typo in ERR

Author: rharabien
Date: Fri Sep  9 16:08:09 2011
New Revision: 53659

URL: http://svn.reactos.org/svn/reactos?rev=53659&view=rev
Log:
[WIN32K]
- Don't use magic type values for IntMapVirtualKeyEx calls
- Fix a typo in ERR

Modified:
    trunk/reactos/subsystems/win32/win32k/ntuser/input.c
    trunk/reactos/subsystems/win32/win32k/ntuser/keyboard.c

Modified: trunk/reactos/subsystems/win32/win32k/ntuser/input.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt…
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntuser/input.c [iso-8859-1] (original)
+++ trunk/reactos/subsystems/win32/win32k/ntuser/input.c [iso-8859-1] Fri Sep  9 16:08:09
2011
@@ -602,7 +602,7 @@
          }
          if(!NT_SUCCESS(Status))
          {
-            ERR("Win32K: Failed to read from mouse.\n");
+            ERR("Win32K: Failed to read from keyboard.\n");
             return; //(Status);
          }
 

Modified: trunk/reactos/subsystems/win32/win32k/ntuser/keyboard.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt…
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntuser/keyboard.c [iso-8859-1] (original)
+++ trunk/reactos/subsystems/win32/win32k/ntuser/keyboard.c [iso-8859-1] Fri Sep  9
16:08:09 2011
@@ -318,7 +318,7 @@
    {
       if( bLigature )
       {
-         TRACE("Not handling ligature (yet)\n" );
+         WARN("Not handling ligature (yet)\n" );
          return 0;
       }
 
@@ -579,7 +579,7 @@
 
    switch( Type )
    {
-      case 0:
+      case MAPVK_VK_TO_VSC:
          if( Code == VK_SHIFT )
             Code = VK_LSHIFT;
          if( Code == VK_MENU )
@@ -589,13 +589,13 @@
          ret = VkToScan( Code, FALSE, keyLayout );
          break;
 
-      case 1:
+      case MAPVK_VSC_TO_VK:
          ret =
             DontDistinguishShifts
-            (IntMapVirtualKeyEx( Code, 3, keyLayout ) );
+            (IntMapVirtualKeyEx( Code, MAPVK_VSC_TO_VK_EX, keyLayout ) );
          break;
 
-      case 2:
+      case MAPVK_VK_TO_CHAR:
          {
             WCHAR wp[2] = {0};
 
@@ -605,10 +605,17 @@
          }
          break;
 
-      case 3:
+      case MAPVK_VSC_TO_VK_EX:
 
          ret = ScanToVk( Code, FALSE, keyLayout );
          break;
+      
+      case MAPVK_VK_TO_VSC_EX:
+         STUB;
+         break;
+      
+      default:
+         ERR("Wrong type value: %u\n", Type);
    }
 
    return ret;
@@ -796,7 +803,7 @@
    {
       WCHAR UCName[2];
 
-      UCName[0] = W32kSimpleToupper(IntMapVirtualKeyEx( VkCode, 2, keyLayout ));
+      UCName[0] = W32kSimpleToupper(IntMapVirtualKeyEx( VkCode, MAPVK_VK_TO_CHAR,
keyLayout ));
       UCName[1] = 0;
       ret = 1;
 
@@ -860,7 +867,7 @@
    TRACE("ScanCode %04x\n",ScanCode);
 
    BaseMapping = Msg->wParam =
-                    IntMapVirtualKeyEx( ScanCode, 1, KeyboardLayout );
+                    IntMapVirtualKeyEx( ScanCode, MAPVK_VSC_TO_VK, KeyboardLayout );
    if( Prefix == 0 )
    {
       if( ScanCode >= KeyboardLayout->bMaxVSCtoVK )