[ros-diffs] [jimtabor] 29092: - Win32k: Removed NtUserCreateMenu replacing it with NtUserCallNoParam.

Author: jimtabor
Date: Tue Sep 18 17:06:27 2007
New Revision: 29092

URL: http://svn.reactos.org/svn/reactos?rev=29092&view=rev
Log:
- Win32k: Removed NtUserCreateMenu replacing it with NtUserCallNoParam.

Modified:
    trunk/reactos/dll/win32/user32/windows/menu.c
    trunk/reactos/include/reactos/win32k/ntuser.h
    trunk/reactos/subsystems/win32/win32k/ntuser/menu.c
    trunk/reactos/subsystems/win32/win32k/ntuser/misc.c
    trunk/reactos/subsystems/win32/win32k/ntuser/window.c
    trunk/reactos/subsystems/win32/win32k/w32ksvc.db

Modified: trunk/reactos/dll/win32/user32/windows/menu.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/user32/windows/me...
==============================================================================
--- trunk/reactos/dll/win32/user32/windows/menu.c (original)
+++ trunk/reactos/dll/win32/user32/windows/menu.c Tue Sep 18 17:06:27 2007
@@ -3899,7 +3899,7 @@
 CreateMenu(VOID)
 {
   MenuLoadBitmaps();
-  return NtUserCreateMenu(FALSE);
+  return (HMENU)NtUserCallNoParam(NOPARAM_ROUTINE_CREATEMENU);
 }

@@ -3910,7 +3910,7 @@
 CreatePopupMenu(VOID)
 {
   MenuLoadBitmaps();
-  return NtUserCreateMenu(TRUE);
+  return (HMENU)NtUserCallNoParam(NOPARAM_ROUTINE_CREATEMENUPOPUP);
 }

Modified: trunk/reactos/include/reactos/win32k/ntuser.h
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/win32k/ntus...
==============================================================================
--- trunk/reactos/include/reactos/win32k/ntuser.h (original)
+++ trunk/reactos/include/reactos/win32k/ntuser.h Tue Sep 18 17:06:27 2007
@@ -106,10 +106,6 @@
   UINT uIDCheckItem,
   UINT uCheck);

-HMENU
-NTAPI
-NtUserCreateMenu(BOOL PopupMenu);
-
 BOOL
 NTAPI
 NtUserDeleteMenu(
@@ -440,6 +436,8 @@
   WPARAM wParam,
   LPARAM lParam);

+#define NOPARAM_ROUTINE_CREATEMENU            0x0
+#define NOPARAM_ROUTINE_CREATEMENUPOPUP       0x1
 #define NOPARAM_ROUTINE_MSQCLEARWAKEMASK      0x3
 #define NOPARAM_ROUTINE_REGISTER_PRIMITIVE	  0xffff0001 /* Private ROS */
 #define NOPARAM_ROUTINE_DESTROY_CARET         0xffff0002

Modified: trunk/reactos/subsystems/win32/win32k/ntuser/menu.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ntu...
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntuser/menu.c (original)
+++ trunk/reactos/subsystems/win32/win32k/ntuser/menu.c Tue Sep 18 17:06:27 2007
@@ -1481,23 +1481,6 @@

return (HMENU)Handle;
 }
-
-HMENU STDCALL
-NtUserCreateMenu(BOOL PopupMenu)
-{
-   DECLARE_RETURN(HMENU);
-
-   DPRINT("Enter NtUserCreateMenu\n");
-   UserEnterExclusive();
-
-   RETURN(UserCreateMenu(PopupMenu));
-
-CLEANUP:
-   DPRINT("Leave NtUserCreateMenu, ret=%i\n",_ret_);
-   UserLeave();
-   END_CLEANUP;
-}
-

/*

Modified: trunk/reactos/subsystems/win32/win32k/ntuser/misc.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ntu...
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntuser/misc.c (original)
+++ trunk/reactos/subsystems/win32/win32k/ntuser/misc.c Tue Sep 18 17:06:27 2007
@@ -86,6 +86,14 @@

switch(Routine)
    {
+      case NOPARAM_ROUTINE_CREATEMENU:
+         Result = (DWORD)UserCreateMenu(FALSE);
+         break;
+
+      case NOPARAM_ROUTINE_CREATEMENUPOPUP:
+         Result = (DWORD)UserCreateMenu(TRUE);
+         break;
+
       case NOPARAM_ROUTINE_DESTROY_CARET:
          Result = (DWORD)co_IntDestroyCaret(PsGetCurrentThread()->Tcb.Win32Thread);
          break;

Modified: trunk/reactos/subsystems/win32/win32k/ntuser/window.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ntu...
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntuser/window.c (original)
+++ trunk/reactos/subsystems/win32/win32k/ntuser/window.c Tue Sep 18 17:06:27 2007
@@ -4730,29 +4730,29 @@
        }
        case otCursorIcon:
        {
-          PCURICON_OBJECT Cursor;
-          if ((Cursor = UserGetCurIconObject((HCURSOR) handle))) return TRUE;
-          return FALSE;
+         PCURICON_OBJECT Cursor;
+         if ((Cursor = UserGetCurIconObject((HCURSOR) handle))) return TRUE;
+         return FALSE;
        }
        case otHook:
        {
-          PHOOK Hook;
-          if ((Hook = IntGetHookObject((HHOOK) handle))) return TRUE;
-          return FALSE;
+         PHOOK Hook;
+         if ((Hook = IntGetHookObject((HHOOK) handle))) return TRUE;
+         return FALSE;
        }
        case otMonitor:
        {
-          PMONITOR_OBJECT Monitor;
-          if ((Monitor = UserGetMonitorObject((HMONITOR) handle))) return TRUE;
-          return FALSE;
+         PMONITOR_OBJECT Monitor;
+         if ((Monitor = UserGetMonitorObject((HMONITOR) handle))) return TRUE;
+         return FALSE;
        }
        case otCallProc:
        {
-           WNDPROC_INFO Proc;
-           return UserGetCallProcInfo( handle, &Proc );
+         WNDPROC_INFO Proc;
+         return UserGetCallProcInfo( handle, &Proc );
        }
        default:
-          SetLastWin32Error(ERROR_INVALID_HANDLE);
+         SetLastWin32Error(ERROR_INVALID_HANDLE);
      }
    }
    else

Modified: trunk/reactos/subsystems/win32/win32k/w32ksvc.db
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/w32...
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/w32ksvc.db (original)
+++ trunk/reactos/subsystems/win32/win32k/w32ksvc.db Tue Sep 18 17:06:27 2007
@@ -709,7 +709,6 @@
 NtGdiUpdateICMRegKey                    4
 NtUserBuildMenuItemList                 4
 NtUserCreateCursorIconHandle            2
-NtUserCreateMenu                        1
 NtUserDestroyCursorIcon                 2
 NtUserEnumClipboardFormats              1 
 NtUserGetActiveWindow                   0