[USER32] Improve the check for item in MENU_FindItem().
[reactos.git] / reactos / win32ss / user / user32 / windows / menu.c
index 7275073..a83666f 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * COPYRIGHT:       See COPYING in the top level directory
  * PROJECT:         ReactOS user32.dll
- * FILE:            user32/windows/menu.c
+ * FILE:            win32ss/user/user32/windows/menu.c
  * PURPOSE:         Menus
  *
  * PROGRAMMERS:     Casper S. Hornstrup
@@ -13,7 +13,7 @@
 #include <user32.h>
 #include <wine/debug.h>
 
-LRESULT DefWndNCPaint(HWND hWnd, HRGN hRgn, BOOL Active);
+BOOL WINAPI GdiValidateHandle(HGDIOBJ hobj);
 
 WINE_DEFAULT_DEBUG_CHANNEL(menu);
 
@@ -22,60 +22,37 @@ WINE_DEFAULT_DEBUG_CHANNEL(menu);
 #define MM_SETMENUHANDLE       (WM_USER + 0)
 #define MM_GETMENUHANDLE       (WM_USER + 1)
 
-/* internal flags for menu tracking */
+#define MENU_TYPE_MASK (MF_STRING | MF_BITMAP | MF_OWNERDRAW | MF_SEPARATOR)
 
-#define TF_ENDMENU              0x10000
-#define TF_SUSPENDPOPUP         0x20000
-#define TF_SKIPREMOVE           0x40000
+#define MENU_ITEM_TYPE(flags) ((flags) & MENU_TYPE_MASK)
 
-#define ITEM_PREV              -1
-#define ITEM_NEXT               1
+#define MNS_STYLE_MASK (MNS_NOCHECK|MNS_MODELESS|MNS_DRAGDROP|MNS_AUTODISMISS|MNS_NOTIFYBYPOS|MNS_CHECKORBMP)
 
-/* Internal MenuTrackMenu() flags */
-#define TPM_INTERNAL           0xF0000000
-#define TPM_BUTTONDOWN         0x40000000              /* menu was clicked before tracking */
-#define TPM_POPUPMENU           0x20000000              /* menu is a popup menu */
+#define MENUITEMINFO_TYPE_MASK \
+                (MFT_STRING | MFT_BITMAP | MFT_OWNERDRAW | MFT_SEPARATOR | \
+                 MFT_MENUBARBREAK | MFT_MENUBREAK | MFT_RADIOCHECK | \
+                 MFT_RIGHTORDER | MFT_RIGHTJUSTIFY /* same as MF_HELP */ )
 
+#define TYPE_MASK  (MENUITEMINFO_TYPE_MASK | MF_POPUP | MF_SYSMENU)
 
-#define MENU_TYPE_MASK (MF_STRING | MF_BITMAP | MF_OWNERDRAW | MF_SEPARATOR)
+#define STATE_MASK (~TYPE_MASK)
 
-#define MENU_ITEM_TYPE(flags) ((flags) & MENU_TYPE_MASK)
+#define MENUITEMINFO_STATE_MASK (STATE_MASK & ~(MF_BYPOSITION | MF_MOUSESELECT))
+
+#define MII_STATE_MASK (MFS_GRAYED|MFS_CHECKED|MFS_HILITE|MFS_DEFAULT)
 
 /* macro to test that flags do not indicate bitmap, ownerdraw or separator */
 #define IS_STRING_ITEM(flags) (MF_STRING == MENU_ITEM_TYPE(flags))
 #define IS_MAGIC_BITMAP(id) ((id) && ((INT_PTR)(id) < 12) && ((INT_PTR)(id) >= -1))
 
 #define IS_SYSTEM_MENU(MenuInfo)  \
-       (0 == ((MenuInfo)->Flags & MF_POPUP) && 0 != ((MenuInfo)->Flags & MF_SYSMENU))
+       (0 == ((MenuInfo)->fFlags & MNF_POPUP) && 0 != ((MenuInfo)->fFlags & MNF_SYSMENU))
 
 #define IS_SYSTEM_POPUP(MenuInfo) \
-       (0 != ((MenuInfo)->Flags & MF_POPUP) && 0 != ((MenuInfo)->Flags & MF_SYSMENU))
+       (0 != ((MenuInfo)->fFlags & MNF_POPUP) && 0 != ((MenuInfo)->fFlags & MNF_SYSMENU))
 
 #define IS_BITMAP_ITEM(flags) (MF_BITMAP == MENU_ITEM_TYPE(flags))
 
-/* Use global popup window because there's no way 2 menus can
- * be tracked at the same time.  */
-static HWND top_popup;
-static HMENU top_popup_hmenu;
-
-/* Flag set by EndMenu() to force an exit from menu tracking */
-static BOOL fEndMenu = FALSE;
-
-#define MENU_ITEM_HBMP_SPACE (5)
-#define MENU_BAR_ITEMS_SPACE (12)
-#define SEPARATOR_HEIGHT (5)
-#define MENU_TAB_SPACE (8)
-
-typedef struct
-{
-  UINT  TrackFlags;
-  HMENU CurrentMenu; /* current submenu (can be equal to hTopMenu)*/
-  HMENU TopMenu;     /* initial menu */
-  HWND  OwnerWnd;    /* where notifications are sent */
-  POINT Pt;
-} MTRACKER;
-
-
 /*********************************************************************
  * PopupMenu class descriptor
  */
@@ -83,8 +60,8 @@ const struct builtin_class_descr POPUPMENU_builtin_class =
 {
     WC_MENU,                     /* name */
     CS_SAVEBITS | CS_DBLCLKS,                  /* style  */
-    (WNDPROC) NULL,                            /* FIXME - procA */
-    (WNDPROC) PopupMenuWndProcW,               /* FIXME - procW */
+    NULL,                                      /* FIXME - procA */
+    PopupMenuWndProcW,                         /* FIXME - procW */
     sizeof(MENUINFO *),                        /* extra */
     (LPCWSTR) IDC_ARROW,                       /* cursor */
     (HBRUSH)(COLOR_MENU + 1)                   /* brush */
@@ -97,3818 +74,603 @@ const struct builtin_class_descr POPUPMENU_builtin_class =
 #define GET_DWORD(ptr) (*(DWORD *)(ptr))
 #endif
 
-HFONT hMenuFont = NULL;
-HFONT hMenuFontBold = NULL;
-
-/* Dimension of the menu bitmaps */
-static HBITMAP BmpSysMenu = NULL;
-
-static SIZE MenuCharSize;
 
 /***********************************************************************
- *           MenuGetRosMenuInfo
+ *           MENU_GetMenu
  *
- * Get full information about menu
+ * Validate the given menu handle and returns the menu structure pointer.
  */
-static BOOL FASTCALL
-MenuGetRosMenuInfo(PROSMENUINFO MenuInfo, HMENU Menu)
+FORCEINLINE PMENU MENU_GetMenu(HMENU hMenu)
 {
-  MenuInfo->cbSize = sizeof(ROSMENUINFO);
-  MenuInfo->fMask = MIM_BACKGROUND | MIM_HELPID | MIM_MAXHEIGHT | MIM_MENUDATA | MIM_STYLE;
-
-  return NtUserMenuInfo(Menu, MenuInfo, FALSE);
+    return ValidateHandleNoErr(hMenu, TYPE_MENU); 
 }
 
 /***********************************************************************
- *           MenuSetRosMenuInfo
+ *           MENU_FindItem
  *
- * Set full information about menu
+ * Find a menu item. Return a pointer on the item, and modifies *hmenu
+ * in case the item was in a sub-menu.
  */
-static BOOL FASTCALL
-MenuSetRosMenuInfo(PROSMENUINFO MenuInfo)
+ITEM *MENU_FindItem( HMENU *hmenu, UINT *nPos, UINT wFlags )
 {
-  MenuInfo->cbSize = sizeof(ROSMENUINFO);
-  MenuInfo->fMask = MIM_BACKGROUND | MIM_HELPID | MIM_MAXHEIGHT | MIM_MENUDATA | MIM_STYLE;
+    MENU *menu;
+    ITEM *fallback = NULL;
+    UINT fallback_pos = 0;
+    UINT i;
+    PITEM pItem;
 
-  return NtUserMenuInfo(MenuInfo->Self, MenuInfo, TRUE);
-}
+    if ((*hmenu == (HMENU)0xffff) || (!(menu = MENU_GetMenu(*hmenu)))) return NULL;
+    if (wFlags & MF_BYPOSITION)
+    {
+       if (*nPos >= menu->cItems) return NULL;
+       pItem = menu->rgItems ? DesktopPtrToUser(menu->rgItems) : NULL;
+       if (pItem) pItem = &pItem[*nPos];
+       return pItem;
+    }
+    else
+    {
+        PITEM item = menu->rgItems ? DesktopPtrToUser(menu->rgItems) : NULL;
+       for (i = 0; item && (i < menu->cItems); i++, item++)
+       {
+           if (item->spSubMenu)
+           {
+               PMENU pSubMenu = DesktopPtrToUser(item->spSubMenu);
+               HMENU hsubmenu = UserHMGetHandle(pSubMenu);
+               ITEM *subitem = MENU_FindItem( &hsubmenu, nPos, wFlags );
+               if (subitem)
+               {
+                   *hmenu = hsubmenu;
+                   return subitem;
+               }
+               else if (item->wID == *nPos)
+               {
+                   /* fallback to this item if nothing else found */
+                   fallback_pos = i;
+                   fallback = item;
+               }
+           }
+           else if (item->wID == *nPos)
+           {
+               *nPos = i;
+               return item;
+           }
+       }
+    }
 
-/***********************************************************************
- *           MenuInitRosMenuItemInfo
- *
- * Initialize a buffer for use with MenuGet/SetRosMenuItemInfo
- */
-static VOID FASTCALL
-MenuInitRosMenuItemInfo(PROSMENUITEMINFO ItemInfo)
-{
-  ZeroMemory(ItemInfo, sizeof(ROSMENUITEMINFO));
-  ItemInfo->cbSize = sizeof(ROSMENUITEMINFO);
+    if (fallback)
+        *nPos = fallback_pos;
+
+    return fallback;
 }
 
-/***********************************************************************
- *           MenuGetRosMenuItemInfo
- *
- * Get full information about a menu item
- */
-static BOOL FASTCALL
-MenuGetRosMenuItemInfo(HMENU Menu, UINT Index, PROSMENUITEMINFO ItemInfo)
+UINT FASTCALL
+IntGetMenuDefaultItem(PMENU Menu, BOOL fByPos, UINT gmdiFlags, DWORD *gismc)
 {
-  UINT Save_Mask = ItemInfo->fMask; /* Save the org mask bits. */
+   UINT i = 0;
+   PITEM Item = Menu->rgItems ? DesktopPtrToUser(Menu->rgItems) : NULL;
 
-  if (ItemInfo->dwTypeData != NULL)
-    {
-      HeapFree(GetProcessHeap(), 0, ItemInfo->dwTypeData);
-    }
+   /* empty menu */
+   if (!Item) return -1;
+
+   while ( !( Item->fState & MFS_DEFAULT ) )
+   {
+      i++; Item++;
+      if  (i >= Menu->cItems ) return -1;
+   }
+
+   /* default: don't return disabled items */
+   if ( (!(GMDI_USEDISABLED & gmdiFlags)) && (Item->fState & MFS_DISABLED )) return -1;
+
+   /* search rekursiv when needed */
+   if ( (Item->fType & MF_POPUP) &&  (gmdiFlags & GMDI_GOINTOPOPUPS) && Item->spSubMenu)
+   {
+      UINT ret;
+      (*gismc)++;
+      ret = IntGetMenuDefaultItem( DesktopPtrToUser(Item->spSubMenu), fByPos, gmdiFlags, gismc );
+      (*gismc)--;
+      if ( -1 != ret ) return ret;
+
+      /* when item not found in submenu, return the popup item */
+   }
+   return ( fByPos ) ? i : Item->wID;
+}
 
+static BOOL GetMenuItemInfo_common ( HMENU hmenu,
+                                     UINT item,
+                                     BOOL bypos,
+                                     LPMENUITEMINFOW lpmii,
+                                     BOOL unicode)
+{
+    ITEM *pItem = MENU_FindItem (&hmenu, &item, bypos ? MF_BYPOSITION : 0);
 
-  ItemInfo->fMask = MIIM_BITMAP | MIIM_CHECKMARKS | MIIM_DATA | MIIM_FTYPE
-             | MIIM_ID | MIIM_STATE | MIIM_STRING | MIIM_SUBMENU | MIIM_TYPE;
-  ItemInfo->dwTypeData = NULL;
+    //debug_print_menuitem("GetMenuItemInfo_common: ", pItem, "");
 
-  if (! NtUserMenuItemInfo(Menu, Index, TRUE, ItemInfo, FALSE))
+    if (!pItem)
     {
-      ItemInfo->fType = 0;
-      return FALSE;
+        SetLastError( ERROR_MENU_ITEM_NOT_FOUND);
+        return FALSE;
     }
 
-  if (MENU_ITEM_TYPE(ItemInfo->fType) == MF_STRING)
+    if( lpmii->fMask & MIIM_TYPE)
     {
-      ItemInfo->cch++;
-      ItemInfo->dwTypeData = HeapAlloc(GetProcessHeap(), 0,
-                                       ItemInfo->cch * sizeof(WCHAR));
-      if (NULL == ItemInfo->dwTypeData)
+        if( lpmii->fMask & ( MIIM_STRING | MIIM_FTYPE | MIIM_BITMAP))
         {
-          return FALSE;
+            ERR("invalid combination of fMask bits used\n");
+            /* this does not happen on Win9x/ME */
+            SetLastError( ERROR_INVALID_PARAMETER);
+            return FALSE;
         }
-
-      if (! NtUserMenuItemInfo(Menu, Index, TRUE, ItemInfo, FALSE))
+       lpmii->fType = pItem->fType & MENUITEMINFO_TYPE_MASK;
+        if( pItem->hbmp) lpmii->fType |= MFT_BITMAP;
+       lpmii->hbmpItem = pItem->hbmp; /* not on Win9x/ME */
+        if( lpmii->fType & MFT_BITMAP)
         {
-          ItemInfo->fType = 0;
-          return FALSE;
+           lpmii->dwTypeData = (LPWSTR) pItem->hbmp;
+           lpmii->cch = 0;
+        }
+        else if( lpmii->fType & (MFT_OWNERDRAW | MFT_SEPARATOR))
+        {
+            /* this does not happen on Win9x/ME */
+           lpmii->dwTypeData = 0;
+           lpmii->cch = 0;
         }
-      ItemInfo->dwTypeData[ItemInfo->cch - 1] = UNICODE_NULL;
     }
-  ItemInfo->fMask =  Save_Mask;
-  return TRUE;
-}
 
-/***********************************************************************
- *           MenuSetRosMenuItemInfo
- *
- * Set selected information about a menu item, need to set the mask bits.
- */
-static BOOL FASTCALL
-MenuSetRosMenuItemInfo(HMENU Menu, UINT Index, PROSMENUITEMINFO ItemInfo)
-{
-  BOOL Ret;
+    /* copy the text string */
+    if ((lpmii->fMask & (MIIM_TYPE|MIIM_STRING)))
+    {
+         if( !pItem->Xlpstr )
+         {                                         // Very strange this fixes a wine test with a crash.
+                if(lpmii->dwTypeData && lpmii->cch && !(GdiValidateHandle((HGDIOBJ)lpmii->dwTypeData)) )
+                {
+                    if( unicode)
+                        *((WCHAR *)lpmii->dwTypeData) = 0;
+                    else
+                        *((CHAR *)lpmii->dwTypeData) = 0;
+                }
+                lpmii->cch = 0;
+         }
+         else
+         {
+            int len;
+            LPWSTR text = DesktopPtrToUser(pItem->Xlpstr);
+            if (unicode)
+            {
+                len = strlenW(text);
+                if(lpmii->dwTypeData && lpmii->cch)
+                    lstrcpynW(lpmii->dwTypeData, text, lpmii->cch);
+            }
+            else
+            {
+                len = WideCharToMultiByte( CP_ACP, 0, text, -1, NULL, 0, NULL, NULL ) - 1;
+                if(lpmii->dwTypeData && lpmii->cch)
+                    if (!WideCharToMultiByte( CP_ACP, 0, text, -1,
+                            (LPSTR)lpmii->dwTypeData, lpmii->cch, NULL, NULL ))
+                        ((LPSTR)lpmii->dwTypeData)[lpmii->cch - 1] = 0;
+            }
+            /* if we've copied a substring we return its length */
+            if(lpmii->dwTypeData && lpmii->cch)
+                if (lpmii->cch <= len + 1)
+                    lpmii->cch--;
+                else
+                    lpmii->cch = len;
+            else
+            {
+                /* return length of string */
+                /* not on Win9x/ME if fType & MFT_BITMAP */
+                lpmii->cch = len;
+            }
+        }
+    }
 
-  if (MENU_ITEM_TYPE(ItemInfo->fType) == MF_STRING &&
-      ItemInfo->dwTypeData != NULL)
-  {
-    ItemInfo->cch = strlenW(ItemInfo->dwTypeData);
-  }
-  Ret = NtUserMenuItemInfo(Menu, Index, TRUE, ItemInfo, TRUE);
+    if (lpmii->fMask & MIIM_FTYPE)
+       lpmii->fType = pItem->fType & MENUITEMINFO_TYPE_MASK;
 
-  return Ret;
-}
+    if (lpmii->fMask & MIIM_BITMAP)
+       lpmii->hbmpItem = pItem->hbmp;
 
-/***********************************************************************
- *           MenuCleanupRosMenuItemInfo
- *
- * Cleanup after use of MenuGet/SetRosMenuItemInfo
- */
-static VOID FASTCALL
-MenuCleanupRosMenuItemInfo(PROSMENUITEMINFO ItemInfo)
-{
-  if (ItemInfo->dwTypeData != NULL)
-    {
-      HeapFree(GetProcessHeap(), 0, ItemInfo->dwTypeData);
-      ItemInfo->dwTypeData = NULL;
-    }
-}
+    if (lpmii->fMask & MIIM_STATE)
+       lpmii->fState = pItem->fState & MENUITEMINFO_STATE_MASK;
 
-/***********************************************************************
- *           MenuGetAllRosMenuItemInfo
- *
- * Get full information about all menu items
- */
-static INT FASTCALL
-MenuGetAllRosMenuItemInfo(HMENU Menu, PROSMENUITEMINFO *ItemInfo)
-{
-  DWORD BufSize;
+    if (lpmii->fMask & MIIM_ID)
+       lpmii->wID = pItem->wID;
 
-  BufSize = NtUserBuildMenuItemList(Menu, (VOID *) 1, 0, 0);
-  if (BufSize == (DWORD) -1 || BufSize == 0)
+    if (lpmii->fMask & MIIM_SUBMENU && pItem->spSubMenu )
+    {
+        PMENU pSubMenu = DesktopPtrToUser(pItem->spSubMenu);
+        HMENU hSubMenu = UserHMGetHandle(pSubMenu);
+       lpmii->hSubMenu = hSubMenu;
+    }
+    else
     {
-      return -1;
+        /* hSubMenu is always cleared 
+         * (not on Win9x/ME ) */
+        lpmii->hSubMenu = 0;
     }
-  *ItemInfo = HeapAlloc(GetProcessHeap(), 0, BufSize);
-  if (NULL == *ItemInfo)
+
+    if (lpmii->fMask & MIIM_CHECKMARKS)
     {
-      return -1;
+       lpmii->hbmpChecked = pItem->hbmpChecked;
+       lpmii->hbmpUnchecked = pItem->hbmpUnchecked;
     }
+    if (lpmii->fMask & MIIM_DATA)
+       lpmii->dwItemData = pItem->dwItemData;
 
-  return NtUserBuildMenuItemList(Menu, *ItemInfo, BufSize, 0);
+  return TRUE;
 }
 
-/***********************************************************************
- *           MenuCleanupAllRosMenuItemInfo
- *
- * Cleanup after use of MenuGetAllRosMenuItemInfo
- */
-static VOID FASTCALL
-MenuCleanupAllRosMenuItemInfo(PROSMENUITEMINFO ItemInfo)
-{
-  HeapFree(GetProcessHeap(), 0, ItemInfo);
-}
 
-/***********************************************************************
- *           MenuInitSysMenuPopup
- *
- * Grey the appropriate items in System menu.
- */
-void FASTCALL MenuInitSysMenuPopup(HMENU hmenu, DWORD style, DWORD clsStyle, LONG HitTest )
+//
+// User side Menu Class Proc.
+//
+LRESULT WINAPI
+PopupMenuWndProcW(HWND Wnd, UINT Message, WPARAM wParam, LPARAM lParam)
 {
-    BOOL gray;
-    UINT DefItem;
-    #if 0
-    MENUITEMINFOW mii;
-    #endif
-
-    gray = !(style & WS_THICKFRAME) || (style & (WS_MAXIMIZE | WS_MINIMIZE));
-    EnableMenuItem( hmenu, SC_SIZE, (gray ? MF_GRAYED : MF_ENABLED) );
-    gray = ((style & WS_MAXIMIZE) != 0);
-    EnableMenuItem( hmenu, SC_MOVE, (gray ? MF_GRAYED : MF_ENABLED) );
-    gray = !(style & WS_MINIMIZEBOX) || (style & WS_MINIMIZE);
-    EnableMenuItem( hmenu, SC_MINIMIZE, (gray ? MF_GRAYED : MF_ENABLED) );
-    gray = !(style & WS_MAXIMIZEBOX) || (style & WS_MAXIMIZE);
-    EnableMenuItem( hmenu, SC_MAXIMIZE, (gray ? MF_GRAYED : MF_ENABLED) );
-    gray = !(style & (WS_MAXIMIZE | WS_MINIMIZE));
-    EnableMenuItem( hmenu, SC_RESTORE, (gray ? MF_GRAYED : MF_ENABLED) );
-    gray = (clsStyle & CS_NOCLOSE) != 0;
-
-    /* The menu item must keep its state if it's disabled */
-    if(gray)
-        EnableMenuItem( hmenu, SC_CLOSE, MF_GRAYED);
-
-    /* Set default menu item */
-    if(style & WS_MINIMIZE) DefItem = SC_RESTORE;
-    else if(HitTest == HTCAPTION) DefItem = ((style & (WS_MAXIMIZE | WS_MINIMIZE)) ? SC_RESTORE : SC_MAXIMIZE);
-    else DefItem = SC_CLOSE;
-#if 0
-    mii.cbSize = sizeof(MENUITEMINFOW);
-    mii.fMask |= MIIM_STATE;
-    if((DefItem != SC_CLOSE) && GetMenuItemInfoW(hmenu, DefItem, FALSE, &mii) &&
-       (mii.fState & (MFS_GRAYED | MFS_DISABLED))) DefItem = SC_CLOSE;
-#endif
-    SetMenuDefaultItem(hmenu, DefItem, MF_BYCOMMAND);
-}
+  LRESULT lResult;
+  PWND pWnd;
 
-/******************************************************************************
- *
- *   UINT MenuGetStartOfNextColumn(
- *     PROSMENUINFO MenuInfo)
- *
- *****************************************************************************/
-static UINT MenuGetStartOfNextColumn(
-    PROSMENUINFO MenuInfo)
-{
-    PROSMENUITEMINFO MenuItems;
-    UINT i;
+  TRACE("PMWPW : hwnd=%x msg=0x%04x wp=0x%04lx lp=0x%08lx\n", Wnd, Message, wParam, lParam);
+
+  pWnd = ValidateHwnd(Wnd);
+  if (pWnd)
+  {
+     if (!pWnd->fnid)
+     {
+        if (Message != WM_NCCREATE)
+        {
+           return DefWindowProcW(Wnd, Message, wParam, lParam);
+        }
+     }
+     else
+     {
+        if (pWnd->fnid != FNID_MENU)
+        {
+           ERR("Wrong window class for Menu!\n");
+           return 0;
+        }
+     }
+  }
+
+  switch(Message)
+    {
+    case WM_DESTROY:
+    case WM_NCDESTROY:
+    case WM_NCCREATE:
+    case WM_CREATE:
+    case MM_SETMENUHANDLE:
+    case MM_GETMENUHANDLE:
+    case MN_SETHMENU:
+    case MN_GETHMENU:
+    case WM_PAINT:
+    case WM_PRINTCLIENT:
+      {
+        TRACE("Menu Class ProcW\n");
+        NtUserMessageCall( Wnd, Message, wParam, lParam, (ULONG_PTR)&lResult, FNID_MENU, FALSE);
+        return lResult;
+      }
+    case WM_MOUSEACTIVATE:  /* We don't want to be activated */
+      return MA_NOACTIVATE;
 
-    i = MenuInfo->FocusedItem;
-    if ( i == NO_SELECTED_ITEM )
-        return i;
+    case WM_ERASEBKGND:
+      return 1;
 
-    if (MenuGetAllRosMenuItemInfo(MenuInfo->Self, &MenuItems) <= 0)
-        return NO_SELECTED_ITEM;
+    case WM_SHOWWINDOW: // Not sure what this does....
+      if (0 != wParam)
+        {
+          if (0 == GetWindowLongPtrW(Wnd, 0))
+            {
+              OutputDebugStringA("no menu to display\n");
+            }
+        }
+      else
+        {
+          //SetWindowLongPtrW(Wnd, 0, 0);
+        }
+      break;
 
-    for (i++ ; i < MenuInfo->MenuItemCount; i++)
-        if (0 != (MenuItems[i].fType & (MF_MENUBREAK | MF_MENUBARBREAK)))
-            return i;
+    default:
+      return DefWindowProcW(Wnd, Message, wParam, lParam);
+    }
 
-    return NO_SELECTED_ITEM;
+  return 0;
 }
 
-/******************************************************************************
- *
- *   UINT MenuGetStartOfPrevColumn(
- *     PROSMENUINFO MenuInfo)
- *
- *****************************************************************************/
-
-static UINT FASTCALL MenuGetStartOfPrevColumn(
-    PROSMENUINFO MenuInfo)
+LRESULT WINAPI PopupMenuWndProcA(HWND Wnd, UINT Message, WPARAM wParam, LPARAM lParam)
 {
-    PROSMENUITEMINFO MenuItems;
-    UINT i;
-
-    if (!MenuInfo->FocusedItem || MenuInfo->FocusedItem == NO_SELECTED_ITEM)
-        return NO_SELECTED_ITEM;
+  PWND pWnd;
 
-    if (MenuGetAllRosMenuItemInfo(MenuInfo->Self, &MenuItems) <= 0)
-        return NO_SELECTED_ITEM;
+  pWnd = ValidateHwnd(Wnd);
+  if (pWnd && !pWnd->fnid && Message != WM_NCCREATE)
+  {
+     return DefWindowProcA(Wnd, Message, wParam, lParam);
+  }    
+  TRACE("YES! hwnd=%x msg=0x%04x wp=0x%04lx lp=0x%08lx\n", Wnd, Message, wParam, lParam);
 
-    /* Find the start of the column */
-    for (i = MenuInfo->FocusedItem;
-         0 != i && 0 == (MenuItems[i].fType & (MF_MENUBREAK | MF_MENUBARBREAK));
-         --i)
+  switch(Message)
     {
-        ; /* empty */
-    }
+    case WM_NCCREATE:
+    case WM_CREATE:
+    case WM_MOUSEACTIVATE:
+    case WM_PAINT:
+    case WM_PRINTCLIENT:
+    case WM_ERASEBKGND:
+    case WM_DESTROY:
+    case WM_NCDESTROY:
+    case WM_SHOWWINDOW:
+    case MM_SETMENUHANDLE:
+    case MM_GETMENUHANDLE:
+    case MN_SETHMENU:
+    case MN_GETHMENU: 
+      return PopupMenuWndProcW(Wnd, Message, wParam, lParam);
 
-    if (i == 0)
-    {
-        MenuCleanupAllRosMenuItemInfo(MenuItems);
-        return NO_SELECTED_ITEM;
+    default:
+      return DefWindowProcA(Wnd, Message, wParam, lParam);
     }
-
-    for (--i; 0 != i; --i)
-        if (MenuItems[i].fType & (MF_MENUBREAK | MF_MENUBARBREAK))
-            break;
-
-    MenuCleanupAllRosMenuItemInfo(MenuItems);
-    TRACE("ret %d.\n", i );
-
-    return i;
+  return 0;
 }
 
-/***********************************************************************
- *           MenuFindSubMenu
+/**********************************************************************
+ *         MENU_ParseResource
  *
- * Find a Sub menu. Return the position of the submenu, and modifies
- * *hmenu in case it is found in another sub-menu.
- * If the submenu cannot be found, NO_SELECTED_ITEM is returned.
+ * Parse a standard menu resource and add items to the menu.
+ * Return a pointer to the end of the resource.
+ *
+ * NOTE: flags is equivalent to the mtOption field
  */
-static UINT FASTCALL MenuFindSubMenu(HMENU *hmenu, HMENU hSubTarget )
+static LPCSTR MENU_ParseResource( LPCSTR res, HMENU hMenu)
 {
-    ROSMENUINFO menu;
-    UINT i;
-    ROSMENUITEMINFO item;
-    if (((*hmenu)==(HMENU)0xffff) ||
-          (!MenuGetRosMenuInfo(&menu, *hmenu)))
-        return NO_SELECTED_ITEM;
-
-    MenuInitRosMenuItemInfo(&item);
-    for (i = 0; i < menu.MenuItemCount; i++) {
-        if (! MenuGetRosMenuItemInfo(menu.Self, i, &item))
+    WORD flags, id = 0;
+    HMENU hSubMenu;
+    LPCWSTR str;
+    BOOL end = FALSE;
+
+    do
+    {
+        flags = GET_WORD(res);
+
+        /* remove MF_END flag before passing it to AppendMenu()! */
+        end = (flags & MF_END);
+        if(end) flags ^= MF_END;
+
+        res += sizeof(WORD);
+        if(!(flags & MF_POPUP))
         {
-            MenuCleanupRosMenuItemInfo(&item);
-            return NO_SELECTED_ITEM;
+            id = GET_WORD(res);
+            res += sizeof(WORD);
         }
-        if (!(item.fType & MF_POPUP)) continue;
-        if (item.hSubMenu == hSubTarget) {
-            MenuCleanupRosMenuItemInfo(&item);
-            return i;
+        str = (LPCWSTR)res;
+        res += (strlenW(str) + 1) * sizeof(WCHAR);
+
+        if (flags & MF_POPUP)
+        {
+            hSubMenu = CreatePopupMenu();
+            if(!hSubMenu) return NULL;
+            if(!(res = MENU_ParseResource(res, hSubMenu))) return NULL;
+            AppendMenuW(hMenu, flags, (UINT_PTR)hSubMenu, (LPCWSTR)str);
         }
-        else  {
-            HMENU hsubmenu = item.hSubMenu;
-            UINT pos = MenuFindSubMenu(&hsubmenu, hSubTarget );
-            if (pos != NO_SELECTED_ITEM) {
-                *hmenu = hsubmenu;
-                return pos;
-           }
+        else  /* Not a popup */
+        {
+            AppendMenuW(hMenu, flags, id, *(LPCWSTR)str ? (LPCWSTR)str : NULL);
         }
-    }
-    MenuCleanupRosMenuItemInfo(&item);
-    return NO_SELECTED_ITEM;
+    } while(!end);
+    return res;
 }
 
-/***********************************************************************
- *           MenuLoadBitmaps
+/**********************************************************************
+ *         MENUEX_ParseResource
  *
- * Load the arrow bitmap. We can't do this from MenuInit since user32
- * can also be used (and thus initialized) from text-mode.
+ * Parse an extended menu resource and add items to the menu.
+ * Return a pointer to the end of the resource.
  */
-static void FASTCALL
-MenuLoadBitmaps(VOID)
+static LPCSTR MENUEX_ParseResource(LPCSTR res, HMENU hMenu)
 {
-  /* Load system buttons bitmaps */
-  if (NULL == BmpSysMenu)
+    WORD resinfo;
+    do
     {
-      BmpSysMenu = LoadBitmapW(0, MAKEINTRESOURCEW(OBM_CLOSE));
-    }
-}
+        MENUITEMINFOW mii;
 
-/***********************************************************************
- *           MenuDrawPopupGlyph
- *
- * Draws popup magic glyphs (can be found in system menu).
- */
-static void FASTCALL
-MenuDrawPopupGlyph(HDC dc, LPRECT r, INT_PTR popupMagic, BOOL inactive, BOOL hilite)
-{
-  LOGFONTW lf;
-  HFONT hFont, hOldFont;
-  COLORREF clrsave;
-  INT bkmode;
-  TCHAR symbol;
-  switch (popupMagic)
-  {
-  case (INT_PTR) HBMMENU_POPUP_RESTORE:
-    symbol = '2';
-    break;
-  case (INT_PTR) HBMMENU_POPUP_MINIMIZE:
-    symbol = '0';
-    break;
-  case (INT_PTR) HBMMENU_POPUP_MAXIMIZE:
-    symbol = '1';
-    break;
-  case (INT_PTR) HBMMENU_POPUP_CLOSE:
-    symbol = 'r';
-    break;
-  default:
-    ERR("Invalid popup magic bitmap %d\n", (int)popupMagic);
-    return;
-  }
-  ZeroMemory(&lf, sizeof(LOGFONTW));
-  InflateRect(r, -2, -2);
-  lf.lfHeight = r->bottom - r->top;
-  lf.lfWidth = 0;
-  lf.lfWeight = FW_NORMAL;
-  lf.lfCharSet = DEFAULT_CHARSET;
-  lstrcpy(lf.lfFaceName, TEXT("Marlett"));
-  hFont = CreateFontIndirect(&lf);
-  /* save font and text color */
-  hOldFont = SelectObject(dc, hFont);
-  clrsave = GetTextColor(dc);
-  bkmode = GetBkMode(dc);
-  /* set color and drawing mode */
-  SetBkMode(dc, TRANSPARENT);
-  if (inactive)
-  {
-    /* draw shadow */
-    if (!hilite)
-    {
-      SetTextColor(dc, GetSysColor(COLOR_HIGHLIGHTTEXT));
-      TextOut(dc, r->left + 1, r->top + 1, &symbol, 1);
-    }
-  }
-  SetTextColor(dc, GetSysColor(inactive ? COLOR_GRAYTEXT : (hilite ? COLOR_HIGHLIGHTTEXT : COLOR_MENUTEXT)));
-  /* draw selected symbol */
-  TextOut(dc, r->left, r->top, &symbol, 1);
-  /* restore previous settings */
-  SetTextColor(dc, clrsave);
-  SelectObject(dc, hOldFont);
-  SetBkMode(dc, bkmode);
-  DeleteObject(hFont);
-}
-
-/***********************************************************************
- *           MenuFindItemByKey
- *
- * Find the menu item selected by a key press.
- * Return item id, -1 if none, -2 if we should close the menu.
- */
-static UINT FASTCALL MenuFindItemByKey(HWND WndOwner, PROSMENUINFO MenuInfo,
-                  WCHAR Key, BOOL ForceMenuChar)
-{
-  ROSMENUINFO SysMenuInfo;
-  PROSMENUITEMINFO Items, ItemInfo;
-  LRESULT MenuChar;
-  UINT i;
-
-  TRACE("\tlooking for '%c' (0x%02x) in [%p]\n", (char) Key, Key, MenuInfo);
-
-  if (NULL == MenuInfo || ! IsMenu(MenuInfo->Self))
-  {
-      if (MenuGetRosMenuInfo(&SysMenuInfo, GetSystemMenu(WndOwner, FALSE)))
-      {
-          MenuInfo = &SysMenuInfo;
-      }
-      else
-      {
-          MenuInfo = NULL;
-      }
-  }
-
-  if (NULL != MenuInfo)
-  {
-      if (MenuGetAllRosMenuItemInfo(MenuInfo->Self, &Items) <= 0)
-      {
-          return -1;
-      }
-      if ( !ForceMenuChar )
-      {
-          ItemInfo = Items;
-          for (i = 0; i < MenuInfo->MenuItemCount; i++, ItemInfo++)
-          {
-              if ((ItemInfo->lpstr) && NULL != ItemInfo->dwTypeData)
-              {
-                  WCHAR *p = (WCHAR *) ItemInfo->dwTypeData - 2;
-                  do
-                  {
-                      p = strchrW (p + 2, '&');
-                  }
-                  while (p != NULL && p [1] == '&');
-                  if (p && (toupperW(p[1]) == toupperW(Key))) return i;
-              }
-          }
-      }
-
-      MenuChar = SendMessageW(WndOwner, WM_MENUCHAR,
-                              MAKEWPARAM(Key, MenuInfo->Flags), (LPARAM) MenuInfo->Self);
-      if (HIWORD(MenuChar) == MNC_EXECUTE) return LOWORD(MenuChar);
-      if (HIWORD(MenuChar) == MNC_CLOSE) return (UINT)(-2);
-    }
-  return (UINT)(-1);
-}
-
-/***********************************************************************
- *           MenuGetBitmapItemSize
- *
- * Get the size of a bitmap item.
- */
-static void FASTCALL MenuGetBitmapItemSize(PROSMENUITEMINFO lpitem, SIZE *size, HWND WndOwner)
-{
-    BITMAP bm;
-    HBITMAP bmp = lpitem->hbmpItem;
-
-    size->cx = size->cy = 0;
-
-    /* check if there is a magic menu item associated with this item */
-    if (IS_MAGIC_BITMAP(bmp))
-    {
-      switch((INT_PTR) bmp)
-        {
-            case (INT_PTR)HBMMENU_CALLBACK:
-            {
-                MEASUREITEMSTRUCT measItem;
-                measItem.CtlType = ODT_MENU;
-                measItem.CtlID = 0;
-                measItem.itemID = lpitem->wID;
-                measItem.itemWidth = lpitem->Rect.right - lpitem->Rect.left;
-                measItem.itemHeight = lpitem->Rect.bottom - lpitem->Rect.top;
-                measItem.itemData = lpitem->dwItemData;
-                SendMessageW( WndOwner, WM_MEASUREITEM, lpitem->wID, (LPARAM)&measItem);
-                size->cx = measItem.itemWidth;
-                size->cy = measItem.itemHeight;
-                return;
-            }
-            break;
-
-          case (INT_PTR) HBMMENU_SYSTEM:
-            if (0 != lpitem->dwItemData)
-              {
-                bmp = (HBITMAP) lpitem->dwItemData;
-                break;
-              }
-            /* fall through */
-          case (INT_PTR) HBMMENU_MBAR_RESTORE:
-          case (INT_PTR) HBMMENU_MBAR_MINIMIZE:
-          case (INT_PTR) HBMMENU_MBAR_CLOSE:
-          case (INT_PTR) HBMMENU_MBAR_MINIMIZE_D:
-          case (INT_PTR) HBMMENU_MBAR_CLOSE_D:
-          case (INT_PTR) HBMMENU_POPUP_CLOSE:
-          case (INT_PTR) HBMMENU_POPUP_RESTORE:
-          case (INT_PTR) HBMMENU_POPUP_MAXIMIZE:
-          case (INT_PTR) HBMMENU_POPUP_MINIMIZE:
-            /* FIXME: Why we need to subtract these magic values? */
-            /* to make them smaller than the menu bar? */
-            size->cx = GetSystemMetrics(SM_CXSIZE) - 2;
-            size->cy = GetSystemMetrics(SM_CYSIZE) - 4;
-            return;
-        }
-    }
-
-    if (GetObjectW(bmp, sizeof(BITMAP), &bm))
-    {
-        size->cx = bm.bmWidth;
-        size->cy = bm.bmHeight;
-    }
-}
-
-/***********************************************************************
- *           MenuDrawBitmapItem
- *
- * Draw a bitmap item.
- */
-static void FASTCALL MenuDrawBitmapItem(HDC hdc, PROSMENUITEMINFO lpitem, const RECT *rect,
-                    HMENU hmenu, HWND WndOwner, UINT odaction, BOOL MenuBar)
-{
-    BITMAP bm;
-    DWORD rop;
-    HDC hdcMem;
-    HBITMAP bmp;
-    int w = rect->right - rect->left;
-    int h = rect->bottom - rect->top;
-    int bmp_xoffset = 0;
-    int left, top;
-    HBITMAP hbmToDraw = lpitem->hbmpItem;
-    bmp = hbmToDraw;
-
-    /* Check if there is a magic menu item associated with this item */
-    if (IS_MAGIC_BITMAP(hbmToDraw))
-    {
-        UINT flags = 0;
-        RECT r;
-
-      r = *rect;
-      switch ((INT_PTR)hbmToDraw)
-        {
-        case (INT_PTR)HBMMENU_SYSTEM:
-            if (lpitem->dwTypeData)
-            {
-                bmp = (HBITMAP)lpitem->dwTypeData;
-                if (!GetObjectW( bmp, sizeof(bm), &bm )) return;
-            }
-            else
-            {
-                if (!BmpSysMenu) BmpSysMenu = LoadBitmapW(0, MAKEINTRESOURCEW(OBM_CLOSE));
-                bmp = BmpSysMenu;
-                if (! GetObjectW(bmp, sizeof(bm), &bm)) return;
-                /* only use right half of the bitmap */
-                bmp_xoffset = bm.bmWidth / 2;
-                bm.bmWidth -= bmp_xoffset;
-            }
-            goto got_bitmap;
-        case (INT_PTR)HBMMENU_MBAR_RESTORE:
-            flags = DFCS_CAPTIONRESTORE;
-            break;
-        case (INT_PTR)HBMMENU_MBAR_MINIMIZE:
-            r.right += 1;
-            flags = DFCS_CAPTIONMIN;
-            break;
-        case (INT_PTR)HBMMENU_MBAR_MINIMIZE_D:
-            r.right += 1;
-            flags = DFCS_CAPTIONMIN | DFCS_INACTIVE;
-            break;
-        case (INT_PTR)HBMMENU_MBAR_CLOSE:
-            flags = DFCS_CAPTIONCLOSE;
-            break;
-        case (INT_PTR)HBMMENU_MBAR_CLOSE_D:
-            flags = DFCS_CAPTIONCLOSE | DFCS_INACTIVE;
-            break;
-        case (INT_PTR)HBMMENU_CALLBACK:
-            {
-                DRAWITEMSTRUCT drawItem;
-                POINT origorg;
-                drawItem.CtlType = ODT_MENU;
-                drawItem.CtlID = 0;
-                drawItem.itemID = lpitem->wID;
-                drawItem.itemAction = odaction;
-                drawItem.itemState = (lpitem->fState & MF_CHECKED)?ODS_CHECKED:0;
-                drawItem.itemState |= (lpitem->fState & MF_DEFAULT)?ODS_DEFAULT:0;
-                drawItem.itemState |= (lpitem->fState & MF_DISABLED)?ODS_DISABLED:0;
-                drawItem.itemState |= (lpitem->fState & MF_GRAYED)?ODS_GRAYED|ODS_DISABLED:0;
-                drawItem.itemState |= (lpitem->fState & MF_HILITE)?ODS_SELECTED:0;
-                drawItem.hwndItem = (HWND)hmenu;
-                drawItem.hDC = hdc;
-                drawItem.rcItem = *rect;
-                drawItem.itemData = lpitem->dwItemData;
-                /* some applications make this assumption on the DC's origin */
-                SetViewportOrgEx( hdc, lpitem->Rect.left, lpitem->Rect.top, &origorg);
-                OffsetRect( &drawItem.rcItem, - lpitem->Rect.left, - lpitem->Rect.top);
-                SendMessageW( WndOwner, WM_DRAWITEM, 0, (LPARAM)&drawItem);
-                SetViewportOrgEx( hdc, origorg.x, origorg.y, NULL);
-                return;
-            }
-            break;
-
-          case (INT_PTR) HBMMENU_POPUP_CLOSE:
-          case (INT_PTR) HBMMENU_POPUP_RESTORE:
-          case (INT_PTR) HBMMENU_POPUP_MAXIMIZE:
-          case (INT_PTR) HBMMENU_POPUP_MINIMIZE:
-            MenuDrawPopupGlyph(hdc, &r, (INT_PTR)hbmToDraw, lpitem->fState & MF_GRAYED, lpitem->fState & MF_HILITE);
-            return;
-        }
-      InflateRect(&r, -1, -1);
-      if (0 != (lpitem->fState & MF_HILITE))
-      {
-          flags |= DFCS_PUSHED;
-      }
-      DrawFrameControl(hdc, &r, DFC_CAPTION, flags);
-      return;
-    }
-
-    if (!bmp || !GetObjectW( bmp, sizeof(bm), &bm )) return;
-
- got_bitmap:
-    hdcMem = CreateCompatibleDC( hdc );
-    SelectObject( hdcMem, bmp );
-
-    /* handle fontsize > bitmap_height */
-    top = (h>bm.bmHeight) ? rect->top+(h-bm.bmHeight)/2 : rect->top;
-    left=rect->left;
-    rop=((lpitem->fState & MF_HILITE) && !IS_MAGIC_BITMAP(hbmToDraw)) ? NOTSRCCOPY : SRCCOPY;
-    if ((lpitem->fState & MF_HILITE) && lpitem->hbmpItem)
-        SetBkColor(hdc, GetSysColor(COLOR_HIGHLIGHT));
-    BitBlt( hdc, left, top, w, h, hdcMem, bmp_xoffset, 0, rop );
-    DeleteDC( hdcMem );
-}
-
-/***********************************************************************
- *           MenuCalcItemSize
- *
- * Calculate the size of the menu item and store it in lpitem->rect.
- */
-static void FASTCALL MenuCalcItemSize( HDC hdc, PROSMENUITEMINFO lpitem, PROSMENUINFO MenuInfo, HWND hwndOwner,
-                 INT orgX, INT orgY, BOOL menuBar)
-{
-    WCHAR *p;
-    UINT check_bitmap_width = GetSystemMetrics( SM_CXMENUCHECK );
-    INT itemheight = 0;
-
-    TRACE("dc=%x owner=%x (%d,%d)\n", hdc, hwndOwner, orgX, orgY);
-
-    MenuCharSize.cx = GdiGetCharDimensions( hdc, NULL, &MenuCharSize.cy );
-
-    SetRect( &lpitem->Rect, orgX, orgY, orgX, orgY );
-
-    if (lpitem->fType & MF_OWNERDRAW)
-    {
-        MEASUREITEMSTRUCT mis;
-        mis.CtlType    = ODT_MENU;
-        mis.CtlID      = 0;
-        mis.itemID     = lpitem->wID;
-        mis.itemData   = lpitem->dwItemData;
-        mis.itemHeight = HIWORD( GetDialogBaseUnits());
-        mis.itemWidth  = 0;
-        SendMessageW( hwndOwner, WM_MEASUREITEM, 0, (LPARAM)&mis );
-        /* Tests reveal that Windows ( Win95 thru WinXP) adds twice the average
-         * width of a menufont character to the width of an owner-drawn menu. 
-         */
-        lpitem->Rect.right += mis.itemWidth + 2 * MenuCharSize.cx;
-
-        if (menuBar) {
-            /* under at least win95 you seem to be given a standard
-               height for the menu and the height value is ignored */
-            lpitem->Rect.bottom += GetSystemMetrics(SM_CYMENUSIZE);
-        } else
-            lpitem->Rect.bottom += mis.itemHeight;
-
-        TRACE("id=%04lx size=%dx%d\n",
-                lpitem->wID, mis.itemWidth, mis.itemHeight);
-        return;
-    }
-
-    if (lpitem->fType & MF_SEPARATOR)
-    {
-        lpitem->Rect.bottom += SEPARATOR_HEIGHT;
-        if( !menuBar)
-            lpitem->Rect.right += check_bitmap_width +  MenuCharSize.cx;
-        return;
-    }
-
-    lpitem->dxTab = 0;
-
-    if (lpitem->hbmpItem)
-    {
-        SIZE size;
-
-        if (!menuBar) {
-            MenuGetBitmapItemSize(lpitem, &size, hwndOwner );
-            /* Keep the size of the bitmap in callback mode to be able
-             * to draw it correctly */
-            lpitem->Rect.right = lpitem->Rect.left + size.cx;
-            if (MenuInfo->maxBmpSize.cx < abs(size.cx) +  MENU_ITEM_HBMP_SPACE ||
-                MenuInfo->maxBmpSize.cy < abs(size.cy))
-            {
-                MenuInfo->maxBmpSize.cx = abs(size.cx) + MENU_ITEM_HBMP_SPACE;
-                MenuInfo->maxBmpSize.cy = abs(size.cy);
-            }
-            MenuSetRosMenuInfo(MenuInfo);
-            itemheight = size.cy + 2;
-
-            if( !(MenuInfo->dwStyle & MNS_NOCHECK))
-                lpitem->Rect.right += 2 * check_bitmap_width;
-            lpitem->Rect.right += 4 + MenuCharSize.cx;
-            lpitem->dxTab = lpitem->Rect.right;
-            lpitem->Rect.right += check_bitmap_width;
-        } else /* hbmpItem & MenuBar */ {
-            MenuGetBitmapItemSize(lpitem, &size, hwndOwner );
-            lpitem->Rect.right  += size.cx;
-            if( lpitem->lpstr) lpitem->Rect.right  += 2;
-            itemheight = size.cy;
-
-            /* Special case: Minimize button doesn't have a space behind it. */
-            if (lpitem->hbmpItem == (HBITMAP)HBMMENU_MBAR_MINIMIZE ||
-                lpitem->hbmpItem == (HBITMAP)HBMMENU_MBAR_MINIMIZE_D)
-            lpitem->Rect.right -= 1;
-        }
-    }
-    else if (!menuBar) {
-        if( !(MenuInfo->dwStyle & MNS_NOCHECK))
-             lpitem->Rect.right += check_bitmap_width;
-        lpitem->Rect.right += 4 + MenuCharSize.cx;
-        lpitem->dxTab = lpitem->Rect.right;
-        lpitem->Rect.right += check_bitmap_width;
-    }
-
-    /* it must be a text item - unless it's the system menu */
-    if (!(lpitem->fType & MF_SYSMENU) && lpitem->lpstr) {
-        HFONT hfontOld = NULL;
-        RECT rc = lpitem->Rect;
-        LONG txtheight, txtwidth;
-
-        if ( lpitem->fState & MFS_DEFAULT ) {
-            hfontOld = SelectObject( hdc, hMenuFontBold );
-        }
-        if (menuBar) {
-            txtheight = DrawTextW( hdc, lpitem->dwTypeData, -1, &rc,
-                    DT_SINGLELINE|DT_CALCRECT); 
-            lpitem->Rect.right  += rc.right - rc.left;
-            itemheight = max( max( itemheight, txtheight),
-                    GetSystemMetrics( SM_CYMENU) - 1);
-            lpitem->Rect.right +=  2 * MenuCharSize.cx;
-        } else {
-            if ((p = strchrW( lpitem->dwTypeData, '\t' )) != NULL) {
-                RECT tmprc = rc;
-                LONG tmpheight;
-                int n = (int)( p - lpitem->dwTypeData);
-                /* Item contains a tab (only meaningful in popup menus) */
-                /* get text size before the tab */
-                txtheight = DrawTextW( hdc, lpitem->dwTypeData, n, &rc,
-                        DT_SINGLELINE|DT_CALCRECT);
-                txtwidth = rc.right - rc.left;
-                p += 1; /* advance past the Tab */
-                /* get text size after the tab */
-                tmpheight = DrawTextW( hdc, p, -1, &tmprc,
-                        DT_SINGLELINE|DT_CALCRECT);
-                lpitem->dxTab += txtwidth;
-                txtheight = max( txtheight, tmpheight);
-                txtwidth += MenuCharSize.cx + /* space for the tab */
-                    tmprc.right - tmprc.left; /* space for the short cut */
-            } else {
-                txtheight = DrawTextW( hdc, lpitem->dwTypeData, -1, &rc,
-                        DT_SINGLELINE|DT_CALCRECT);
-                txtwidth = rc.right - rc.left;
-                lpitem->dxTab += txtwidth;
-            }
-            lpitem->Rect.right  += 2 + txtwidth;
-            itemheight = max( itemheight,
-                                   max( txtheight + 2, MenuCharSize.cy + 4));
-        }
-        if (hfontOld) SelectObject (hdc, hfontOld);
-    } else if( menuBar) {
-        itemheight = max( itemheight, GetSystemMetrics(SM_CYMENU)-1);
-    }
-    lpitem->Rect.bottom += itemheight;
-    TRACE("(%ld,%ld)-(%ld,%ld)\n", lpitem->Rect.left, lpitem->Rect.top, lpitem->Rect.right, lpitem->Rect.bottom);
-}
-
-/***********************************************************************
- *           MenuPopupMenuCalcSize
- *
- * Calculate the size of a popup menu.
- */
-static void FASTCALL MenuPopupMenuCalcSize(PROSMENUINFO MenuInfo, HWND WndOwner)
-{
-    ROSMENUITEMINFO lpitem;
-    HDC hdc;
-    int start, i;
-    int orgX, orgY, maxX, maxTab, maxTabWidth;
-
-    MenuInfo->Width = MenuInfo->Height = 0;
-    if (MenuInfo->MenuItemCount == 0)
-    {
-        MenuSetRosMenuInfo(MenuInfo);
-        return;
-    }
-
-    hdc = GetDC(NULL);
-    SelectObject( hdc, hMenuFont );
-
-    start = 0;
-    maxX = 2 + 1;
-
-    MenuInfo->maxBmpSize.cx = 0;
-    MenuInfo->maxBmpSize.cy = 0;
-
-    MenuInitRosMenuItemInfo(&lpitem);
-    while (start < MenuInfo->MenuItemCount)
-    {
-      orgX = maxX;
-      orgY = 2;
-
-      maxTab = maxTabWidth = 0;
-
-      /* Parse items until column break or end of menu */
-      for (i = start; i < MenuInfo->MenuItemCount; i++)
-      {
-          if (! MenuGetRosMenuItemInfo(MenuInfo->Self, i, &lpitem))
-          {
-              MenuCleanupRosMenuItemInfo(&lpitem);
-              MenuSetRosMenuInfo(MenuInfo);
-              return;
-          }
-          if (i != start &&
-               (lpitem.fType & (MF_MENUBREAK | MF_MENUBARBREAK))) break;
-
-          MenuCalcItemSize(hdc, &lpitem, MenuInfo, WndOwner, orgX, orgY, FALSE);
-          if (! MenuSetRosMenuItemInfo(MenuInfo->Self, i, &lpitem))
-          {
-              MenuCleanupRosMenuItemInfo(&lpitem);
-              MenuSetRosMenuInfo(MenuInfo);
-              return;
-          }
-// Not sure here,, The patch from wine removes this.
-//        if ((lpitem.fType & MF_MENUBARBREAK) != 0)
-//        {
-//              OrgX++;
-//        }
-          maxX = max(maxX, lpitem.Rect.right);
-          orgY = lpitem.Rect.bottom;
-          if ((lpitem.lpstr) && lpitem.dxTab )
-             {
-              maxTab = max( maxTab, lpitem.dxTab );
-              maxTabWidth = max(maxTabWidth, lpitem.Rect.right - lpitem.dxTab);
-          }
-        }
-
-        /* Finish the column (set all items to the largest width found) */
-        maxX = max( maxX, maxTab + maxTabWidth );
-        while (start < i)
-        {
-            if (MenuGetRosMenuItemInfo(MenuInfo->Self, start, &lpitem))
-            {
-                lpitem.Rect.right = maxX;
-                if ((lpitem.lpstr) && 0 != lpitem.dxTab)
-                {
-                    lpitem.dxTab = maxTab;
-                }
-                MenuSetRosMenuItemInfo(MenuInfo->Self, start, &lpitem);
-            }
-            start++;
-           }
-        MenuInfo->Height = max(MenuInfo->Height, orgY);
-    }
-
-    MenuInfo->Width  = maxX;
-
-    /* space for 3d border */
-    MenuInfo->Height += 2;
-    MenuInfo->Width += 2;
-
-    MenuCleanupRosMenuItemInfo(&lpitem);
-    MenuSetRosMenuInfo(MenuInfo);
-    ReleaseDC( 0, hdc );
-}
-
-/***********************************************************************
- *           MenuMenuBarCalcSize
- *
- * FIXME: Word 6 implements its own MDI and its own 'close window' bitmap
- * height is off by 1 pixel which causes lengthy window relocations when
- * active document window is maximized/restored.
- *
- * Calculate the size of the menu bar.
- */
-static void FASTCALL MenuMenuBarCalcSize( HDC hdc, LPRECT lprect,
-                                          PROSMENUINFO MenuInfo, HWND hwndOwner )
-{
-    ROSMENUITEMINFO ItemInfo;
-    int start, i, orgX, orgY, maxY, helpPos;
-
-    if ((lprect == NULL) || (MenuInfo == NULL)) return;
-    if (MenuInfo->MenuItemCount == 0) return;
-    TRACE("left=%ld top=%ld right=%ld bottom=%ld\n", lprect->left, lprect->top, lprect->right, lprect->bottom);
-    MenuInfo->Width = lprect->right - lprect->left;
-    MenuInfo->Height = 0;
-    maxY = lprect->top + 1;
-    start = 0;
-    helpPos = -1;
-
-    MenuInfo->maxBmpSize.cx = 0;
-    MenuInfo->maxBmpSize.cy = 0;
-
-    MenuInitRosMenuItemInfo(&ItemInfo);
-    while (start < MenuInfo->MenuItemCount)
-    {
-        if (! MenuGetRosMenuItemInfo(MenuInfo->Self, start, &ItemInfo))
-        {
-            MenuCleanupRosMenuItemInfo(&ItemInfo);
-            return;
-        }
-        orgX = lprect->left;
-        orgY = maxY;
-
-        /* Parse items until line break or end of menu */
-        for (i = start; i < MenuInfo->MenuItemCount; i++)
-           {
-            if ((helpPos == -1) && (ItemInfo.fType & MF_RIGHTJUSTIFY)) helpPos = i;
-            if ((i != start) &&
-               (ItemInfo.fType & (MF_MENUBREAK | MF_MENUBARBREAK))) break;
-
-            TRACE("calling MENU_CalcItemSize org=(%d, %d)\n", orgX, orgY);
-            MenuCalcItemSize(hdc, &ItemInfo, MenuInfo, hwndOwner, orgX, orgY, TRUE);
-            if (! MenuSetRosMenuItemInfo(MenuInfo->Self, i, &ItemInfo))
-            {
-                MenuCleanupRosMenuItemInfo(&ItemInfo);
-                return;
-            }
-
-            if (ItemInfo.Rect.right > lprect->right)
-            {
-                if (i != start) break;
-                else ItemInfo.Rect.right = lprect->right;
-            }
-            maxY = max( maxY, ItemInfo.Rect.bottom );
-            orgX = ItemInfo.Rect.right;
-            if (i + 1 < MenuInfo->MenuItemCount)
-            {
-                if (! MenuGetRosMenuItemInfo(MenuInfo->Self, i + 1, &ItemInfo))
-                {
-                    MenuCleanupRosMenuItemInfo(&ItemInfo);
-                    return;
-                }
-            }
-       }
-
-/* FIXME: Is this really needed? */ /*NO! it is not needed, why make the
-HBMMENU_MBAR_CLOSE, MINIMIZE & RESTORE, look the same size as the menu bar! */
-#if 0
-        /* Finish the line (set all items to the largest height found) */
-        while (start < i)
-        {
-            if (MenuGetRosMenuItemInfo(MenuInfo->Self, start, &ItemInfo))
-            {
-                ItemInfo.Rect.bottom = maxY;
-                MenuSetRosMenuItemInfo(MenuInfo->Self, start, &ItemInfo);
-            }
-            start++;
-        }
-#else
-        start = i; /* This works! */
-#endif
-    }
-
-    lprect->bottom = maxY;
-    MenuInfo->Height = lprect->bottom - lprect->top;
-    MenuSetRosMenuInfo(MenuInfo);
-
-    if (helpPos != -1)
-    {
-      /* Flush right all items between the MF_RIGHTJUSTIFY and */
-      /* the last item (if several lines, only move the last line) */
-      if (! MenuGetRosMenuItemInfo(MenuInfo->Self, MenuInfo->MenuItemCount - 1, &ItemInfo))
-      {
-          MenuCleanupRosMenuItemInfo(&ItemInfo);
-          return;
-      }
-      orgY = ItemInfo.Rect.top;
-      orgX = lprect->right;
-      for (i = MenuInfo->MenuItemCount - 1; helpPos <= i; i--)
-        {
-          if (i < helpPos)
-          {
-              break;                           /* done */
-          }
-          if (ItemInfo.Rect.top != orgY)
-          {
-              break;                           /* Other line */
-          }
-          if (orgX <= ItemInfo.Rect.right)
-          {
-              break;                           /* Too far right already */
-          }
-          ItemInfo.Rect.left += orgX - ItemInfo.Rect.right;
-          ItemInfo.Rect.right = orgX;
-          orgX = ItemInfo.Rect.left;
-          MenuSetRosMenuItemInfo(MenuInfo->Self, i, &ItemInfo);
-          if (helpPos + 1 <= i &&
-              ! MenuGetRosMenuItemInfo(MenuInfo->Self, i - 1, &ItemInfo))
-            {
-                MenuCleanupRosMenuItemInfo(&ItemInfo);
-                return;
-            }
-        }
-    }
-
-    MenuCleanupRosMenuItemInfo(&ItemInfo);
-}
-
-/***********************************************************************
- *           MenuDrawMenuItem
- *
- * Draw a single menu item.
- */
-static void FASTCALL MenuDrawMenuItem(HWND hWnd, PROSMENUINFO MenuInfo, HWND WndOwner, HDC hdc,
-                 PROSMENUITEMINFO lpitem, UINT Height, BOOL menuBar, UINT odaction)
-{
-    RECT rect;
-    PWCHAR Text;
-    BOOL flat_menu = FALSE;
-    int bkgnd;
-    PWND Wnd = ValidateHwnd(hWnd);
-
-    if (!Wnd)
-      return;
-
-    if (lpitem->fType & MF_SYSMENU)
-    {
-        if ( (Wnd->style & WS_MINIMIZE))
-        {
-          UserGetInsideRectNC(Wnd, &rect);
-          UserDrawSysMenuButton(hWnd, hdc, &rect,
-                                lpitem->fState & (MF_HILITE | MF_MOUSESELECT));
-           }
-        return;
-    }
-
-    SystemParametersInfoW (SPI_GETFLATMENU, 0, &flat_menu, 0);
-    bkgnd = (menuBar && flat_menu) ? COLOR_MENUBAR : COLOR_MENU;
-  
-    /* Setup colors */
-
-    if (lpitem->fState & MF_HILITE)
-    {
-        if(menuBar && !flat_menu) {
-            SetTextColor(hdc, GetSysColor(COLOR_MENUTEXT));
-            SetBkColor(hdc, GetSysColor(COLOR_MENU));
-        } else {
-            if (lpitem->fState & MF_GRAYED)
-                SetTextColor(hdc, GetSysColor(COLOR_GRAYTEXT));
-            else
-                SetTextColor(hdc, GetSysColor(COLOR_HIGHLIGHTTEXT));
-            SetBkColor(hdc, GetSysColor(COLOR_HIGHLIGHT));
-        }
-    }
-    else
-    {
-        if (lpitem->fState & MF_GRAYED)
-            SetTextColor( hdc, GetSysColor( COLOR_GRAYTEXT ) );
-        else
-            SetTextColor( hdc, GetSysColor( COLOR_MENUTEXT ) );
-        SetBkColor( hdc, GetSysColor( bkgnd ) );
-    }
-
-    rect = lpitem->Rect;
-
-    if (lpitem->fType & MF_OWNERDRAW)
-    {
-        /*
-        ** Experimentation under Windows reveals that an owner-drawn
-        ** menu is given the rectangle which includes the space it requested
-        ** in its response to WM_MEASUREITEM _plus_ width for a checkmark
-        ** and a popup-menu arrow.  This is the value of lpitem->rect.
-        ** Windows will leave all drawing to the application except for
-        ** the popup-menu arrow.  Windows always draws that itself, after
-        ** the menu owner has finished drawing.
-        */
-        DRAWITEMSTRUCT dis;
-
-        dis.CtlType   = ODT_MENU;
-        dis.CtlID     = 0;
-        dis.itemID    = lpitem->wID;
-        dis.itemData  = (DWORD)lpitem->dwItemData;
-        dis.itemState = 0;
-        if (lpitem->fState & MF_CHECKED) dis.itemState |= ODS_CHECKED;
-        if (lpitem->fState & MF_GRAYED)  dis.itemState |= ODS_GRAYED | ODS_DISABLED;
-        if (lpitem->fState & MF_HILITE)  dis.itemState |= ODS_SELECTED;
-        dis.itemAction = odaction; /* ODA_DRAWENTIRE | ODA_SELECT | ODA_FOCUS; */
-        dis.hwndItem   = (HWND) MenuInfo->Self;
-        dis.hDC        = hdc;
-        dis.rcItem     = rect;
-        TRACE("Ownerdraw: owner=%p itemID=%d, itemState=%d, itemAction=%d, "
-               "hwndItem=%p, hdc=%p, rcItem={%ld,%ld,%ld,%ld}\n", hWnd,
-               dis.itemID, dis.itemState, dis.itemAction, dis.hwndItem,
-               dis.hDC, dis.rcItem.left, dis.rcItem.top, dis.rcItem.right,
-               dis.rcItem.bottom);
-        SendMessageW(WndOwner, WM_DRAWITEM, 0, (LPARAM) &dis);
-        /* Draw the popup-menu arrow */
-        if (lpitem->fType & MF_POPUP)
-        {
-            RECT rectTemp;
-            CopyRect(&rectTemp, &rect);
-            rectTemp.left = rectTemp.right - GetSystemMetrics(SM_CXMENUCHECK);
-            DrawFrameControl(hdc, &rectTemp, DFC_MENU, DFCS_MENUARROW);
-        }
-        return;
-    }
-
-    if (menuBar && (lpitem->fType & MF_SEPARATOR)) return;
-
-    if (lpitem->fState & MF_HILITE)
-    {
-        if (flat_menu)
-        {
-            InflateRect (&rect, -1, -1);
-            FillRect(hdc, &rect, GetSysColorBrush(COLOR_MENUHILIGHT));
-            InflateRect (&rect, 1, 1);
-            FrameRect(hdc, &rect, GetSysColorBrush(COLOR_HIGHLIGHT));
-        }
-        else
-        {
-            if(menuBar)
-                DrawEdge(hdc, &rect, BDR_SUNKENOUTER, BF_RECT);
-            else
-                FillRect(hdc, &rect, GetSysColorBrush(COLOR_HIGHLIGHT));
-        }
-    }
-    else
-        FillRect( hdc, &rect, GetSysColorBrush(bkgnd) );
-
-    SetBkMode( hdc, TRANSPARENT );
-
-    /* vertical separator */
-    if (!menuBar && (lpitem->fType & MF_MENUBARBREAK))
-    {
-        HPEN oldPen;
-        RECT rc = rect;
-
-        rc.left -= 3;
-        rc.top = 3;
-        rc.bottom = Height - 3;
-        if (flat_menu)
-        {
-            oldPen = SelectObject( hdc, GetStockObject(DC_PEN) );
-            SetDCPenColor(hdc, GetSysColor(COLOR_BTNSHADOW));
-            MoveToEx( hdc, rc.left, rc.top, NULL );
-            LineTo( hdc, rc.left, rc.bottom );
-            SelectObject( hdc, oldPen );
-        }
-        else
-            DrawEdge (hdc, &rc, EDGE_ETCHED, BF_LEFT);
-    }
-
-    /* horizontal separator */
-    if (lpitem->fType & MF_SEPARATOR)
-    {
-        HPEN oldPen;
-        RECT rc = rect;
-
-        rc.left++;
-        rc.right--;
-        rc.top += SEPARATOR_HEIGHT / 2;
-        if (flat_menu)
-        {
-            oldPen = SelectObject( hdc, GetStockObject(DC_PEN) );
-            SetDCPenColor( hdc, GetSysColor(COLOR_BTNSHADOW));
-            MoveToEx( hdc, rc.left, rc.top, NULL );
-            LineTo( hdc, rc.right, rc.top );
-            SelectObject( hdc, oldPen );
-        }
-        else
-            DrawEdge (hdc, &rc, EDGE_ETCHED, BF_TOP);
-        return;
-    }
-
-#if 0
-    /* helper lines for debugging */
-    /* This is a very good test tool when hacking menus! (JT) 07/16/2006 */
-    FrameRect(hdc, &rect, GetStockObject(BLACK_BRUSH));
-    SelectObject(hdc, GetStockObject(DC_PEN));
-    SetDCPenColor(hdc, GetSysColor(COLOR_WINDOWFRAME));
-    MoveToEx(hdc, rect.left, (rect.top + rect.bottom) / 2, NULL);
-    LineTo(hdc, rect.right, (rect.top + rect.bottom) / 2);
-#endif
-
-    if (!menuBar)
-    {
-        HBITMAP bm;
-        INT y = rect.top + rect.bottom;
-        RECT rc = rect;
-        int checked = FALSE;
-        UINT check_bitmap_width = GetSystemMetrics( SM_CXMENUCHECK );
-        UINT check_bitmap_height = GetSystemMetrics( SM_CYMENUCHECK );
-        /* Draw the check mark
-         *
-         * FIXME:
-         * Custom checkmark bitmaps are monochrome but not always 1bpp.
-         */
-        if( !(MenuInfo->dwStyle & MNS_NOCHECK)) {
-            bm = (lpitem->fState & MF_CHECKED) ? lpitem->hbmpChecked : 
-                lpitem->hbmpUnchecked;
-            if (bm)  /* we have a custom bitmap */
-            {
-                HDC hdcMem = CreateCompatibleDC( hdc );
-
-                SelectObject( hdcMem, bm );
-                BitBlt( hdc, rc.left, (y - check_bitmap_height) / 2,
-                        check_bitmap_width, check_bitmap_height,
-                        hdcMem, 0, 0, SRCCOPY );
-                DeleteDC( hdcMem );
-                checked = TRUE;
-            }
-            else if (lpitem->fState & MF_CHECKED) /* standard bitmaps */
-            {
-                RECT r;
-                CopyRect(&r, &rect);
-                r.right = r.left + GetSystemMetrics(SM_CXMENUCHECK);
-                DrawFrameControl( hdc, &r, DFC_MENU,
-                                 (lpitem->fType & MFT_RADIOCHECK) ?
-                                 DFCS_MENUBULLET : DFCS_MENUCHECK);
-                checked = TRUE;
-            }
-        }
-        if ( lpitem->hbmpItem )
-        {
-            RECT bmpRect;
-            CopyRect(&bmpRect, &rect);
-            if (!(MenuInfo->dwStyle & MNS_CHECKORBMP) && !(MenuInfo->dwStyle & MNS_NOCHECK))
-                bmpRect.left += check_bitmap_width + 2;
-            if (!(checked && (MenuInfo->dwStyle & MNS_CHECKORBMP)))
-            {
-                bmpRect.right = bmpRect.left + MenuInfo->maxBmpSize.cx;
-                MenuDrawBitmapItem(hdc, lpitem, &bmpRect, MenuInfo->Self, WndOwner, odaction, menuBar);
-            }
-        }
-        /* Draw the popup-menu arrow */
-        if (lpitem->fType & MF_POPUP)
-        {
-            RECT rectTemp;
-            CopyRect(&rectTemp, &rect);
-            rectTemp.left = rectTemp.right - GetSystemMetrics(SM_CXMENUCHECK);
-            DrawFrameControl(hdc, &rectTemp, DFC_MENU, DFCS_MENUARROW);
-        }
-        rect.left += 4;
-        if( !(MenuInfo->dwStyle & MNS_NOCHECK))
-            rect.left += check_bitmap_width;
-        rect.right -= check_bitmap_width;
-    }
-    else if( lpitem->hbmpItem)
-    { /* Draw the bitmap */
-        MenuDrawBitmapItem(hdc, lpitem, &rect, MenuInfo->Self, WndOwner, odaction, menuBar);
-    }
-
-    /* process text if present */
-    if (lpitem->lpstr)
-    {
-        register int i = 0;
-        HFONT hfontOld = 0;
-
-        UINT uFormat = menuBar ? DT_CENTER | DT_VCENTER | DT_SINGLELINE
-                       : DT_LEFT | DT_VCENTER | DT_SINGLELINE;
-
-        if(MenuInfo->dwStyle & MNS_CHECKORBMP)
-             rect.left += max(0, MenuInfo->maxBmpSize.cx - GetSystemMetrics(SM_CXMENUCHECK));
-        else
-             rect.left += MenuInfo->maxBmpSize.cx;
-
-        if ( lpitem->fState & MFS_DEFAULT )
-        {
-            hfontOld = SelectObject(hdc, hMenuFontBold);
-        }
-
-        if (menuBar) {
-            rect.left += MENU_BAR_ITEMS_SPACE / 2;
-            rect.right -= MENU_BAR_ITEMS_SPACE / 2;
-        }
-
-        Text = (PWCHAR) lpitem->dwTypeData;
-        if(Text)
-        {
-            for (i = 0; L'\0' != Text[i]; i++)
-                if (Text[i] == L'\t' || Text[i] == L'\b')
-                    break;
-        }
-
-        if(lpitem->fState & MF_GRAYED)
-        {
-            if (!(lpitem->fState & MF_HILITE) )
-            {
-                ++rect.left; ++rect.top; ++rect.right; ++rect.bottom;
-                SetTextColor(hdc, RGB(0xff, 0xff, 0xff));
-                DrawTextW( hdc, Text, i, &rect, uFormat );
-                --rect.left; --rect.top; --rect.right; --rect.bottom;
-               }
-            SetTextColor(hdc, RGB(0x80, 0x80, 0x80));
-        }
-
-        DrawTextW( hdc, Text, i, &rect, uFormat);
-
-        /* paint the shortcut text */
-        if (!menuBar && L'\0' != Text[i])  /* There's a tab or flush-right char */
-        {
-            if (L'\t' == Text[i])
-            {
-                rect.left = lpitem->dxTab;
-                uFormat = DT_LEFT | DT_VCENTER | DT_SINGLELINE;
-            }
-            else
-            {
-                rect.right = lpitem->dxTab;
-                uFormat = DT_RIGHT | DT_VCENTER | DT_SINGLELINE;
-            }
-
-            if (lpitem->fState & MF_GRAYED)
-            {
-                if (!(lpitem->fState & MF_HILITE) )
-                {
-                    ++rect.left; ++rect.top; ++rect.right; ++rect.bottom;
-                    SetTextColor(hdc, RGB(0xff, 0xff, 0xff));
-                    DrawTextW( hdc, Text + i + 1, -1, &rect, uFormat);
-                    --rect.left; --rect.top; --rect.right; --rect.bottom;
-                }
-                SetTextColor(hdc, RGB(0x80, 0x80, 0x80));
-               }
-          DrawTextW( hdc, Text + i + 1, -1, &rect, uFormat );
-        }
-
-        if (hfontOld)
-          SelectObject (hdc, hfontOld);
-    }
-}
-
-/***********************************************************************
- *           MenuDrawPopupMenu
- *
- * Paint a popup menu.
- */
-static void FASTCALL MenuDrawPopupMenu(HWND hwnd, HDC hdc, HMENU hmenu )
-{
-    HBRUSH hPrevBrush = 0;
-    RECT rect;
-
-    TRACE("wnd=%p dc=%p menu=%p\n", hwnd, hdc, hmenu);
-
-    GetClientRect( hwnd, &rect );
-
-    if((hPrevBrush = SelectObject( hdc, GetSysColorBrush(COLOR_MENU) ))
-        && (SelectObject( hdc, hMenuFont)))
-    {
-        HPEN hPrevPen;
-
-        Rectangle( hdc, rect.left, rect.top, rect.right, rect.bottom );
-
-        hPrevPen = SelectObject( hdc, GetStockObject( NULL_PEN ) );
-        if ( hPrevPen )
-        {
-            BOOL flat_menu = FALSE;
-            ROSMENUINFO MenuInfo;
-            ROSMENUITEMINFO ItemInfo;
-
-            SystemParametersInfoW (SPI_GETFLATMENU, 0, &flat_menu, 0);
-            if (flat_menu)
-               FrameRect(hdc, &rect, GetSysColorBrush(COLOR_BTNSHADOW));
-            else
-               DrawEdge (hdc, &rect, EDGE_RAISED, BF_RECT);
-
-            /* draw menu items */
-            if (MenuGetRosMenuInfo(&MenuInfo, hmenu) && MenuInfo.MenuItemCount)
-            {
-                UINT u;
-                
-                               MenuInitRosMenuItemInfo(&ItemInfo);
-
-                for (u = 0; u < MenuInfo.MenuItemCount; u++)
-                {
-                    if (MenuGetRosMenuItemInfo(MenuInfo.Self, u, &ItemInfo))
-                    {
-                        MenuDrawMenuItem(hwnd, &MenuInfo, MenuInfo.WndOwner, hdc, &ItemInfo,
-                        MenuInfo.Height, FALSE, ODA_DRAWENTIRE);
-                    }
-                }
-
-                MenuCleanupRosMenuItemInfo(&ItemInfo);
-            }
-         } else
-         {
-             SelectObject( hdc, hPrevBrush );
-         }
-    }
-}
-
-/***********************************************************************
- *           MenuDrawMenuBar
- *
- * Paint a menu bar. Returns the height of the menu bar.
- * called from [windows/nonclient.c]
- */
-UINT MenuDrawMenuBar( HDC hDC, LPRECT lprect, HWND hwnd,
-                         BOOL suppress_draw)
-{
-    ROSMENUINFO lppop;
-    HFONT hfontOld = 0;
-    HMENU hMenu = GetMenu(hwnd);
-
-    if (! MenuGetRosMenuInfo(&lppop, hMenu) || lprect == NULL)
-    {
-        return GetSystemMetrics(SM_CYMENU);
-    }
-
-    if (suppress_draw)
-    {
-        hfontOld = SelectObject(hDC, hMenuFont);
-
-        MenuMenuBarCalcSize(hDC, lprect, &lppop, hwnd);
-
-        lprect->bottom = lprect->top + lppop.Height;
-
-        if (hfontOld) SelectObject( hDC, hfontOld);
-        return lppop.Height;
-    }
-       else
-        return DrawMenuBarTemp(hwnd, hDC, lprect, hMenu, NULL);
-}
-
-/***********************************************************************
- *           MenuShowPopup
- *
- * Display a popup menu.
- */
-static BOOL FASTCALL MenuShowPopup(HWND hwndOwner, HMENU hmenu, UINT id, UINT flags,
-                              INT x, INT y, INT xanchor, INT yanchor )
-{
-    ROSMENUINFO MenuInfo;
-    ROSMENUITEMINFO ItemInfo;
-    UINT width, height;
-    POINT pt;
-    HMONITOR monitor;
-    MONITORINFO info;
-
-    TRACE("owner=%p hmenu=%p id=0x%04x x=0x%04x y=0x%04x xa=0x%04x ya=0x%04x\n",
-          hwndOwner, hmenu, id, x, y, xanchor, yanchor);
-
-    if (! MenuGetRosMenuInfo(&MenuInfo, hmenu)) return FALSE;
-    if (MenuInfo.FocusedItem != NO_SELECTED_ITEM)
-    {
-        MenuInitRosMenuItemInfo(&ItemInfo);
-        if (MenuGetRosMenuItemInfo(MenuInfo.Self, MenuInfo.FocusedItem, &ItemInfo))
-        {
-            ItemInfo.fMask |= MIIM_STATE;
-            ItemInfo.fState &= ~(MF_HILITE|MF_MOUSESELECT);
-            MenuSetRosMenuItemInfo(MenuInfo.Self, MenuInfo.FocusedItem, &ItemInfo);
-        }
-        MenuCleanupRosMenuItemInfo(&ItemInfo);
-        MenuInfo.FocusedItem = NO_SELECTED_ITEM;
-    }
-
-    /* store the owner for DrawItem */
-    if (!IsWindow(hwndOwner))
-    {
-       SetLastError( ERROR_INVALID_WINDOW_HANDLE );
-       return FALSE;
-    }
-    MenuInfo.WndOwner = hwndOwner;
-    MenuSetRosMenuInfo(&MenuInfo);
-
-    MenuPopupMenuCalcSize(&MenuInfo, hwndOwner);
-
-    /* adjust popup menu pos so that it fits within the desktop */
-
-    width = MenuInfo.Width + GetSystemMetrics(SM_CXBORDER);
-    height = MenuInfo.Height + GetSystemMetrics(SM_CYBORDER);
-
-    /* FIXME: should use item rect */
-    pt.x = x;
-    pt.y = y;
-    monitor = MonitorFromPoint( pt, MONITOR_DEFAULTTONEAREST );
-    info.cbSize = sizeof(info);
-    GetMonitorInfoW( monitor, &info );
-
-    if( flags & TPM_RIGHTALIGN ) x -= width;
-    if( flags & TPM_CENTERALIGN ) x -= width / 2;
-
-    if( flags & TPM_BOTTOMALIGN ) y -= height;
-    if( flags & TPM_VCENTERALIGN ) y -= height / 2;
-
-    if( x + width > info.rcMonitor.right)
-    {
-        if( xanchor && x >= width - xanchor )
-            x -= width - xanchor;
-
-        if( x + width > info.rcMonitor.right)
-            x = info.rcMonitor.right - width;
-    }
-    if( x < info.rcMonitor.left ) x = info.rcMonitor.left;
-
-    if( y + height > info.rcMonitor.bottom)
-    {
-        if( yanchor && y >= height + yanchor )
-            y -= height + yanchor;
-
-        if( y + height > info.rcMonitor.bottom)
-            y = info.rcMonitor.bottom - height;
-    }
-    if( y < info.rcMonitor.top ) y = info.rcMonitor.top;
-
-    /* NOTE: In Windows, top menu popup is not owned. */
-    MenuInfo.Wnd = CreateWindowExW( 0, WC_MENU, NULL,
-                                  WS_POPUP, x, y, width, height,
-                                  hwndOwner, 0, (HINSTANCE) GetWindowLongPtrW(hwndOwner, GWLP_HINSTANCE),
-                                 (LPVOID) MenuInfo.Self);
-    if ( !MenuInfo.Wnd || ! MenuSetRosMenuInfo(&MenuInfo)) return FALSE;
-    if (!top_popup) {
-        top_popup = MenuInfo.Wnd;
-        top_popup_hmenu = hmenu;
-    }
-
-    IntNotifyWinEvent(EVENT_SYSTEM_MENUPOPUPSTART, MenuInfo.Wnd, OBJID_CLIENT, CHILDID_SELF, 0);
-
-    /* Display the window */
-
-    SetWindowPos( MenuInfo.Wnd, HWND_TOPMOST, 0, 0, 0, 0,
-                  SWP_SHOWWINDOW | SWP_NOSIZE | SWP_NOMOVE | SWP_NOACTIVATE);
-    UpdateWindow( MenuInfo.Wnd );
-    return TRUE;
-}
-
-
-/***********************************************************************
- *           MenuSelectItem
- */
-static void FASTCALL MenuSelectItem(HWND hwndOwner, PROSMENUINFO hmenu, UINT wIndex,
-                                    BOOL sendMenuSelect, HMENU topmenu)
-{
-    ROSMENUITEMINFO ItemInfo;
-    ROSMENUINFO TopMenuInfo;
-    HDC hdc;
-
-    TRACE("owner=%p menu=%p index=0x%04x select=0x%04x\n", hwndOwner, hmenu, wIndex, sendMenuSelect);
-
-     if (!hmenu || !hmenu->MenuItemCount || !hmenu->Wnd) return;
-    if (hmenu->FocusedItem == wIndex) return;
-    if (hmenu->Flags & MF_POPUP) hdc = GetDC(hmenu->Wnd);
-    else hdc = GetDCEx(hmenu->Wnd, 0, DCX_CACHE | DCX_WINDOW);
-    if (!top_popup) {
-        top_popup = hmenu->Wnd;
-        top_popup_hmenu = hmenu->Self;
-    }
-
-    SelectObject( hdc, hMenuFont );
-
-    MenuInitRosMenuItemInfo(&ItemInfo);
-
-     /* Clear previous highlighted item */
-    if (hmenu->FocusedItem != NO_SELECTED_ITEM)
-    {
-        if (MenuGetRosMenuItemInfo(hmenu->Self, hmenu->FocusedItem, &ItemInfo))
-        {
-            ItemInfo.fMask |= MIIM_STATE;
-            ItemInfo.fState &= ~(MF_HILITE|MF_MOUSESELECT);
-            MenuSetRosMenuItemInfo(hmenu->Self, hmenu->FocusedItem, &ItemInfo);
-        }
-        MenuDrawMenuItem(hmenu->Wnd, hmenu, hwndOwner, hdc, &ItemInfo,
-                       hmenu->Height, ! (hmenu->Flags & MF_POPUP),
-                       ODA_SELECT);
-    }
-
-    /* Highlight new item (if any) */
-    hmenu->FocusedItem = wIndex;
-    MenuSetRosMenuInfo(hmenu);
-    if (hmenu->FocusedItem != NO_SELECTED_ITEM)
-    {
-        if (MenuGetRosMenuItemInfo(hmenu->Self, hmenu->FocusedItem, &ItemInfo))
-        {
-            if (!(ItemInfo.fType & MF_SEPARATOR))
-            {
-                ItemInfo.fMask |= MIIM_STATE;
-                ItemInfo.fState |= MF_HILITE;
-                MenuSetRosMenuItemInfo(hmenu->Self, hmenu->FocusedItem, &ItemInfo);
-                MenuDrawMenuItem(hmenu->Wnd, hmenu, hwndOwner, hdc,
-                               &ItemInfo, hmenu->Height, ! (hmenu->Flags & MF_POPUP),
-                               ODA_SELECT);
-            }
-            if (sendMenuSelect)
-            {
-                SendMessageW(hwndOwner, WM_MENUSELECT,
-                           MAKELONG(ItemInfo.fType & MF_POPUP ? wIndex : ItemInfo.wID,
-                                    ItemInfo.fType | ItemInfo.fState | MF_MOUSESELECT |
-                                    (hmenu->Flags & MF_SYSMENU)), (LPARAM) hmenu->Self);
-            }
-        }
-    }
-    else if (sendMenuSelect) {
-        if(topmenu) {
-            int pos;
-            pos = MenuFindSubMenu(&topmenu, hmenu->Self);
-            if (pos != NO_SELECTED_ITEM)
-            {
-                if (MenuGetRosMenuInfo(&TopMenuInfo, topmenu)
-                    && MenuGetRosMenuItemInfo(topmenu, pos, &ItemInfo))
-                {
-                    SendMessageW(hwndOwner, WM_MENUSELECT,
-                               MAKELONG(Pos, ItemInfo.fType | ItemInfo.fState
-                                             | MF_MOUSESELECT
-                                             | (TopMenuInfo.Flags & MF_SYSMENU)),
-                               (LPARAM) topmenu);
-                }
-            }
-        }
-    }
-    MenuCleanupRosMenuItemInfo(&ItemInfo);
-    ReleaseDC(hmenu->Wnd, hdc);
-}
-
-/***********************************************************************
- *           MenuMoveSelection
- *
- * Moves currently selected item according to the Offset parameter.
- * If there is no selection then it should select the last item if
- * Offset is ITEM_PREV or the first item if Offset is ITEM_NEXT.
- */
-static void FASTCALL
-MenuMoveSelection(HWND WndOwner, PROSMENUINFO MenuInfo, INT Offset)
-{
-  INT i;
-  ROSMENUITEMINFO ItemInfo;
-  INT OrigPos;
-
-  TRACE("hwnd=%x menu=%x off=0x%04x\n", WndOwner, MenuInfo, Offset);
-
-  /* Prevent looping */
-  if (0 == MenuInfo->MenuItemCount || 0 == Offset)
-    return;
-  else if (Offset < -1)
-    Offset = -1;
-  else if (Offset > 1)
-    Offset = 1;
-
-  MenuInitRosMenuItemInfo(&ItemInfo);
-
-  OrigPos = MenuInfo->FocusedItem;
-  if (OrigPos == NO_SELECTED_ITEM) /* NO_SELECTED_ITEM is not -1 ! */
-    {
-       OrigPos = 0;
-       i = -1;
-    }
-  else
-    {
-      i = MenuInfo->FocusedItem;
-    }
-
-  do
-    {
-      /* Step */
-      i += Offset;
-      /* Clip and wrap around */
-      if (i < 0)
-        {
-          i = MenuInfo->MenuItemCount - 1;
-        }
-      else if (i >= MenuInfo->MenuItemCount)
-        {
-          i = 0;
-        }
-      /* If this is a good candidate; */
-      if (MenuGetRosMenuItemInfo(MenuInfo->Self, i, &ItemInfo) &&
-          0 == (ItemInfo.fType & MF_SEPARATOR))
-        {
-          MenuSelectItem(WndOwner, MenuInfo, i, TRUE, NULL);
-          MenuCleanupRosMenuItemInfo(&ItemInfo);
-          return;
-        }
-    } while (i != OrigPos);
-
-  /* Not found */
-  MenuCleanupRosMenuItemInfo(&ItemInfo);
-}
-
-//
-// This breaks some test results. Should handle A2U if called!
-//
-LRESULT WINAPI PopupMenuWndProcA(HWND Wnd, UINT Message, WPARAM wParam, LPARAM lParam)
-{
-#ifdef __REACTOS__
-  PWND pWnd;
-
-  pWnd = ValidateHwnd(Wnd);
-  if (pWnd)
-  {
-     if (!pWnd->fnid)
-     {
-        NtUserSetWindowFNID(Wnd, FNID_MENU);
-     }
-     else
-     {
-        if (pWnd->fnid != FNID_MENU)
-        {
-           ERR("Wrong window class for Menu!\n");
-           return 0;
-        }
-     }
-  }
-#endif    
-
-  TRACE("YES! hwnd=%x msg=0x%04x wp=0x%04lx lp=0x%08lx\n", Wnd, Message, wParam, lParam);
-
-  switch(Message)
-    {
-    case WM_CREATE:
-      {
-        CREATESTRUCTA *cs = (CREATESTRUCTA *) lParam;
-        SetWindowLongPtrA(Wnd, 0, (LONG_PTR)cs->lpCreateParams);
-        return 0;
-      }
-
-    case WM_MOUSEACTIVATE:  /* We don't want to be activated */
-      return MA_NOACTIVATE;
-
-    case WM_PAINT:
-      {
-        PAINTSTRUCT ps;
-        BeginPaint(Wnd, &ps);
-        MenuDrawPopupMenu(Wnd, ps.hdc, (HMENU)GetWindowLongPtrA(Wnd, 0));
-        EndPaint(Wnd, &ps);
-        return 0;
-      }
-
-    case WM_PRINTCLIENT:
-      {
-        MenuDrawPopupMenu( Wnd, (HDC)wParam,
-                                (HMENU)GetWindowLongPtrW( Wnd, 0 ) );
-        return 0;
-      }
-
-    case WM_ERASEBKGND:
-      return 1;
-
-    case WM_DESTROY:
-      /* zero out global pointer in case resident popup window was destroyed. */
-      if (Wnd == top_popup)
-        {
-          top_popup = NULL;
-          top_popup_hmenu = NULL;
-        }
-      break;
-
-#ifdef __REACTOS__
-    case WM_NCDESTROY:
-      NtUserSetWindowFNID(Wnd, FNID_DESTROY);
-      break;
-#endif
-
-    case WM_SHOWWINDOW:
-      if (0 != wParam)
-        {
-          if (0 == GetWindowLongPtrA(Wnd, 0))
-            {
-              OutputDebugStringA("no menu to display\n");
-            }
-        }
-      else
-        {
-          SetWindowLongPtrA(Wnd, 0, 0);
-        }
-      break;
-
-    case MM_SETMENUHANDLE:
-      SetWindowLongPtrA(Wnd, 0, wParam);
-      break;
-
-    case MM_GETMENUHANDLE:
-    case MN_GETHMENU: 
-      return GetWindowLongPtrA(Wnd, 0);
-
-    default:
-      return DefWindowProcA(Wnd, Message, wParam, lParam);
-    }
-  return 0;
-}
-
-LRESULT WINAPI
-PopupMenuWndProcW(HWND Wnd, UINT Message, WPARAM wParam, LPARAM lParam)
-{
-#ifdef __REACTOS__ // Do this now, remove after Server side is fixed.
-  PWND pWnd;
-
-  pWnd = ValidateHwnd(Wnd);
-  if (pWnd)
-  {
-     if (!pWnd->fnid)
-     {
-        if (Message != WM_NCCREATE)
-        {
-           return DefWindowProcW(Wnd, Message, wParam, lParam);
-        }
-        NtUserSetWindowFNID(Wnd, FNID_MENU);
-     }
-     else
-     {
-        if (pWnd->fnid != FNID_MENU)
-        {
-           ERR("Wrong window class for Menu!\n");
-           return 0;
-        }
-     }
-  }
-#endif    
-
-  TRACE("hwnd=%x msg=0x%04x wp=0x%04lx lp=0x%08lx\n", Wnd, Message, wParam, lParam);
-
-  switch(Message)
-    {
-    case WM_CREATE:
-      {
-        CREATESTRUCTW *cs = (CREATESTRUCTW *) lParam;
-        SetWindowLongPtrW(Wnd, 0, (LONG_PTR)cs->lpCreateParams);
-        return 0;
-      }
-
-    case WM_MOUSEACTIVATE:  /* We don't want to be activated */
-      return MA_NOACTIVATE;
-
-    case WM_PAINT:
-      {
-        PAINTSTRUCT ps;
-        BeginPaint(Wnd, &ps);
-        MenuDrawPopupMenu(Wnd, ps.hdc, (HMENU)GetWindowLongPtrW(Wnd, 0));
-        EndPaint(Wnd, &ps);
-        return 0;
-      }
-
-    case WM_PRINTCLIENT:
-      {
-         MenuDrawPopupMenu( Wnd, (HDC)wParam,
-                                (HMENU)GetWindowLongPtrW( Wnd, 0 ) );
-         return 0;
-      }
-
-    case WM_ERASEBKGND:
-      return 1;
-
-    case WM_DESTROY:
-      /* zero out global pointer in case resident popup window was destroyed. */
-      if (Wnd == top_popup)
-        {
-          top_popup = NULL;
-          top_popup_hmenu = NULL;
-        }
-      break;
-
-    case WM_SHOWWINDOW:
-      if (0 != wParam)
-        {
-          if (0 == GetWindowLongPtrW(Wnd, 0))
-            {
-              OutputDebugStringA("no menu to display\n");
-            }
-        }
-      else
-        {
-          SetWindowLongPtrW(Wnd, 0, 0);
-        }
-      break;
-
-    case MM_SETMENUHANDLE:
-      SetWindowLongPtrW(Wnd, 0, wParam);
-      break;
-
-    case MM_GETMENUHANDLE:
-    case MN_GETHMENU:
-      return GetWindowLongPtrW(Wnd, 0);
-
-    default:
-      return DefWindowProcW(Wnd, Message, wParam, lParam);
-    }
-
-  return 0;
-}
-
-/**********************************************************************
- *         MENU_ParseResource
- *
- * Parse a standard menu resource and add items to the menu.
- * Return a pointer to the end of the resource.
- *
- * NOTE: flags is equivalent to the mtOption field
- */
-static LPCSTR MENU_ParseResource( LPCSTR res, HMENU hMenu, BOOL unicode )
-{
-    WORD flags, id = 0;
-    HMENU hSubMenu;
-    LPCSTR str;
-    BOOL end = FALSE;
-
-    do
-    {
-        flags = GET_WORD(res);
-
-        /* remove MF_END flag before passing it to AppendMenu()! */
-        end = (flags & MF_END);
-        if(end) flags ^= MF_END;
-
-        res += sizeof(WORD);
-        if(!(flags & MF_POPUP))
-        {
-            id = GET_WORD(res);
-            res += sizeof(WORD);
-        }
-        str = res;
-        if(!unicode)
-            res += strlen(str) + 1;
-        else
-            res += (strlenW((LPCWSTR)str) + 1) * sizeof(WCHAR);
-        if (flags & MF_POPUP)
-        {
-            hSubMenu = CreatePopupMenu();
-            if(!hSubMenu) return NULL;
-            if(!(res = MENU_ParseResource(res, hSubMenu, unicode)))
-                return NULL;
-            if(!unicode)
-                AppendMenuA(hMenu, flags, (UINT)hSubMenu, str);
-            else
-                AppendMenuW(hMenu, flags, (UINT)hSubMenu, (LPCWSTR)str);
-        }
-        else  /* Not a popup */
-        {
-            if(!unicode)
-            {
-                if (*str == 0)
-                    flags = MF_SEPARATOR;
-            }
-            else
-            {
-                if (*(LPCWSTR)str == 0)
-                    flags = MF_SEPARATOR;
-            }
-
-            if (flags & MF_SEPARATOR)
-            {
-                if (!(flags & (MF_GRAYED | MF_DISABLED)))
-                    flags |= MF_GRAYED | MF_DISABLED;
-            }
-
-            if(!unicode)
-                AppendMenuA(hMenu, flags, id, *str ? str : NULL);
-            else
-                AppendMenuW(hMenu, flags, id,
-                    *(LPCWSTR)str ? (LPCWSTR)str : NULL);
-        }
-    } while(!end);
-    return res;
-}
-
-
-/**********************************************************************
- *         MENUEX_ParseResource
- *
- * Parse an extended menu resource and add items to the menu.
- * Return a pointer to the end of the resource.
- */
-static LPCSTR MENUEX_ParseResource(LPCSTR res, HMENU hMenu)
-{
-    WORD resinfo;
-    MENUITEMINFOW mii;
-
-    do
-    {
-        mii.cbSize = sizeof(mii);
-        mii.fMask = MIIM_STATE | MIIM_ID | MIIM_TYPE;
-        mii.fType = GET_DWORD(res);
-        res += sizeof(DWORD);
-        mii.fState = GET_DWORD(res);
-        res += sizeof(DWORD);
-        mii.wID = GET_DWORD(res);
-        res += sizeof(DWORD);
-        resinfo = GET_WORD(res);
-        res += sizeof(WORD);
-        /* Align the text on a word boundary.  */
-        res += (~((UINT_PTR)res - 1)) & 1;
-        mii.dwTypeData = (LPWSTR)res;
-        mii.cch = strlenW(mii.dwTypeData);
-        res += (1 + strlenW(mii.dwTypeData)) * sizeof(WCHAR);
-        /* Align the following fields on a dword boundary.  */
-        res += (~((UINT_PTR)res - 1)) & 3;
-
-        TRACE("Menu item: [%08x,%08x,%04x,%04x,%S]\n",
-              mii.fType, mii.fState, mii.wID, resinfo, mii.dwTypeData);
-
-               if (resinfo & 1) /* Pop-up? */
-               {
-            /* DWORD helpid = GET_DWORD(res); FIXME: use this. */
-            res += sizeof(DWORD);
-            mii.hSubMenu = CreatePopupMenu();
-            if (!mii.hSubMenu)
-            {
-                ERR("CreatePopupMenu failed\n");
-                return NULL;
-            }
-
-            if (!(res = MENUEX_ParseResource(res, mii.hSubMenu)))
-            {
-                ERR("MENUEX_ParseResource failed\n");
-                DestroyMenu(mii.hSubMenu);
-                return NULL;
-            }
-            mii.fMask |= MIIM_SUBMENU;
-            mii.fType |= MF_POPUP;
-            mii.wID = (UINT)mii.hSubMenu;
-        }
-        else if (!mii.dwTypeData[0])
-            mii.fType |= MF_SEPARATOR;
-
-        if (!InsertMenuItemW(hMenu, -1, MF_BYPOSITION, &mii))
-            ERR("InsertMenuItemW failed\n");
-    } while (!(resinfo & MF_END));
-    return res;
-}
-
-NTSTATUS WINAPI
-User32LoadSysMenuTemplateForKernel(PVOID Arguments, ULONG ArgumentLength)
-{
-  HMENU hmenu = LoadMenuW(User32Instance, L"SYSMENU");
-  LRESULT Result = (LRESULT)hmenu;
-  MENUINFO menuinfo = {0};
-  MENUITEMINFOW info = {0};
-
-  // removing space for checkboxes from menu
-  menuinfo.cbSize = sizeof(menuinfo);
-  menuinfo.fMask = MIM_STYLE;
-  GetMenuInfo(hmenu, &menuinfo);
-  menuinfo.dwStyle |= MNS_NOCHECK;
-  SetMenuInfo(hmenu, &menuinfo);
-
-  // adding bitmaps to menu items
-  info.cbSize = sizeof(info);
-  info.fMask |= MIIM_BITMAP;
-  info.hbmpItem = HBMMENU_POPUP_MINIMIZE;
-  SetMenuItemInfoW(hmenu, SC_MINIMIZE, FALSE, &info);
-  info.hbmpItem = HBMMENU_POPUP_RESTORE;
-  SetMenuItemInfoW(hmenu, SC_RESTORE, FALSE, &info);
-  info.hbmpItem = HBMMENU_POPUP_MAXIMIZE;
-  SetMenuItemInfoW(hmenu, SC_MAXIMIZE, FALSE, &info);
-  info.hbmpItem = HBMMENU_POPUP_CLOSE;
-  SetMenuItemInfoW(hmenu, SC_CLOSE, FALSE, &info);
-
-  return(ZwCallbackReturn(&Result, sizeof(LRESULT), STATUS_SUCCESS));
-}
-
-
-BOOL
-MenuInit(VOID)
-{
-  NONCLIENTMETRICSW ncm;
-
-  /* get the menu font */
-  if(!hMenuFont || !hMenuFontBold)
-  {
-    ncm.cbSize = sizeof(ncm);
-    if(!SystemParametersInfoW(SPI_GETNONCLIENTMETRICS, sizeof(ncm), &ncm, 0))
-    {
-      ERR("MenuInit(): SystemParametersInfoW(SPI_GETNONCLIENTMETRICS) failed!\n");
-      return FALSE;
-    }
-
-    hMenuFont = CreateFontIndirectW(&ncm.lfMenuFont);
-    if(hMenuFont == NULL)
-    {
-      ERR("MenuInit(): CreateFontIndirectW(hMenuFont) failed!\n");
-      return FALSE;
-    }
-
-    ncm.lfMenuFont.lfWeight = max(ncm.lfMenuFont.lfWeight + 300, 1000);
-    hMenuFontBold = CreateFontIndirectW(&ncm.lfMenuFont);
-    if(hMenuFontBold == NULL)
-    {
-      ERR("MenuInit(): CreateFontIndirectW(hMenuFontBold) failed!\n");
-      DeleteObject(hMenuFont);
-      hMenuFont = NULL;
-      return FALSE;
-    }
-  }
-
-  return TRUE;
-}
-
-VOID
-MenuCleanup(VOID)
-{
-  if (hMenuFont)
-  {
-    DeleteObject(hMenuFont);
-    hMenuFont = NULL;
-  }
-
-  if (hMenuFontBold)
-  {
-    DeleteObject(hMenuFontBold);
-    hMenuFontBold = NULL;
-  }
-}
-
-/***********************************************************************
- *           DrawMenuBarTemp   (USER32.@)
- *
- * UNDOCUMENTED !!
- *
- * called by W98SE desk.cpl Control Panel Applet
- *
- * Not 100% sure about the param names, but close.
- *
- * @implemented
- */
-DWORD WINAPI
-DrawMenuBarTemp(HWND Wnd, HDC DC, LPRECT Rect, HMENU Menu, HFONT Font)
-{
-  ROSMENUINFO MenuInfo;
-  ROSMENUITEMINFO ItemInfo;
-  UINT i;
-  HFONT FontOld = NULL;
-  BOOL flat_menu = FALSE;
-
-  SystemParametersInfoW (SPI_GETFLATMENU, 0, &flat_menu, 0);
-
-  if (NULL == Menu)
-    {
-      Menu = GetMenu(Wnd);
-    }
-
-  if (NULL == Font)
-    {
-      Font = hMenuFont;
-    }
-
-  if (NULL == Rect || ! MenuGetRosMenuInfo(&MenuInfo, Menu))
-    {
-      return GetSystemMetrics(SM_CYMENU);
-    }
-
-  TRACE("(%x, %x, %p, %x, %x)\n", Wnd, DC, Rect, Menu, Font);
-
-  FontOld = SelectObject(DC, Font);
-
-  if (0 == MenuInfo.Height)
-    {
-      MenuMenuBarCalcSize(DC, Rect, &MenuInfo, Wnd);
-    }
-
-  Rect->bottom = Rect->top + MenuInfo.Height;
-
-  FillRect(DC, Rect, GetSysColorBrush(flat_menu ? COLOR_MENUBAR : COLOR_MENU));
-
-  SelectObject(DC, GetStockObject(DC_PEN));
-  SetDCPenColor(DC, GetSysColor(COLOR_3DFACE));
-  MoveToEx(DC, Rect->left, Rect->bottom - 1, NULL);
-  LineTo(DC, Rect->right, Rect->bottom - 1);
-
-  if (0 == MenuInfo.MenuItemCount)
-    {
-      SelectObject(DC, FontOld);
-      return GetSystemMetrics(SM_CYMENU);
-    }
-
-  MenuInitRosMenuItemInfo(&ItemInfo);
-  for (i = 0; i < MenuInfo.MenuItemCount; i++)
-    {
-      if (MenuGetRosMenuItemInfo(MenuInfo.Self, i, &ItemInfo))
-        {
-          MenuDrawMenuItem(Wnd, &MenuInfo, Wnd, DC, &ItemInfo,
-                           MenuInfo.Height, TRUE, ODA_DRAWENTIRE);
-        }
-    }
-  MenuCleanupRosMenuItemInfo(&ItemInfo);
-
-  SelectObject(DC, FontOld);
-
-  return MenuInfo.Height;
-}
-
-/***********************************************************************
- *           MenuShowSubPopup
- *
- * Display the sub-menu of the selected item of this menu.
- * Return the handle of the submenu, or menu if no submenu to display.
- */
-static HMENU FASTCALL
-MenuShowSubPopup(HWND WndOwner, PROSMENUINFO MenuInfo, BOOL SelectFirst, UINT Flags)
-{
-  extern void FASTCALL NcGetSysPopupPos(HWND Wnd, RECT *Rect);
-  RECT Rect;
-  ROSMENUITEMINFO ItemInfo;
-  ROSMENUINFO SubMenuInfo;
-  HDC Dc;
-  HMENU Ret;
-
-  TRACE("owner=%x menu=%p 0x%04x\n", WndOwner, MenuInfo, SelectFirst);
-
-  if (NO_SELECTED_ITEM == MenuInfo->FocusedItem)
-    {
-      return MenuInfo->Self;
-    }
-
-  MenuInitRosMenuItemInfo(&ItemInfo);
-  if (! MenuGetRosMenuItemInfo(MenuInfo->Self, MenuInfo->FocusedItem, &ItemInfo))
-    {
-      MenuCleanupRosMenuItemInfo(&ItemInfo);
-      return MenuInfo->Self;
-    }
-  if (0 == (ItemInfo.fType & MF_POPUP) || 0 != (ItemInfo.fState & (MF_GRAYED | MF_DISABLED)))
-    {
-      MenuCleanupRosMenuItemInfo(&ItemInfo);
-      return MenuInfo->Self;
-    }
-
-  /* message must be sent before using item,
-     because nearly everything may be changed by the application ! */
-
-  /* Send WM_INITMENUPOPUP message only if TPM_NONOTIFY flag is not specified */
-  if (0 == (Flags & TPM_NONOTIFY))
-    {
-      SendMessageW(WndOwner, WM_INITMENUPOPUP, (WPARAM) ItemInfo.hSubMenu,
-                   MAKELONG(MenuInfo->FocusedItem, IS_SYSTEM_MENU(MenuInfo)));
-    }
-
-  if (! MenuGetRosMenuItemInfo(MenuInfo->Self, MenuInfo->FocusedItem, &ItemInfo))
-    {
-      MenuCleanupRosMenuItemInfo(&ItemInfo);
-      return MenuInfo->Self;
-    }
-  Rect = ItemInfo.Rect;
-
-  /* correct item if modified as a reaction to WM_INITMENUPOPUP message */
-  if (0 == (ItemInfo.fState & MF_HILITE))
-    {
-      if (0 != (MenuInfo->Flags & MF_POPUP))
-        {
-          Dc = GetDC(MenuInfo->Wnd);
-        }
-      else
-        {
-          Dc = GetDCEx(MenuInfo->Wnd, 0, DCX_CACHE | DCX_WINDOW);
-        }
-
-      SelectObject(Dc, hMenuFont);
-      ItemInfo.fMask |= MIIM_STATE;
-      ItemInfo.fState |= MF_HILITE;
-      MenuSetRosMenuItemInfo(MenuInfo->Self, MenuInfo->FocusedItem, &ItemInfo);
-      MenuDrawMenuItem(MenuInfo->Wnd, MenuInfo, WndOwner, Dc, &ItemInfo, MenuInfo->Height,
-                       ! (MenuInfo->Flags & MF_POPUP), ODA_DRAWENTIRE);
-      ReleaseDC(MenuInfo->Wnd, Dc);
-    }
-
-  if (0 == ItemInfo.Rect.top && 0 == ItemInfo.Rect.left
-      && 0 == ItemInfo.Rect.bottom && 0 == ItemInfo.Rect.right)
-    {
-      ItemInfo.Rect = Rect;
-    }
-
-  ItemInfo.fMask |= MIIM_STATE;
-  ItemInfo.fState |= MF_MOUSESELECT;
-  MenuSetRosMenuItemInfo(MenuInfo->Self, MenuInfo->FocusedItem, &ItemInfo);
-
-  if (IS_SYSTEM_MENU(MenuInfo))
-    {
-      MenuInitSysMenuPopup(ItemInfo.hSubMenu, GetWindowLongPtrW(MenuInfo->Wnd, GWL_STYLE),
-                           GetClassLongPtrW(MenuInfo->Wnd, GCL_STYLE), HTSYSMENU);
-
-      NcGetSysPopupPos(MenuInfo->Wnd, &Rect);
-      Rect.top = Rect.bottom;
-      Rect.right = GetSystemMetrics(SM_CXSIZE);
-      Rect.bottom = GetSystemMetrics(SM_CYSIZE);
-    }
-  else
-    {
-      GetWindowRect(MenuInfo->Wnd, &Rect);
-      if (0 != (MenuInfo->Flags & MF_POPUP))
-       {
-          Rect.left += ItemInfo.Rect.right - GetSystemMetrics(SM_CXBORDER);
-          Rect.top += ItemInfo.Rect.top - 3;
-          Rect.right = ItemInfo.Rect.left - ItemInfo.Rect.right + GetSystemMetrics(SM_CXBORDER);
-          Rect.bottom = ItemInfo.Rect.top - ItemInfo.Rect.bottom - 3 - 2
-                                                    - GetSystemMetrics(SM_CYBORDER);
-        }
-      else
-        {
-          Rect.left += ItemInfo.Rect.left;
-          Rect.top += ItemInfo.Rect.bottom;
-          Rect.right = ItemInfo.Rect.right - ItemInfo.Rect.left;
-          Rect.bottom = ItemInfo.Rect.bottom - ItemInfo.Rect.top;
-        }
-    }
-
-  MenuShowPopup(WndOwner, ItemInfo.hSubMenu, MenuInfo->FocusedItem, Flags,
-                Rect.left, Rect.top, Rect.right, Rect.bottom );
-  if (SelectFirst && MenuGetRosMenuInfo(&SubMenuInfo, ItemInfo.hSubMenu))
-    {
-      MenuMoveSelection(WndOwner, &SubMenuInfo, ITEM_NEXT);
-    }
-
-  Ret = ItemInfo.hSubMenu;
-  MenuCleanupRosMenuItemInfo(&ItemInfo);
-
-  return Ret;
-}
-
-/**********************************************************************
- *         MENU_EndMenu
- *
- * Calls EndMenu() if the hwnd parameter belongs to the menu owner
- *
- * Does the (menu stuff) of the default window handling of WM_CANCELMODE
- */
-void MENU_EndMenu( HWND hwnd )
-{
-    ROSMENUINFO MenuInfo;
-    BOOL Ret = FALSE;
-    if (top_popup_hmenu)
-       Ret = MenuGetRosMenuInfo(&MenuInfo, top_popup_hmenu);
-    if (Ret && hwnd == MenuInfo.WndOwner) EndMenu();
-}
-
-/***********************************************************************
- *           MenuHideSubPopups
- *
- * Hide the sub-popup menus of this menu.
- */
-static void FASTCALL
-MenuHideSubPopups(HWND WndOwner, PROSMENUINFO MenuInfo,
-                               BOOL SendMenuSelect, UINT wFlags)
-{
-  ROSMENUINFO SubMenuInfo;
-  ROSMENUITEMINFO ItemInfo;
-
-  TRACE("owner=%x menu=%x 0x%04x\n", WndOwner, MenuInfo, SendMenuSelect);
-
-  if (NULL != MenuInfo && NULL != top_popup && NO_SELECTED_ITEM != MenuInfo->FocusedItem)
-  {
-      MenuInitRosMenuItemInfo(&ItemInfo);
-      ItemInfo.fMask |= MIIM_FTYPE | MIIM_STATE;
-      if (! MenuGetRosMenuItemInfo(MenuInfo->Self, MenuInfo->FocusedItem, &ItemInfo)
-          || 0 == (ItemInfo.fType & MF_POPUP)
-          || 0 == (ItemInfo.fState & MF_MOUSESELECT))
-      {
-          MenuCleanupRosMenuItemInfo(&ItemInfo);
-          return;
-      }
-      ItemInfo.fState &= ~MF_MOUSESELECT;
-      ItemInfo.fMask |= MIIM_STATE;
-      MenuSetRosMenuItemInfo(MenuInfo->Self, MenuInfo->FocusedItem, &ItemInfo);
-      if (MenuGetRosMenuInfo(&SubMenuInfo, ItemInfo.hSubMenu))
-      {
-          MenuHideSubPopups(WndOwner, &SubMenuInfo, FALSE, wFlags);
-          MenuSelectItem(WndOwner, &SubMenuInfo, NO_SELECTED_ITEM, SendMenuSelect, NULL);
-          DestroyWindow(SubMenuInfo.Wnd);
-          SubMenuInfo.Wnd = NULL;
-          MenuSetRosMenuInfo(&SubMenuInfo);
-
-          if (!(wFlags & TPM_NONOTIFY))
-             SendMessageW( WndOwner, WM_UNINITMENUPOPUP, (WPARAM)ItemInfo.hSubMenu,
-                                 MAKELPARAM(0, IS_SYSTEM_MENU(&SubMenuInfo)) );
-      }
-  }
-}
-
-/***********************************************************************
- *           MenuSwitchTracking
- *
- * Helper function for menu navigation routines.
- */
-static void FASTCALL
-MenuSwitchTracking(MTRACKER* Mt, PROSMENUINFO PtMenuInfo, UINT Index, UINT wFlags)
-{
-  ROSMENUINFO TopMenuInfo;
-
-  TRACE("%x menu=%x 0x%04x\n", Mt, PtMenuInfo->Self, Index);
-
-  if (MenuGetRosMenuInfo(&TopMenuInfo, Mt->TopMenu) &&
-      Mt->TopMenu != PtMenuInfo->Self &&
-      0 == ((PtMenuInfo->Flags | TopMenuInfo.Flags) & MF_POPUP))
-    {
-      /* both are top level menus (system and menu-bar) */
-      MenuHideSubPopups(Mt->OwnerWnd, &TopMenuInfo, FALSE, wFlags);
-      MenuSelectItem(Mt->OwnerWnd, &TopMenuInfo, NO_SELECTED_ITEM, FALSE, NULL);
-      Mt->TopMenu = PtMenuInfo->Self;
-    }
-  else
-    {
-      MenuHideSubPopups(Mt->OwnerWnd, PtMenuInfo, FALSE, wFlags);
-    }
-
-  MenuSelectItem(Mt->OwnerWnd, PtMenuInfo, Index, TRUE, NULL);
-}
-
-/***********************************************************************
- *           MenuExecFocusedItem
- *
- * Execute a menu item (for instance when user pressed Enter).
- * Return the wID of the executed item. Otherwise, -1 indicating
- * that no menu item was executed, -2 if a popup is shown;
- * Have to receive the flags for the TrackPopupMenu options to avoid
- * sending unwanted message.
- *
- */
-static INT FASTCALL
-MenuExecFocusedItem(MTRACKER *Mt, PROSMENUINFO MenuInfo, UINT Flags)
-{
-  ROSMENUITEMINFO ItemInfo;
-  UINT wID;
-
-  TRACE("%p menu=%p\n", Mt, MenuInfo);
-
-  if (0 == MenuInfo->MenuItemCount || NO_SELECTED_ITEM == MenuInfo->FocusedItem)
-    {
-      return -1;
-    }
-
-  MenuInitRosMenuItemInfo(&ItemInfo);
-  if (! MenuGetRosMenuItemInfo(MenuInfo->Self, MenuInfo->FocusedItem, &ItemInfo))
-    {
-      MenuCleanupRosMenuItemInfo(&ItemInfo);
-      return -1;
-    }
-
-  TRACE("%p %08x %p\n", MenuInfo, ItemInfo.wID, ItemInfo.hSubMenu);
-
-  if (0 == (ItemInfo.fType & MF_POPUP))
-    {
-      if (0 == (ItemInfo.fState & (MF_GRAYED | MF_DISABLED))
-          && 0 == (ItemInfo.fType & MF_SEPARATOR))
-        {
-          /* If TPM_RETURNCMD is set you return the id, but
-            do not send a message to the owner */
-          if (0 == (Flags & TPM_RETURNCMD))
-           {
-              if (0 != (MenuInfo->Flags & MF_SYSMENU))
-                {
-                  PostMessageW(Mt->OwnerWnd, WM_SYSCOMMAND, ItemInfo.wID,
-                               MAKELPARAM((SHORT) Mt->Pt.x, (SHORT) Mt->Pt.y));
-                }
-              else
-                {
-                  if (MenuInfo->dwStyle & MNS_NOTIFYBYPOS)
-                      PostMessageW(Mt->OwnerWnd, WM_MENUCOMMAND,
-                                                 MenuInfo->FocusedItem,
-                                                       (LPARAM)MenuInfo->Self);
-                  else
-                    PostMessageW(Mt->OwnerWnd, WM_COMMAND, ItemInfo.wID, 0);
-                }
-            }
-          wID = ItemInfo.wID;
-          MenuCleanupRosMenuItemInfo(&ItemInfo);
-         return wID;
-        }
-    }
-  else
-    {
-      Mt->CurrentMenu = MenuShowSubPopup(Mt->OwnerWnd, MenuInfo, TRUE, Flags);
-      return -2;
-    }
-
-  return -1;
-}
-
-/***********************************************************************
- *           MenuButtonDown
- *
- * Return TRUE if we can go on with menu tracking.
- */
-static BOOL FASTCALL
-MenuButtonDown(MTRACKER* Mt, HMENU PtMenu, UINT Flags)
-{
-  int Index;
-  ROSMENUINFO MenuInfo;
-  ROSMENUITEMINFO Item;
-
-  TRACE("%x PtMenu=%p\n", Mt, PtMenu);
-
-  if (NULL != PtMenu)
-    {
-      if (! MenuGetRosMenuInfo(&MenuInfo, PtMenu))
-        {
-          return FALSE;
-        }
-      if (IS_SYSTEM_MENU(&MenuInfo))
-        {
-          Index = 0;
-        }
-      else
-        {
-          Index = NtUserMenuItemFromPoint(Mt->OwnerWnd, PtMenu, Mt->Pt.x, Mt->Pt.y);
-        }
-      MenuInitRosMenuItemInfo(&Item);
-      if (NO_SELECTED_ITEM == Index || ! MenuGetRosMenuItemInfo(PtMenu, Index, &Item))
-        {
-          MenuCleanupRosMenuItemInfo(&Item);
-          return FALSE;
-        }
-
-      if (!(Item.fType & MF_SEPARATOR) &&
-          !(Item.fState & (MFS_DISABLED | MFS_GRAYED)) )
-       {
-          if (MenuInfo.FocusedItem != Index)
-            {
-              MenuSwitchTracking(Mt, &MenuInfo, Index, Flags);
-            }
-
-          /* If the popup menu is not already "popped" */
-          if (0 == (Item.fState & MF_MOUSESELECT))
-            {
-              Mt->CurrentMenu = MenuShowSubPopup(Mt->OwnerWnd, &MenuInfo, FALSE, Flags);
-            }
-        }
-
-      MenuCleanupRosMenuItemInfo(&Item);
-
-      return TRUE;
-    }
-
-  /* else the click was on the menu bar, finish the tracking */
-
-  return FALSE;
-}
-
-/***********************************************************************
- *           MenuButtonUp
- *
- * Return the value of MenuExecFocusedItem if
- * the selected item was not a popup. Else open the popup.
- * A -1 return value indicates that we go on with menu tracking.
- *
- */
-static INT FASTCALL
-MenuButtonUp(MTRACKER *Mt, HMENU PtMenu, UINT Flags)
-{
-  INT Id;
-  ROSMENUINFO MenuInfo;
-  ROSMENUITEMINFO ItemInfo;
-
-  TRACE("%p hmenu=%x\n", Mt, PtMenu);
-
-  if (NULL != PtMenu)
-    {
-      Id = 0;
-      if (! MenuGetRosMenuInfo(&MenuInfo, PtMenu))
-        {
-          return -1;
-        }
-
-      if (! IS_SYSTEM_MENU(&MenuInfo))
-        {
-          Id = NtUserMenuItemFromPoint(Mt->OwnerWnd, MenuInfo.Self, Mt->Pt.x, Mt->Pt.y);
-        }
-      MenuInitRosMenuItemInfo(&ItemInfo);
-      if (0 <= Id && MenuGetRosMenuItemInfo(MenuInfo.Self, Id, &ItemInfo) &&
-          MenuInfo.FocusedItem == Id)
-        {
-          if (0 == (ItemInfo.fType & MF_POPUP))
-            {
-              INT ExecutedMenuId = MenuExecFocusedItem(Mt, &MenuInfo, Flags);
-              MenuCleanupRosMenuItemInfo(&ItemInfo);
-              return (ExecutedMenuId < 0) ? -1 : ExecutedMenuId;
-            }
-          MenuCleanupRosMenuItemInfo(&ItemInfo);
-
-          /* If we are dealing with the top-level menu            */
-          /* and this is a click on an already "popped" item:     */
-          /* Stop the menu tracking and close the opened submenus */
-          if (Mt->TopMenu == MenuInfo.Self && MenuInfo.TimeToHide)
-            {
-              MenuCleanupRosMenuItemInfo(&ItemInfo);
-              return 0;
-            }
-        }
-      MenuCleanupRosMenuItemInfo(&ItemInfo);
-      MenuInfo.TimeToHide = TRUE;
-      MenuSetRosMenuInfo(&MenuInfo);
-    }
-
-  return -1;
-}
-
-/***********************************************************************
- *           MenuPtMenu
- *
- * Walks menu chain trying to find a menu pt maps to.
- */
-static HMENU FASTCALL
-MenuPtMenu(HMENU Menu, POINT Pt)
-{
-  extern LRESULT DefWndNCHitTest(HWND hWnd, POINT Point);
-  ROSMENUINFO MenuInfo;
-  ROSMENUITEMINFO ItemInfo;
-  HMENU Ret = NULL;
-  INT Ht;
-
-  if (! MenuGetRosMenuInfo(&MenuInfo, Menu))
-    {
-      return NULL;
-    }
-
-  /* try subpopup first (if any) */
-  if (NO_SELECTED_ITEM != MenuInfo.FocusedItem)
-    {
-      MenuInitRosMenuItemInfo(&ItemInfo);
-      if (MenuGetRosMenuItemInfo(MenuInfo.Self, MenuInfo.FocusedItem, &ItemInfo) &&
-          0 != (ItemInfo.fType & MF_POPUP) &&
-          0 != (ItemInfo.fState & MF_MOUSESELECT))
-        {
-          Ret = MenuPtMenu(ItemInfo.hSubMenu, Pt);
-          if (NULL != Ret)
-            {
-              MenuCleanupRosMenuItemInfo(&ItemInfo);
-              return Ret;
-            }
-        }
-      MenuCleanupRosMenuItemInfo(&ItemInfo);
-    }
-
-  /* check the current window (avoiding WM_HITTEST) */
-  Ht = DefWndNCHitTest(MenuInfo.Wnd, Pt);
-  if (0 != (MenuInfo.Flags & MF_POPUP))
-    {
-      if (HTNOWHERE != Ht && HTERROR != Ht)
-        {
-          Ret = Menu;
-        }
-    }
-  else if (HTSYSMENU == Ht)
-    {
-      Ret = NtUserGetSystemMenu(MenuInfo.Wnd, FALSE);
-    }
-  else if (HTMENU == Ht)
-    {
-      Ret = GetMenu(MenuInfo.Wnd);
-    }
-
-  return Ret;
-}
-
-/***********************************************************************
- *           MenuMouseMove
- *
- * Return TRUE if we can go on with menu tracking.
- */
-static BOOL FASTCALL
-MenuMouseMove(MTRACKER *Mt, HMENU PtMenu, UINT Flags)
-{
-  INT Index;
-  ROSMENUINFO MenuInfo;
-  ROSMENUITEMINFO ItemInfo;
-
-  if (NULL != PtMenu)
-    {
-      if (! MenuGetRosMenuInfo(&MenuInfo, PtMenu))
-        {
-          return TRUE;
-        }
-      if (IS_SYSTEM_MENU(&MenuInfo))
-        {
-          Index = 0;
-        }
-      else
-        {
-          Index = NtUserMenuItemFromPoint(Mt->OwnerWnd, PtMenu, Mt->Pt.x, Mt->Pt.y);
-        }
-    }
-  else
-    {
-      Index = NO_SELECTED_ITEM;
-    }
-
-  if (NO_SELECTED_ITEM == Index)
-    {
-      if (Mt->CurrentMenu == MenuInfo.Self ||
-          MenuGetRosMenuInfo(&MenuInfo, Mt->CurrentMenu))
-        {
-          MenuSelectItem(Mt->OwnerWnd, &MenuInfo, NO_SELECTED_ITEM,
-                         TRUE, Mt->TopMenu);
-        }
-    }
-  else if (MenuInfo.FocusedItem != Index)
-    {
-       MenuInitRosMenuItemInfo(&ItemInfo);
-       if (MenuGetRosMenuItemInfo(MenuInfo.Self, Index, &ItemInfo) &&
-           !(ItemInfo.fType & MF_SEPARATOR))
-       {
-           MenuSwitchTracking(Mt, &MenuInfo, Index, Flags);
-        if (!(ItemInfo.fState & (MFS_DISABLED | MFS_GRAYED)))
-               Mt->CurrentMenu = MenuShowSubPopup(Mt->OwnerWnd, &MenuInfo, FALSE, Flags);
-       }
-       MenuCleanupRosMenuItemInfo(&ItemInfo);
-    }
-
-  return TRUE;
-}
-
-/***********************************************************************
- *           MenuGetSubPopup
- *
- * Return the handle of the selected sub-popup menu (if any).
- */
-static HMENU FASTCALL
-MenuGetSubPopup(HMENU Menu)
-{
-  ROSMENUINFO MenuInfo;
-  ROSMENUITEMINFO ItemInfo;
-
-  if (! MenuGetRosMenuInfo(&MenuInfo, Menu)
-      || NO_SELECTED_ITEM == MenuInfo.FocusedItem)
-    {
-      return NULL;
-    }
-
-  MenuInitRosMenuItemInfo(&ItemInfo);
-  if (! MenuGetRosMenuItemInfo(MenuInfo.Self, MenuInfo.FocusedItem, &ItemInfo))
-    {
-      MenuCleanupRosMenuItemInfo(&ItemInfo);
-      return NULL;
-    }
-  if (0 != (ItemInfo.fType & MF_POPUP) && 0 != (ItemInfo.fState & MF_MOUSESELECT))
-    {
-      MenuCleanupRosMenuItemInfo(&ItemInfo);
-      return ItemInfo.hSubMenu;
-    }
-
-  MenuCleanupRosMenuItemInfo(&ItemInfo);
-  return NULL;
-}
-
-/***********************************************************************
- *           MenuDoNextMenu
- *
- * NOTE: WM_NEXTMENU documented in Win32 is a bit different.
- */
-static LRESULT FASTCALL
-MenuDoNextMenu(MTRACKER* Mt, UINT Vk, UINT wFlags)
-{
-  ROSMENUINFO TopMenuInfo;
-  ROSMENUINFO MenuInfo;
-
-  if (! MenuGetRosMenuInfo(&TopMenuInfo, Mt->TopMenu))
-    {
-      return (LRESULT) FALSE;
-    }
-
-  if ((VK_LEFT == Vk && 0 == TopMenuInfo.FocusedItem)
-      || (VK_RIGHT == Vk && TopMenuInfo.FocusedItem == TopMenuInfo.MenuItemCount - 1))
-    {
-      MDINEXTMENU NextMenu;
-      HMENU NewMenu;
-      HWND NewWnd;
-      UINT Id = 0;
-
-      NextMenu.hmenuIn = (IS_SYSTEM_MENU(&TopMenuInfo)) ? GetSubMenu(Mt->TopMenu, 0) : Mt->TopMenu;
-      NextMenu.hmenuNext = NULL;
-      NextMenu.hwndNext = NULL;
-      SendMessageW(Mt->OwnerWnd, WM_NEXTMENU, Vk, (LPARAM) &NextMenu);
-
-      TRACE("%p [%p] -> %p [%p]\n",
-             Mt->CurrentMenu, Mt->OwnerWnd, NextMenu.hmenuNext, NextMenu.hwndNext );
-
-      if (NULL == NextMenu.hmenuNext || NULL == NextMenu.hwndNext)
-        {
-          DWORD Style = GetWindowLongPtrW(Mt->OwnerWnd, GWL_STYLE);
-          NewWnd = Mt->OwnerWnd;
-          if (IS_SYSTEM_MENU(&TopMenuInfo))
-            {
-              /* switch to the menu bar */
-
-              if (0 != (Style & WS_CHILD)
-                  || NULL == (NewMenu = GetMenu(NewWnd)))
-                {
-                  return FALSE;
-                }
-
-              if (VK_LEFT == Vk)
-                {
-                  if (! MenuGetRosMenuInfo(&MenuInfo, NewMenu))
-                    {
-                      return FALSE;
-                    }
-                  Id = MenuInfo.MenuItemCount - 1;
-                }
-            }
-          else if (0 != (Style & WS_SYSMENU))
-            {
-              /* switch to the system menu */
-              NewMenu = NtUserGetSystemMenu(NewWnd, FALSE);
-            }
-          else
-            {
-              return FALSE;
-            }
-        }
-      else    /* application returned a new menu to switch to */
-        {
-          NewMenu = NextMenu.hmenuNext;
-          NewWnd = NextMenu.hwndNext;
-
-          if (IsMenu(NewMenu) && IsWindow(NewWnd))
-            {
-              DWORD Style = GetWindowLongPtrW(NewWnd, GWL_STYLE);
-
-              if (0 != (Style & WS_SYSMENU)
-                  && GetSystemMenu(NewWnd, FALSE) == NewMenu)
-                {
-                  /* get the real system menu */
-                  NewMenu = NtUserGetSystemMenu(NewWnd, FALSE);
-                }
-              else if (0 != (Style & WS_CHILD) || GetMenu(NewWnd) != NewMenu)
-                {
-                  /* FIXME: Not sure what to do here;
-                   * perhaps try to track NewMenu as a popup? */
-
-                  WARN(" -- got confused.\n");
-                  return FALSE;
-                }
-            }
-          else
-            {
-              return FALSE;
-            }
-        }
-
-      if (NewMenu != Mt->TopMenu)
-        {
-          MenuSelectItem(Mt->OwnerWnd, &TopMenuInfo, NO_SELECTED_ITEM,
-                         FALSE, 0 );
-          if (Mt->CurrentMenu != Mt->TopMenu)
-            {
-              MenuHideSubPopups(Mt->OwnerWnd, &TopMenuInfo, FALSE, wFlags);
-            }
-        }
-
-      if (NewWnd != Mt->OwnerWnd)
-        {
-          Mt->OwnerWnd = NewWnd;
-          NtUserxSetGUIThreadHandle(MSQ_STATE_MENUOWNER, Mt->OwnerWnd); // 1
-          SetCapture(Mt->OwnerWnd);                                          // 2
-        }
-
-      Mt->TopMenu = Mt->CurrentMenu = NewMenu; /* all subpopups are hidden */
-      if (MenuGetRosMenuInfo(&TopMenuInfo, Mt->TopMenu))
-        {
-          MenuSelectItem(Mt->OwnerWnd, &TopMenuInfo, Id, TRUE, 0);
-        }
-
-      return TRUE;
-    }
-
-  return FALSE;
-}
-
-/***********************************************************************
- *           MenuSuspendPopup
- *
- * The idea is not to show the popup if the next input message is
- * going to hide it anyway.
- */
-static BOOL FASTCALL
-MenuSuspendPopup(MTRACKER* Mt, UINT uMsg)
-{
-    MSG msg;
-
-    msg.hwnd = Mt->OwnerWnd;
-
-    PeekMessageW( &msg, 0, uMsg, uMsg, PM_NOYIELD | PM_REMOVE); // ported incorrectly since 8317 GvG
-//    Mt->TrackFlags |= TF_SKIPREMOVE; // This sends TrackMenu into a loop with arrow keys!!!!
-
-    switch( uMsg )
-    {
-       case WM_KEYDOWN:
-            PeekMessageW( &msg, 0, 0, 0, PM_NOYIELD | PM_NOREMOVE);
-            if( msg.message == WM_KEYUP || msg.message == WM_PAINT )
-            {
-                PeekMessageW( &msg, 0, 0, 0, PM_NOYIELD | PM_REMOVE);
-                PeekMessageW( &msg, 0, 0, 0, PM_NOYIELD | PM_NOREMOVE);
-                if( msg.message == WM_KEYDOWN &&
-                   (msg.wParam == VK_LEFT || msg.wParam == VK_RIGHT))
-                {
-                    Mt->TrackFlags |= TF_SUSPENDPOPUP;
-                    return TRUE;
-                }
-            }
-            break;
-    }
-    /* failures go through this */
-    Mt->TrackFlags &= ~TF_SUSPENDPOPUP;
-    return FALSE;
-}
-
-/***********************************************************************
- *           MenuKeyEscape
- *
- * Handle a VK_ESCAPE key event in a menu.
- */
-static BOOL FASTCALL
-MenuKeyEscape(MTRACKER *Mt, UINT Flags)
-{
-  BOOL EndMenu = TRUE;
-  ROSMENUINFO MenuInfo;
-  HMENU MenuTmp, MenuPrev;
-
-  if (Mt->CurrentMenu != Mt->TopMenu)
-    {
-      if (MenuGetRosMenuInfo(&MenuInfo, Mt->CurrentMenu)
-          && 0 != (MenuInfo.Flags & MF_POPUP))
-        {
-          MenuPrev = MenuTmp = Mt->TopMenu;
-
-          /* close topmost popup */
-          while (MenuTmp != Mt->CurrentMenu)
-            {
-              MenuPrev = MenuTmp;
-              MenuTmp = MenuGetSubPopup(MenuPrev);
-            }
-
-          if (MenuGetRosMenuInfo(&MenuInfo, MenuPrev))
-            {
-              MenuHideSubPopups(Mt->OwnerWnd, &MenuInfo, TRUE, Flags);
-            }
-          Mt->CurrentMenu = MenuPrev;
-          EndMenu = FALSE;
-        }
-    }
-
-  return EndMenu;
-}
-
-/***********************************************************************
- *           MenuKeyLeft
- *
- * Handle a VK_LEFT key event in a menu.
- */
-static void FASTCALL
-MenuKeyLeft(MTRACKER* Mt, UINT Flags)
-{
-  ROSMENUINFO MenuInfo;
-  ROSMENUINFO TopMenuInfo;
-  ROSMENUINFO PrevMenuInfo;
-  HMENU MenuTmp, MenuPrev;
-  UINT PrevCol;
-
-  MenuPrev = MenuTmp = Mt->TopMenu;
-
-  if (! MenuGetRosMenuInfo(&MenuInfo, Mt->CurrentMenu))
-    {
-      return;
-    }
-
-  /* Try to move 1 column left (if possible) */
-  if ( (PrevCol = MenuGetStartOfPrevColumn(&MenuInfo)) != NO_SELECTED_ITEM)
-  {
-     if (MenuGetRosMenuInfo(&MenuInfo, Mt->CurrentMenu))
-     {
-          MenuSelectItem(Mt->OwnerWnd, &MenuInfo, PrevCol, TRUE, 0);
-     }
-     return;
-  }
-
-  /* close topmost popup */
-  while (MenuTmp != Mt->CurrentMenu)
-    {
-      MenuPrev = MenuTmp;
-      MenuTmp = MenuGetSubPopup(MenuPrev);
-    }
-
-  if (! MenuGetRosMenuInfo(&PrevMenuInfo, MenuPrev))
-    {
-      return;
-    }
-  MenuHideSubPopups(Mt->OwnerWnd, &PrevMenuInfo, TRUE, Flags);
-  Mt->CurrentMenu = MenuPrev;
-
-  if (! MenuGetRosMenuInfo(&TopMenuInfo, Mt->TopMenu))
-    {
-      return;
-    }
-  if ((MenuPrev == Mt->TopMenu) && !(TopMenuInfo.Flags & MF_POPUP))
-    {
-      /* move menu bar selection if no more popups are left */
-
-      if (!MenuDoNextMenu(Mt, VK_LEFT, Flags))
-        {
-          MenuMoveSelection(Mt->OwnerWnd, &TopMenuInfo, ITEM_PREV);
-        }
-
-      if (MenuPrev != MenuTmp || Mt->TrackFlags & TF_SUSPENDPOPUP)
-        {
-          /* A sublevel menu was displayed - display the next one
-           * unless there is another displacement coming up */
-
-          if (! MenuSuspendPopup(Mt, WM_KEYDOWN)
-              && MenuGetRosMenuInfo(&TopMenuInfo, Mt->TopMenu))
-            {
-              Mt->CurrentMenu = MenuShowSubPopup(Mt->OwnerWnd, &TopMenuInfo,
-                                                 TRUE, Flags);
-            }
-        }
-    }
-}
-
-/***********************************************************************
- *           MenuKeyRight
- *
- * Handle a VK_RIGHT key event in a menu.
- */
-static void FASTCALL MenuKeyRight(MTRACKER *Mt, UINT Flags)
-{
-    HMENU hmenutmp;
-    ROSMENUINFO MenuInfo;
-    ROSMENUINFO CurrentMenuInfo;
-    UINT NextCol;
-
-    TRACE("MenuKeyRight called, cur %p, top %p.\n",
-         Mt->CurrentMenu, Mt->TopMenu);
-
-    if (! MenuGetRosMenuInfo(&MenuInfo, Mt->TopMenu)) return;
-    if ((MenuInfo.Flags & MF_POPUP) || (Mt->CurrentMenu != Mt->TopMenu))
-    {
-      /* If already displaying a popup, try to display sub-popup */
-
-      hmenutmp = Mt->CurrentMenu;
-      if (MenuGetRosMenuInfo(&CurrentMenuInfo, Mt->CurrentMenu))
-        {
-          Mt->CurrentMenu = MenuShowSubPopup(Mt->OwnerWnd, &CurrentMenuInfo, TRUE, Flags);
-        }
-
-      /* if subpopup was displayed then we are done */
-      if (hmenutmp != Mt->CurrentMenu) return;
-    }
-
-    if (! MenuGetRosMenuInfo(&CurrentMenuInfo, Mt->CurrentMenu))
-    {
-      return;
-    }
-
-    /* Check to see if there's another column */
-    if ( (NextCol = MenuGetStartOfNextColumn(&CurrentMenuInfo)) != NO_SELECTED_ITEM)
-    {
-       TRACE("Going to %d.\n", NextCol);
-       if (MenuGetRosMenuInfo(&MenuInfo, Mt->CurrentMenu))
-       {
-          MenuSelectItem(Mt->OwnerWnd, &MenuInfo, NextCol, TRUE, 0);
-       }
-       return;
-    }
-
-    if (!(MenuInfo.Flags & MF_POPUP)) /* menu bar tracking */
-    {
-      if (Mt->CurrentMenu != Mt->TopMenu)
-        {
-          MenuHideSubPopups(Mt->OwnerWnd, &MenuInfo, FALSE, Flags);
-          hmenutmp = Mt->CurrentMenu = Mt->TopMenu;
-        }
-      else
-        {
-          hmenutmp = NULL;
-        }
-
-      /* try to move to the next item */
-      if ( !MenuDoNextMenu(Mt, VK_RIGHT, Flags))
-          MenuMoveSelection(Mt->OwnerWnd, &MenuInfo, ITEM_NEXT);
-
-      if ( hmenutmp || Mt->TrackFlags & TF_SUSPENDPOPUP )
-        {
-          if (! MenuSuspendPopup(Mt, WM_KEYDOWN)
-              && MenuGetRosMenuInfo(&MenuInfo, Mt->TopMenu))
-            {
-              Mt->CurrentMenu = MenuShowSubPopup(Mt->OwnerWnd, &MenuInfo,
-                                                 TRUE, Flags);
-            }
-        }
-    }
-}
-
-/***********************************************************************
- *           MenuTrackMenu
- *
- * Menu tracking code.
- */
-static INT FASTCALL MenuTrackMenu(HMENU hmenu, UINT wFlags, INT x, INT y,
-                            HWND hwnd, const RECT *lprect )
-{
-    MSG msg;
-    ROSMENUINFO MenuInfo;
-    ROSMENUITEMINFO ItemInfo;
-    BOOL fRemove;
-    INT executedMenuId = -1;
-    MTRACKER mt;
-    HWND capture_win;
-    BOOL enterIdleSent = FALSE;
-
-    mt.TrackFlags = 0;
-    mt.CurrentMenu = hmenu;
-    mt.TopMenu = hmenu;
-    mt.OwnerWnd = hwnd;
-    mt.Pt.x = x;
-    mt.Pt.y = y;
-
-    TRACE("hmenu=%p flags=0x%08x (%d,%d) hwnd=%x (%ld,%ld)-(%ld,%ld)\n",
-         hmenu, wFlags, x, y, hwnd, lprect ? lprect->left : 0, lprect ? lprect->top : 0,
-         lprect ? lprect->right : 0, lprect ? lprect->bottom : 0);
-
-    if (!IsMenu(hmenu))
-    {
-        WARN("Invalid menu handle %p\n", hmenu);
-        SetLastError( ERROR_INVALID_MENU_HANDLE );
-        return FALSE;
-    }
-
-    fEndMenu = FALSE;
-    if (! MenuGetRosMenuInfo(&MenuInfo, hmenu))
-    {
-        return FALSE;
-    }
-
-    if (wFlags & TPM_BUTTONDOWN)
-    {
-        /* Get the result in order to start the tracking or not */
-        fRemove = MenuButtonDown( &mt, hmenu, wFlags );
-        fEndMenu = !fRemove;
-    }
-
-    if (wFlags & TF_ENDMENU) fEndMenu = TRUE;
-
-    /* owner may not be visible when tracking a popup, so use the menu itself */
-    capture_win = (wFlags & TPM_POPUPMENU) ? MenuInfo.Wnd : mt.OwnerWnd;
-    NtUserxSetGUIThreadHandle(MSQ_STATE_MENUOWNER, capture_win); // 1
-    SetCapture(capture_win);                                          // 2
-
-    while (! fEndMenu)
-    {
-        BOOL ErrorExit = FALSE;
-        PVOID menu = ValidateHandle(mt.CurrentMenu, TYPE_MENU);
-        if (!menu) /* sometimes happens if I do a window manager close */
-           break;
+        mii.cbSize = sizeof(mii);
+        mii.fMask = MIIM_STATE | MIIM_ID | MIIM_TYPE;
+        mii.fType = GET_DWORD(res);
+        res += sizeof(DWORD);
+        mii.fState = GET_DWORD(res);
+        res += sizeof(DWORD);
+        mii.wID = GET_DWORD(res);
+        res += sizeof(DWORD);
+        resinfo = GET_WORD(res);
+        res += sizeof(WORD);
+        /* Align the text on a word boundary.  */
+        res += (~((UINT_PTR)res - 1)) & 1;
+        mii.dwTypeData = (LPWSTR)res;
+        mii.cch = strlenW(mii.dwTypeData);
+        res += (1 + strlenW(mii.dwTypeData)) * sizeof(WCHAR);
+        /* Align the following fields on a dword boundary.  */
+        res += (~((UINT_PTR)res - 1)) & 3;
 
-        /* we have to keep the message in the queue until it's
-         * clear that menu loop is not over yet. */
+        TRACE("Menu item: [%08x,%08x,%04x,%04x,%S]\n",
+              mii.fType, mii.fState, mii.wID, resinfo, mii.dwTypeData);
 
-        for (;;)
+        if (resinfo & 1) /* Pop-up? */
         {
-            if (PeekMessageW( &msg, 0, 0, 0, PM_NOREMOVE ))
-            {
-                if (!CallMsgFilterW( &msg, MSGF_MENU )) break;
-                /* remove the message from the queue */
-                PeekMessageW( &msg, 0, msg.message, msg.message, PM_REMOVE );
-            }
-            else
+            /* DWORD helpid = GET_DWORD(res); FIXME: use this. */
+            res += sizeof(DWORD);
+            mii.hSubMenu = CreatePopupMenu();
+            if (!mii.hSubMenu)
             {
-                /* ReactOS Check */
-                if (!ValidateHwnd(mt.OwnerWnd) || !ValidateHwnd(MenuInfo.Wnd))
-                {
-                   ErrorExit = TRUE; // Do not wait on dead windows, now test_capture_4 works.
-                   break;
-                }
-                if (!enterIdleSent)
-                {
-                  HWND win = MenuInfo.Flags & MF_POPUP ? MenuInfo.Wnd : NULL;
-                  enterIdleSent = TRUE;
-                  SendMessageW( mt.OwnerWnd, WM_ENTERIDLE, MSGF_MENU, (LPARAM) win);
-                }
-                WaitMessage();
+                ERR("CreatePopupMenu failed\n");
+                return NULL;
             }
-        }
-
-        if (ErrorExit) break; // Gracefully dropout.
-
-        /* check if EndMenu() tried to cancel us, by posting this message */
-        if (msg.message == WM_CANCELMODE)
-        {
-            /* we are now out of the loop */
-            fEndMenu = TRUE;
 
-            /* remove the message from the queue */
-            PeekMessageW( &msg, 0, msg.message, msg.message, PM_REMOVE );
-
-            /* break out of internal loop, ala ESCAPE */
-            break;
-        }
-
-        TranslateMessage( &msg );
-        mt.Pt = msg.pt;
-
-        if ( (msg.hwnd == MenuInfo.Wnd) || (msg.message!=WM_TIMER) )
-            enterIdleSent=FALSE;
-
-        fRemove = FALSE;
-        if ((msg.message >= WM_MOUSEFIRST) && (msg.message <= WM_MOUSELAST))
-        {
-            /*
-             * Use the mouse coordinates in lParam instead of those in the MSG
-             * struct to properly handle synthetic messages. They are already
-             * in screen coordinates.
-             */
-            mt.Pt.x = (short)LOWORD(msg.lParam);
-            mt.Pt.y = (short)HIWORD(msg.lParam);
-
-            /* Find a menu for this mouse event */
-            hmenu = MenuPtMenu(mt.TopMenu, mt.Pt);
-
-            switch(msg.message)
-            {
-                /* no WM_NC... messages in captured state */
-
-                case WM_RBUTTONDBLCLK:
-                case WM_RBUTTONDOWN:
-                     if (!(wFlags & TPM_RIGHTBUTTON)) break;
-                    /* fall through */
-                case WM_LBUTTONDBLCLK:
-                case WM_LBUTTONDOWN:
-                    /* If the message belongs to the menu, removes it from the queue */
-                    /* Else, end menu tracking */
-                    fRemove = MenuButtonDown(&mt, hmenu, wFlags);
-                    fEndMenu = !fRemove;
-                    break;
-
-                case WM_RBUTTONUP:
-                    if (!(wFlags & TPM_RIGHTBUTTON)) break;
-                    /* fall through */
-                case WM_LBUTTONUP:
-                    /* Check if a menu was selected by the mouse */
-                    if (hmenu)
-                    {
-                        executedMenuId = MenuButtonUp( &mt, hmenu, wFlags);
-                        TRACE("executedMenuId %d\n", executedMenuId);
-
-                    /* End the loop if executedMenuId is an item ID */
-                    /* or if the job was done (executedMenuId = 0). */
-                        fEndMenu = fRemove = (executedMenuId != -1);
-                    }
-                    /* No menu was selected by the mouse */
-                    /* if the function was called by TrackPopupMenu, continue
-                       with the menu tracking. If not, stop it */
-                    else
-                        fEndMenu = ((wFlags & TPM_POPUPMENU) ? FALSE : TRUE);
-
-                    break;
-
-                case WM_MOUSEMOVE:
-                    /* the selected menu item must be changed every time */
-                    /* the mouse moves. */
-
-                    if (hmenu)
-                        fEndMenu |= !MenuMouseMove( &mt, hmenu, wFlags );
-
-           } /* switch(msg.message) - mouse */
-        }
-        else if ((msg.message >= WM_KEYFIRST) && (msg.message <= WM_KEYLAST))
-        {
-            fRemove = TRUE;  /* Keyboard messages are always removed */
-            switch(msg.message)
-            {
-                case WM_KEYDOWN:
-                case WM_SYSKEYDOWN:
-                switch(msg.wParam)
-                {
-                    case VK_MENU:
-                    case VK_F10:
-                        fEndMenu = TRUE;
-                        break;
-
-                    case VK_HOME:
-                    case VK_END:
-                        if (MenuGetRosMenuInfo(&MenuInfo, mt.CurrentMenu))
-                        {
-                            MenuSelectItem(mt.OwnerWnd, &MenuInfo,
-                                    NO_SELECTED_ITEM, FALSE, 0 );
-                            MenuMoveSelection(mt.OwnerWnd, &MenuInfo,
-                                              VK_HOME == msg.wParam ? ITEM_NEXT : ITEM_PREV);
-                        }
-                        break;
-
-                    case VK_UP:
-                    case VK_DOWN: /* If on menu bar, pull-down the menu */
-                        if (MenuGetRosMenuInfo(&MenuInfo, mt.CurrentMenu))
-                        {
-                            if (!(MenuInfo.Flags & MF_POPUP))
-                            {
-                                if (MenuGetRosMenuInfo(&MenuInfo, mt.TopMenu))
-                                    mt.CurrentMenu = MenuShowSubPopup(mt.OwnerWnd, &MenuInfo, TRUE, wFlags);
-                            }
-                            else      /* otherwise try to move selection */
-                                MenuMoveSelection(mt.OwnerWnd, &MenuInfo,
-                                       (msg.wParam == VK_UP)? ITEM_PREV : ITEM_NEXT );
-                        }
-                        break;
-
-                    case VK_LEFT:
-                        MenuKeyLeft( &mt, wFlags );
-                        break;
-
-                    case VK_RIGHT:
-                        MenuKeyRight( &mt, wFlags );
-                        break;
-
-                    case VK_ESCAPE:
-                        fEndMenu = MenuKeyEscape(&mt, wFlags);
-                        break;
-
-                    case VK_F1:
-                    {
-                        HELPINFO hi;
-                        hi.cbSize = sizeof(HELPINFO);
-                        hi.iContextType = HELPINFO_MENUITEM;
-                        if (MenuGetRosMenuInfo(&MenuInfo, mt.CurrentMenu))
-                        {
-                            if (MenuInfo.FocusedItem == NO_SELECTED_ITEM)
-                                hi.iCtrlId = 0;
-                            else
-                            {
-                                MenuInitRosMenuItemInfo(&ItemInfo);
-                                if (MenuGetRosMenuItemInfo(MenuInfo.Self,
-                                                           MenuInfo.FocusedItem,
-                                                           &ItemInfo))
-                                {
-                                    hi.iCtrlId = ItemInfo.wID;
-                                }
-                                else
-                                {
-                                    hi.iCtrlId = 0;
-                                }
-                                MenuCleanupRosMenuItemInfo(&ItemInfo);
-                            }
-                        }
-                        hi.hItemHandle = hmenu;
-                        hi.dwContextId = MenuInfo.dwContextHelpID;
-                        hi.MousePos = msg.pt;
-                        SendMessageW(hwnd, WM_HELP, 0, (LPARAM)&hi);
-                        break;
-                    }
-
-                    default:
-                        break;
-                }
-                break;  /* WM_KEYDOWN */
-
-                case WM_CHAR:
-                case WM_SYSCHAR:
-                {
-                    UINT pos;
-
-                    if (! MenuGetRosMenuInfo(&MenuInfo, mt.CurrentMenu)) break;
-                    if (msg.wParam == L'\r' || msg.wParam == L' ')
-                    {
-                        executedMenuId = MenuExecFocusedItem(&mt, &MenuInfo, wFlags);
-                        fEndMenu = (executedMenuId != -2);
-                        break;
-                    }
-
-                    /* Hack to avoid control chars. */
-                    /* We will find a better way real soon... */
-                    if (msg.wParam < 32) break;
-
-                    pos = MenuFindItemByKey(mt.OwnerWnd, &MenuInfo,
-                                          LOWORD(msg.wParam), FALSE);
-                    if (pos == (UINT)-2) fEndMenu = TRUE;
-                    else if (pos == (UINT)-1) MessageBeep(0);
-                    else
-                    {
-                        MenuSelectItem(mt.OwnerWnd, &MenuInfo, pos,
-                            TRUE, 0);
-                        executedMenuId = MenuExecFocusedItem(&mt, &MenuInfo, wFlags);
-                        fEndMenu = (executedMenuId != -2);
-                    }
-                }
-                break;
-            }  /* switch(msg.message) - kbd */
-        }
-        else
-        {
-            PeekMessageW( &msg, 0, msg.message, msg.message, PM_REMOVE );
-            DispatchMessageW( &msg );
-            continue;
-        }
-
-        if (!fEndMenu) fRemove = TRUE;
-
-        /* finally remove message from the queue */
-
-        if (fRemove && !(mt.TrackFlags & TF_SKIPREMOVE) )
-            PeekMessageW( &msg, 0, msg.message, msg.message, PM_REMOVE );
-        else mt.TrackFlags &= ~TF_SKIPREMOVE;
-    }
-
-    NtUserxSetGUIThreadHandle(MSQ_STATE_MENUOWNER, NULL);
-    SetCapture(NULL);  /* release the capture */
-
-    /* If dropdown is still painted and the close box is clicked on
-       then the menu will be destroyed as part of the DispatchMessage above.
-       This will then invalidate the menu handle in mt.hTopMenu. We should
-       check for this first.  */
-    if( IsMenu( mt.TopMenu ) )
-    {
-        if (IsWindow(mt.OwnerWnd))
-        {
-            if (MenuGetRosMenuInfo(&MenuInfo, mt.TopMenu))
+            if (!(res = MENUEX_ParseResource(res, mii.hSubMenu)))
             {
-                MenuHideSubPopups(mt.OwnerWnd, &MenuInfo, FALSE, wFlags);
-
-                if (MenuInfo.Flags & MF_POPUP)
-                {
-                    IntNotifyWinEvent(EVENT_SYSTEM_MENUPOPUPEND, MenuInfo.Wnd, OBJID_CLIENT, CHILDID_SELF, 0);
-                    DestroyWindow(MenuInfo.Wnd);
-                    MenuInfo.Wnd = NULL;
-
-                    if (!(MenuInfo.Flags & TPM_NONOTIFY))
-                      SendMessageW( mt.OwnerWnd, WM_UNINITMENUPOPUP, (WPARAM)mt.TopMenu,
-                                 MAKELPARAM(0, IS_SYSTEM_MENU(&MenuInfo)) );
-
-                }
-                MenuSelectItem( mt.OwnerWnd, &MenuInfo, NO_SELECTED_ITEM, FALSE, 0 );
+                ERR("MENUEX_ParseResource failed\n");
+                DestroyMenu(mii.hSubMenu);
+                return NULL;
             }
-
-            SendMessageW( mt.OwnerWnd, WM_MENUSELECT, MAKEWPARAM(0, 0xffff), 0 );
+            mii.fMask |= MIIM_SUBMENU;
         }
-
-        /* Reset the variable for hiding menu */
-        if (MenuGetRosMenuInfo(&MenuInfo, mt.TopMenu))
+        else if (!mii.dwTypeData[0] && !(mii.fType & MF_SEPARATOR))
         {
-            MenuInfo.TimeToHide = FALSE;
-            MenuSetRosMenuInfo(&MenuInfo);
+            WARN("Converting NULL menu item %04x, type %04x to SEPARATOR\n",
+                mii.wID, mii.fType);
+            mii.fType |= MF_SEPARATOR;
         }
-    }
-
-    /* The return value is only used by TrackPopupMenu */
-    if (!(wFlags & TPM_RETURNCMD)) return TRUE;
-    if (executedMenuId == -1) executedMenuId = 0;
-    return executedMenuId;
-}
-
-/***********************************************************************
- *           MenuInitTracking
- */
-static BOOL FASTCALL MenuInitTracking(HWND hWnd, HMENU hMenu, BOOL bPopup, UINT wFlags)
-{
-    ROSMENUINFO MenuInfo;
-    
-    TRACE("hwnd=%p hmenu=%p\n", hWnd, hMenu);
-
-    HideCaret(0);
-
-    /* This makes the menus of applications built with Delphi work.
-     * It also enables menus to be displayed in more than one window,
-     * but there are some bugs left that need to be fixed in this case.
-     */
-    if (MenuGetRosMenuInfo(&MenuInfo, hMenu))
-    {
-        MenuInfo.Wnd = hWnd;
-        MenuSetRosMenuInfo(&MenuInfo);
-    }
-
-    /* Send WM_ENTERMENULOOP and WM_INITMENU message only if TPM_NONOTIFY flag is not specified */
-    if (!(wFlags & TPM_NONOTIFY))
-       SendMessageW( hWnd, WM_ENTERMENULOOP, bPopup, 0 );
-
-    SendMessageW( hWnd, WM_SETCURSOR, (WPARAM)hWnd, HTCAPTION );
-
-    if (!(wFlags & TPM_NONOTIFY))
-    {
-       SendMessageW( hWnd, WM_INITMENU, (WPARAM)hMenu, 0 );
-       /* If an app changed/recreated menu bar entries in WM_INITMENU
-        * menu sizes will be recalculated once the menu created/shown.
-        */
-
-       if (!MenuInfo.Height)
-       {
-          /* app changed/recreated menu bar entries in WM_INITMENU
-             Recalculate menu sizes else clicks will not work */
-             SetWindowPos(hWnd, 0, 0, 0, 0, 0, SWP_NOSIZE | SWP_NOMOVE |
-                       SWP_NOACTIVATE | SWP_NOZORDER | SWP_FRAMECHANGED );
-
-       }
-    }
-
-    IntNotifyWinEvent( EVENT_SYSTEM_MENUSTART,
-                       hWnd,
-                       MenuInfo.Flags & MF_SYSMENU ? OBJID_SYSMENU : OBJID_MENU,
-                       CHILDID_SELF, 0);
-    return TRUE;
+        InsertMenuItemW(hMenu, -1, MF_BYPOSITION, &mii);
+    } while (!(resinfo & MF_END));
+    return res;
 }
-/***********************************************************************
- *           MenuExitTracking
- */
-static BOOL FASTCALL MenuExitTracking(HWND hWnd, BOOL bPopup)
-{
-    TRACE("hwnd=%p\n", hWnd);
 
-    IntNotifyWinEvent( EVENT_SYSTEM_MENUEND, hWnd, OBJID_WINDOW, CHILDID_SELF, 0);
-    SendMessageW( hWnd, WM_EXITMENULOOP, bPopup, 0 );
-    ShowCaret(0);
-    top_popup = 0;
-    top_popup_hmenu = NULL;
-    return TRUE;
-}
 
-/***********************************************************************
- *           MenuTrackMouseMenuBar
+/**********************************************************************
+ *         MENU_mnu2mnuii
  *
- * Menu-bar tracking upon a mouse event. Called from NC_HandleSysCommand().
- */
-VOID MenuTrackMouseMenuBar( HWND hWnd, ULONG ht, POINT pt)
-{
-    HMENU hMenu = (ht == HTSYSMENU) ? NtUserGetSystemMenu( hWnd, FALSE) : GetMenu(hWnd);
-    UINT wFlags = TPM_BUTTONDOWN | TPM_LEFTALIGN | TPM_LEFTBUTTON;
-
-    TRACE("wnd=%p ht=0x%04x (%ld,%ld)\n", hWnd, ht, pt.x, pt.y);
-
-    if (IsMenu(hMenu))
-    {
-        /* map point to parent client coordinates */
-        HWND Parent = GetAncestor(hWnd, GA_PARENT );
-        if (Parent != GetDesktopWindow())
+ * Uses flags, id and text ptr, passed by InsertMenu() and
+ * ModifyMenu() to setup a MenuItemInfo structure.
+ */
+static void MENU_mnu2mnuii( UINT flags, UINT_PTR id, LPCWSTR str, LPMENUITEMINFOW pmii, BOOL Unicode)
+{
+    RtlZeroMemory( pmii, sizeof( MENUITEMINFOW));
+    pmii->cbSize = sizeof( MENUITEMINFOW);
+    pmii->fMask = MIIM_STATE | MIIM_ID | MIIM_FTYPE;
+    /* setting bitmap clears text and vice versa */
+    if( IS_STRING_ITEM(flags)) {
+        pmii->fMask |= MIIM_STRING | MIIM_BITMAP;
+        if( !str)
+            flags |= MF_SEPARATOR;
+        /* Item beginning with a backspace is a help item */
+        /* FIXME: wrong place, this is only true in win16 */
+        else
         {
-            ScreenToClient(Parent, &pt);
+            if (Unicode)
+           {
+              if (*str == '\b')
+              {
+                flags |= MF_HELP;
+                str++;
+              }
+           }
+           else
+           {
+              LPCSTR NewItemA = (LPCSTR) str;
+              if (*NewItemA == '\b')
+              {
+                 flags |= MF_HELP;
+                 NewItemA++;
+                 str = (LPCWSTR) NewItemA;
+              }
+              TRACE("A cch %d\n",strlen(NewItemA));
+           }
         }
-
-        MenuInitTracking(hWnd, hMenu, FALSE, wFlags);
-        MenuTrackMenu(hMenu, wFlags, pt.x, pt.y, hWnd, NULL);
-        MenuExitTracking(hWnd, FALSE);
-    }
-}
-
-
-/***********************************************************************
- *           MenuTrackKbdMenuBar
- *
- * Menu-bar tracking upon a keyboard event. Called from NC_HandleSysCommand().
- */
-VOID MenuTrackKbdMenuBar(HWND hwnd, UINT wParam, WCHAR wChar)
-{
-    UINT uItem = NO_SELECTED_ITEM;
-    HMENU hTrackMenu;
-    ROSMENUINFO MenuInfo;
-    UINT wFlags = TPM_LEFTALIGN | TPM_LEFTBUTTON;
-
-    TRACE("hwnd %p wParam 0x%04x wChar 0x%04x\n", hwnd, wParam, wChar);
-
-    /* find window that has a menu */
-
-    while (!((GetWindowLongPtrW( hwnd, GWL_STYLE ) &
-                                         (WS_CHILD | WS_POPUP)) != WS_CHILD))
-        if (!(hwnd = GetAncestor( hwnd, GA_PARENT ))) return;
-
-    /* check if we have to track a system menu */
-
-    hTrackMenu = GetMenu( hwnd );
-    if (!hTrackMenu || IsIconic(hwnd) || wChar == ' ' )
-    {
-        if (!(GetWindowLongPtrW( hwnd, GWL_STYLE ) & WS_SYSMENU)) return;
-        hTrackMenu = NtUserGetSystemMenu(hwnd, FALSE);
-        uItem = 0;
-        wParam |= HTSYSMENU; /* prevent item lookup */
-    }
-
-    if (!IsMenu( hTrackMenu )) return;
-
-    MenuInitTracking( hwnd, hTrackMenu, FALSE, wFlags );
-
-    if (! MenuGetRosMenuInfo(&MenuInfo, hTrackMenu))
-    {
-      goto track_menu;
+        pmii->dwTypeData = (LPWSTR)str;
+    } else if( flags & MFT_BITMAP){
+        pmii->fMask |= MIIM_BITMAP | MIIM_STRING;
+        pmii->hbmpItem = (HBITMAP)str;
     }
-
-    if( wChar && wChar != ' ' )
-    {
-        uItem = MenuFindItemByKey( hwnd, &MenuInfo, wChar, (wParam & HTSYSMENU) );
-        if ( uItem >= (UINT)(-2) )
-        {
-            if( uItem == (UINT)(-1) ) MessageBeep(0);
-            /* schedule end of menu tracking */
-            wFlags |= TF_ENDMENU;
-            goto track_menu;
-        }
+    if( flags & MF_OWNERDRAW){
+        pmii->fMask |= MIIM_DATA;
+        pmii->dwItemData = (ULONG_PTR) str;
     }
-
-    MenuSelectItem( hwnd, &MenuInfo, uItem, TRUE, 0 );
-
-    if (!(wParam & HTSYSMENU) || wChar == ' ')
-    {
-        if( uItem == NO_SELECTED_ITEM )
-            MenuMoveSelection( hwnd, &MenuInfo, ITEM_NEXT );
-        else
-            PostMessageW( hwnd, WM_KEYDOWN, VK_RETURN, 0 );
+    if( flags & MF_POPUP && MENU_GetMenu((HMENU)id)) {
+        pmii->fMask |= MIIM_SUBMENU;
+        pmii->hSubMenu = (HMENU)id;
     }
-
-track_menu:
-    MenuTrackMenu( hTrackMenu, wFlags, 0, 0, hwnd, NULL );
-    MenuExitTracking( hwnd, FALSE );
+    if( flags & MF_SEPARATOR) flags |= MF_GRAYED | MF_DISABLED;
+    pmii->fState = flags & MENUITEMINFO_STATE_MASK & ~MFS_DEFAULT;
+    pmii->fType = flags & MENUITEMINFO_TYPE_MASK;
+    pmii->wID = (UINT)id;
 }
 
 /**********************************************************************
- *           TrackPopupMenuEx   (USER32.@)
- */
-BOOL WINAPI TrackPopupMenuEx( HMENU Menu, UINT Flags, int x, int y,
-                              HWND Wnd, LPTPMPARAMS Tpm)
-{
-    BOOL ret = FALSE;
-    ROSMENUINFO MenuInfo;
-
-    if (!IsMenu(Menu))    
-    {
-      SetLastError( ERROR_INVALID_MENU_HANDLE );
-      return FALSE;
-    }
-
-    /* ReactOS Check */
-    if (!ValidateHwnd(Wnd))
-    {
-       return FALSE;
+ *             MENU_NormalizeMenuItemInfoStruct
+ *
+ * Helper for SetMenuItemInfo and InsertMenuItemInfo:
+ * check, copy and extend the MENUITEMINFO struct from the version that the application
+ * supplied to the version used by wine source. */
+static BOOL MENU_NormalizeMenuItemInfoStruct( const MENUITEMINFOW *pmii_in,
+                                              MENUITEMINFOW *pmii_out )
+{
+    /* do we recognize the size? */
+    if( !pmii_in || (pmii_in->cbSize != sizeof( MENUITEMINFOW) &&
+            pmii_in->cbSize != sizeof( MENUITEMINFOW) - sizeof( pmii_in->hbmpItem)) ) {
+        SetLastError( ERROR_INVALID_PARAMETER);
+        return FALSE;
     }
-
-    MenuGetRosMenuInfo(&MenuInfo, Menu);
-    if (IsWindow(MenuInfo.Wnd))
-    {
-        SetLastError( ERROR_POPUP_ALREADY_ACTIVE );
+    /* copy the fields that we have */
+    memcpy( pmii_out, pmii_in, pmii_in->cbSize);
+    /* if the hbmpItem member is missing then extend */
+    if( pmii_in->cbSize != sizeof( MENUITEMINFOW)) {
+        pmii_out->cbSize = sizeof( MENUITEMINFOW);
+        pmii_out->hbmpItem = NULL;
+    }
+    /* test for invalid bit combinations */
+    if( (pmii_out->fMask & MIIM_TYPE &&
+         pmii_out->fMask & (MIIM_STRING | MIIM_FTYPE | MIIM_BITMAP)) ||
+        (pmii_out->fMask & MIIM_FTYPE && pmii_out->fType & MFT_BITMAP)) {
+        ERR("invalid combination of fMask bits used\n");
+        /* this does not happen on Win9x/ME */
+        SetLastError( ERROR_INVALID_PARAMETER);
         return FALSE;
     }
-
-    MenuInitTracking(Wnd, Menu, TRUE, Flags);
-
-    /* Send WM_INITMENUPOPUP message only if TPM_NONOTIFY flag is not specified */
-    if (!(Flags & TPM_NONOTIFY))
-        SendMessageW(Wnd, WM_INITMENUPOPUP, (WPARAM) Menu, 0);
-
-    if (MenuShowPopup(Wnd, Menu, 0, Flags, x, y, 0, 0 ))
-       ret = MenuTrackMenu(Menu, Flags | TPM_POPUPMENU, 0, 0, Wnd,
-                           Tpm ? &Tpm->rcExclude : NULL);
-    MenuExitTracking(Wnd, TRUE);
-    return ret;
-}
-
-/**********************************************************************
- *           TrackPopupMenu     (USER32.@)
- */
-BOOL WINAPI TrackPopupMenu( HMENU Menu, UINT Flags, int x, int y,
-                            int Reserved, HWND Wnd, CONST RECT *Rect)
-{
-    return TrackPopupMenuEx( Menu, Flags, x, y, Wnd, NULL);
-}
-
-/*
- *  From MSDN:
- *  The MFT_BITMAP, MFT_SEPARATOR, and MFT_STRING values cannot be combined
- *  with one another. Also MFT_OWNERDRAW. Set fMask to MIIM_TYPE to use fType.
- *
- *  Windows 2K/XP: fType is used only if fMask has a value of MIIM_FTYPE.
- *
- *  MIIM_TYPE: Retrieves or sets the fType and dwTypeData members. Windows
- *  2K/XP: MIIM_TYPE is replaced by  MIIM_BITMAP, MIIM_FTYPE, and MIIM_STRING.
- *  MFT_STRING is replaced by MIIM_STRING.
- *  (So, I guess we should use MIIM_STRING only for strings?)
- *
- *  MIIM_FTYPE: Windows 2K/Windows XP: Retrieves or sets the fType member.
- *
- *  Based on wine, SetMenuItemInfo_common:
- *  1) set MIIM_STRING | MIIM_FTYPE | MIIM_BITMAP any one with MIIM_TYPE,
- *     it will result in a error.
- *  2) set menu mask to MIIM_FTYPE and MFT_BITMAP ftype it will result in a error.
- *     These conditions are addressed in Win32k IntSetMenuItemInfo.
- *
- */
-static
-BOOL
-FASTCALL
-MenuSetItemData(
-  LPMENUITEMINFOW mii,
-  UINT Flags,
-  UINT_PTR IDNewItem,
-  LPCWSTR NewItem,
-  BOOL Unicode)
-{
-/*
- * Let us assume MIIM_FTYPE is set and building a new menu item structure.
- */
-  if(Flags & MF_BITMAP)
-  {
-     mii->fMask |= MIIM_BITMAP;   /* Use the new way of seting hbmpItem.*/
-     mii->hbmpItem = (HBITMAP) NewItem;
-
-     if (Flags & MF_HELP)
-     {
-         /* increase ident */
-         mii->fType |= MF_HELP;
-     }
-  }
-  else if(Flags & MF_OWNERDRAW)
-  {
-    mii->fType |= MFT_OWNERDRAW;
-    mii->fMask |= MIIM_DATA;
-    mii->dwItemData = (DWORD_PTR) NewItem;
-  }
-  else if (Flags & MF_SEPARATOR)
-  {
-    mii->fType |= MFT_SEPARATOR;
-    if (!(Flags & (MF_GRAYED|MF_DISABLED)))
-      Flags |= MF_GRAYED|MF_DISABLED;
-  }
-  else /* Default action MF_STRING. */
-  {
-    /* Item beginning with a backspace is a help item */
-    if (NewItem != NULL)
+    /* convert old style (MIIM_TYPE) to the new and keep the old one too */
+    if( pmii_out->fMask & MIIM_TYPE){
+        pmii_out->fMask |= MIIM_FTYPE;
+        if( IS_STRING_ITEM(pmii_out->fType)){
+            pmii_out->fMask |= MIIM_STRING;
+        } else if( (pmii_out->fType) & MFT_BITMAP){
+            pmii_out->fMask |= MIIM_BITMAP;
+            pmii_out->hbmpItem = UlongToHandle(LOWORD(pmii_out->dwTypeData));
+        }
+    }
+    if (pmii_out->fMask & MIIM_FTYPE )
     {
-       if (Unicode)
-       {
-          if (*NewItem == '\b')
-          {
-             mii->fType |= MF_HELP;
-             NewItem++;
-          }
-       }
-       else
-       {
-          LPCSTR NewItemA = (LPCSTR) NewItem;
-          if (*NewItemA == '\b')
-          {
-             mii->fType |= MF_HELP;
-             NewItemA++;
-             NewItem = (LPCWSTR) NewItemA;
-          }
-       }
-
-       if (Flags & MF_HELP)
-         mii->fType |= MF_HELP;
-       mii->fMask |= MIIM_STRING;
-       mii->fType |= MFT_STRING; /* Zero */
-       mii->dwTypeData = (LPWSTR)NewItem;
-       if (Unicode)
-         mii->cch = (NULL == NewItem ? 0 : strlenW(NewItem));
-       else
-         mii->cch = (NULL == NewItem ? 0 : strlen((LPCSTR)NewItem));
+        pmii_out->fType &= ~MENUITEMINFO_TYPE_MASK;
+        pmii_out->fType |= pmii_in->fType & MENUITEMINFO_TYPE_MASK;
     }
-    else
+    if (pmii_out->fMask & MIIM_STATE)
+    /* Other menu items having MFS_DEFAULT are not converted
+       to normal items */
+       pmii_out->fState = pmii_in->fState & MENUITEMINFO_STATE_MASK;
+
+    if (pmii_out->fMask & MIIM_SUBMENU)
     {
-      mii->fType |= MFT_SEPARATOR;
-      if (!(Flags & (MF_GRAYED|MF_DISABLED)))
-        Flags |= MF_GRAYED|MF_DISABLED;
+       if ((pmii_out->hSubMenu != NULL) && !IsMenu(pmii_out->hSubMenu))
+          return FALSE;
     }
-  }
 
-  if(Flags & MF_RIGHTJUSTIFY) /* Same as MF_HELP */
-  {
-    mii->fType |= MFT_RIGHTJUSTIFY;
-  }
+    return TRUE;
+}
 
-  if(Flags & MF_MENUBREAK)
-  {
-    mii->fType |= MFT_MENUBREAK;
-  }
-  else if(Flags & MF_MENUBARBREAK)
-  {
-    mii->fType |= MFT_MENUBARBREAK;
-  }
+BOOL
+MenuInit(VOID)
+{
+  return TRUE;
+}
 
-  if(Flags & MF_GRAYED || Flags & MF_DISABLED)
-  {
-    if (Flags & MF_GRAYED)
-      mii->fState |= MF_GRAYED;
+VOID
+MenuCleanup(VOID)
+{
+}
 
-    if (Flags & MF_DISABLED)
-      mii->fState |= MF_DISABLED;
 
-    mii->fMask |= MIIM_STATE;
-  }
-  else if (Flags & MF_HILITE)
-  {
-    mii->fState |= MF_HILITE;
-    mii->fMask |= MIIM_STATE;
-  }
-  else /* default state */
-  {
-    mii->fState |= MFS_ENABLED;
-    mii->fMask |= MIIM_STATE;
-  }
+NTSTATUS WINAPI
+User32LoadSysMenuTemplateForKernel(PVOID Arguments, ULONG ArgumentLength)
+{
+  LRESULT Result = 0;
 
-  if(Flags & MF_POPUP)
-  {
-    mii->fType |= MF_POPUP;
-    mii->fMask |= MIIM_SUBMENU;
-    mii->hSubMenu = (HMENU)IDNewItem;
-  }
-  else
-  {
-    mii->fMask |= MIIM_ID;
-    mii->wID = (UINT)IDNewItem;
-  }
-  return TRUE;
+  // Use this for Menu Ole!!
+
+  return(ZwCallbackReturn(&Result, sizeof(LRESULT), STATUS_SUCCESS));
 }
 
 NTSTATUS WINAPI
 User32CallLoadMenuFromKernel(PVOID Arguments, ULONG ArgumentLength)
 {
   PLOADMENU_CALLBACK_ARGUMENTS Common;
-  LRESULT Result;  
+  LRESULT Result;
 
   Common = (PLOADMENU_CALLBACK_ARGUMENTS) Arguments;
-  
-  Result = (LRESULT)LoadMenuW( Common->hModule,
-                               IS_INTRESOURCE(Common->MenuName[0]) ?
-                                  MAKEINTRESOURCE(Common->MenuName[0]) :
-                                        (LPCWSTR)&Common->MenuName);
+
+  Result = (LRESULT)LoadMenuW( Common->hModule, Common->InterSource ? MAKEINTRESOURCE(Common->InterSource) : (LPCWSTR)&Common->MenuName);
 
   return ZwCallbackReturn(&Result, sizeof(LRESULT), STATUS_SUCCESS);
 }
@@ -3916,13 +678,6 @@ User32CallLoadMenuFromKernel(PVOID Arguments, ULONG ArgumentLength)
 
 /* FUNCTIONS *****************************************************************/
 
-/*static BOOL
-MenuIsStringItem(ULONG TypeData)
-{
-  return(MF_STRING == MENU_ITEM_TYPE(ItemInfo->fType));
-}*/
-
-
 /*
  * @implemented
  */
@@ -3932,10 +687,34 @@ AppendMenuA(HMENU hMenu,
            UINT_PTR uIDNewItem,
            LPCSTR lpNewItem)
 {
-  return(InsertMenuA(hMenu, -1, uFlags | MF_BYPOSITION, uIDNewItem,
-                    lpNewItem));
-}
+  MENUITEMINFOW mii;
+  UNICODE_STRING UnicodeString;
+  BOOL res;
+
+  RtlInitUnicodeString(&UnicodeString, 0);
+
+  MENU_mnu2mnuii( uFlags, uIDNewItem, (LPCWSTR)lpNewItem, &mii, FALSE);
 
+  /* copy the text string, it will be one or the other */
+  if (lpNewItem && mii.fMask & MIIM_STRING && !mii.hbmpItem && mii.dwTypeData)
+  {
+      if (!RtlCreateUnicodeStringFromAsciiz(&UnicodeString, (LPSTR)mii.dwTypeData))
+      {
+        SetLastError (ERROR_NOT_ENOUGH_MEMORY);
+        return FALSE;
+      }
+      mii.dwTypeData = UnicodeString.Buffer;
+      mii.cch = UnicodeString.Length / sizeof(WCHAR);
+  }
+  else
+  {
+      TRACE("AMA Handle bitmaps\n");
+  }
+  ////// Answer a question, why a -1? To hunt for the end of the item list. Get it, to Append?
+  res = NtUserThunkedMenuItemInfo(hMenu, -1, TRUE, TRUE, &mii, &UnicodeString);
+  if ( UnicodeString.Buffer ) RtlFreeUnicodeString ( &UnicodeString );
+  return res;
+}
 
 /*
  * @implemented
@@ -3946,10 +725,24 @@ AppendMenuW(HMENU hMenu,
            UINT_PTR uIDNewItem,
            LPCWSTR lpNewItem)
 {
-  return(InsertMenuW(hMenu, -1, uFlags | MF_BYPOSITION, uIDNewItem,
-                    lpNewItem));
-}
+  MENUITEMINFOW mii;
+  UNICODE_STRING MenuText;
+  BOOL res;
+
+  RtlInitUnicodeString(&MenuText, 0);    
+
+  MENU_mnu2mnuii( uFlags, uIDNewItem, lpNewItem, &mii, TRUE);
 
+  /* copy the text string, it will be one or the other */
+  if (lpNewItem && mii.fMask & MIIM_STRING && !mii.hbmpItem && mii.dwTypeData)
+  {
+    RtlInitUnicodeString(&MenuText, (PWSTR)mii.dwTypeData);
+    mii.dwTypeData = MenuText.Buffer;
+    mii.cch = MenuText.Length / sizeof(WCHAR);
+  }
+  res = NtUserThunkedMenuItemInfo(hMenu, -1, TRUE, TRUE, &mii, &MenuText);
+  return res;
+}
 
 /*
  * @implemented
@@ -3959,182 +752,110 @@ CheckMenuItem(HMENU hmenu,
              UINT uIDCheckItem,
              UINT uCheck)
 {
-  return NtUserCheckMenuItem(hmenu, uIDCheckItem, uCheck);
-}
-
-static
-BOOL
-MenuCheckMenuRadioItem(HMENU hMenu, UINT idFirst, UINT idLast, UINT idCheck, UINT uFlags, BOOL bCheck, PUINT pChecked, PUINT pUnchecked, PUINT pMenuChanged)
-{
-  UINT ItemCount, i;
-  PROSMENUITEMINFO Items = NULL;
-  UINT cChecked, cUnchecked;
-  BOOL bRet = TRUE;
-  //ROSMENUINFO mi;
-
-  if(idFirst > idLast)
-      return FALSE;
-
-  ItemCount = GetMenuItemCount(hMenu);
-
-  //mi.cbSize = sizeof(ROSMENUINFO);
-  //if(!NtUserMenuInfo(hmenu, &mi, FALSE)) return ret;
-
-
-  if(MenuGetAllRosMenuItemInfo(hMenu, &Items) <= 0)
-  {
-    ERR("MenuGetAllRosMenuItemInfo failed\n");
-    return FALSE;
-  }
-
-  cChecked = cUnchecked = 0;
-
-  for (i = 0 ; i < ItemCount; i++)
-  {
-    BOOL check = FALSE;
-    if (0 != (Items[i].fType & MF_MENUBARBREAK)) continue;
-    if (0 != (Items[i].fType & MF_SEPARATOR)) continue;
-
-    if ((Items[i].fType & MF_POPUP) && (uFlags == MF_BYCOMMAND))
-    {
-      MenuCheckMenuRadioItem(Items[i].hSubMenu, idFirst, idLast, idCheck, uFlags, bCheck, pChecked, pUnchecked, pMenuChanged);
-      continue;
-    }
-    if (uFlags & MF_BYPOSITION)
-    {
-      if (i < idFirst || i > idLast)
-        continue;
-
-      if (i == idCheck)
-      {
-        cChecked++;
-        check = TRUE;
-      }
-      else
-      {
-        cUnchecked++;
-      }
-    }
-      else
-      {
-        if (Items[i].wID < idFirst || Items[i].wID > idLast)
-          continue;
-
-        if (Items[i].wID == idCheck)
-        {
-          cChecked++;
-          check = TRUE;
-        }
-        else
-        {
-          cUnchecked++;
-        }
-      }
-
-      if (!bCheck)
-        continue;
-
-      Items[i].fMask = MIIM_STATE | MIIM_FTYPE;
-      if (check)
-      {
-        Items[i].fType |= MFT_RADIOCHECK;
-        Items[i].fState |= MFS_CHECKED;
-      }
-      else
-      {
-        Items[i].fState &= ~MFS_CHECKED;
-      }
+  PITEM item;
+  DWORD Ret;
+  UINT uID = uIDCheckItem;
 
-      if(!MenuSetRosMenuItemInfo(hMenu, i ,&Items[i]))
-      {
-        ERR("MenuSetRosMenuItemInfo failed\n");
-        bRet = FALSE;
-        break;
-      }
-  }
-  HeapFree(GetProcessHeap(), 0, Items);
+  if (!ValidateHandle(hmenu, TYPE_MENU))
+     return -1;
 
-  *pChecked += cChecked;
-  *pUnchecked += cUnchecked;
+  if (!(item = MENU_FindItem( &hmenu, &uID, uCheck ))) return -1;
 
-  if (cChecked || cUnchecked)
-    (*pMenuChanged)++;
+  Ret = item->fState & MFS_CHECKED;
+  if ( Ret == (uCheck & MFS_CHECKED)) return Ret; // Already Checked...
 
-  return bRet;
+  return NtUserCheckMenuItem(hmenu, uIDCheckItem, uCheck);
 }
 
 /*
  * @implemented
  */
 BOOL WINAPI
-CheckMenuRadioItem(HMENU hmenu,
-                  UINT idFirst,
-                  UINT idLast,
-                  UINT idCheck,
-                  UINT uFlags)
+CheckMenuRadioItem(HMENU hMenu,
+                   UINT first,
+                   UINT last,
+                   UINT check,
+                   UINT bypos)
 {
-  UINT cChecked = 0;
-  UINT cUnchecked = 0;
-  UINT cMenuChanged = 0;
-
-  if (!MenuCheckMenuRadioItem(hmenu, idFirst, idLast, idCheck, uFlags, FALSE, &cChecked, &cUnchecked, &cMenuChanged))
-    return FALSE;
+    BOOL done = FALSE;
+    UINT i;
+    PITEM mi_first = NULL, mi_check;
+    HMENU m_first, m_check;
+    MENUITEMINFOW mii;
+    mii.cbSize = sizeof( mii);
 
-  if (cMenuChanged > 1)
-    return FALSE;
+    for (i = first; i <= last; i++)
+    {
+        UINT pos = i;
 
-  cMenuChanged = 0;
-  cChecked = 0;
-  cUnchecked = 0;
+        if (!mi_first)
+        {
+            m_first = hMenu;
+            mi_first = MENU_FindItem(&m_first, &pos, bypos);
+            if (!mi_first) continue;
+            mi_check = mi_first;
+            m_check = m_first;
+        }
+        else
+        {
+            m_check = hMenu;
+            mi_check = MENU_FindItem(&m_check, &pos, bypos);
+            if (!mi_check) continue;
+        }
 
-  if (!MenuCheckMenuRadioItem(hmenu, idFirst, idLast, idCheck, uFlags, TRUE, &cChecked, &cUnchecked, &cMenuChanged))
-    return FALSE;
+        if (m_first != m_check) continue;
+        if (mi_check->fType == MFT_SEPARATOR) continue;
 
-  return (cChecked != 0);
+        if (i == check)
+        {
+            if (!(mi_check->fType & MFT_RADIOCHECK) || !(mi_check->fState & MFS_CHECKED))
+            {
+               mii.fMask = MIIM_FTYPE | MIIM_STATE;
+               mii.fType = (mi_check->fType & MENUITEMINFO_TYPE_MASK) | MFT_RADIOCHECK;
+               mii.fState = (mi_check->fState & MII_STATE_MASK) | MFS_CHECKED;
+               NtUserThunkedMenuItemInfo(m_check, i, bypos, FALSE, &mii, NULL);
+            }
+            done = TRUE;
+        }
+        else
+        {
+            /* MSDN is wrong, Windows does not remove MFT_RADIOCHECK */
+            if (mi_check->fState & MFS_CHECKED)
+            {
+               mii.fMask = MIIM_STATE;
+               mii.fState = (mi_check->fState & MII_STATE_MASK) & ~MFS_CHECKED;
+               NtUserThunkedMenuItemInfo(m_check, i, bypos, FALSE, &mii, NULL);
+            }
+        }
+    }
+    return done;
 }
 
-
 /*
  * @implemented
  */
 HMENU WINAPI
 CreateMenu(VOID)
 {
-  MenuLoadBitmaps();
   return NtUserxCreateMenu();
 }
 
-
 /*
  * @implemented
  */
 HMENU WINAPI
 CreatePopupMenu(VOID)
 {
-  MenuLoadBitmaps();
   return NtUserxCreatePopupMenu();
 }
 
-
 /*
  * @implemented
  */
 BOOL WINAPI
 DrawMenuBar(HWND hWnd)
 {
-//  return NtUserxDrawMenuBar(hWnd);
-  ROSMENUINFO MenuInfo;
-  HMENU hMenu;
-  hMenu = GetMenu(hWnd);
-  if (!hMenu)
-     return FALSE;
-  MenuGetRosMenuInfo(&MenuInfo, hMenu);
-  MenuInfo.Height = 0; // make sure to recalc size
-  MenuSetRosMenuInfo(&MenuInfo);
-  
-  SetWindowPos( hWnd, 0, 0, 0, 0, 0, SWP_NOSIZE | SWP_NOMOVE |
-                  SWP_NOZORDER | SWP_FRAMECHANGED );
-  return TRUE;
+  return NtUserxDrawMenuBar(hWnd);
 }
 
 /*
@@ -4148,59 +869,6 @@ EnableMenuItem(HMENU hMenu,
   return NtUserEnableMenuItem(hMenu, uIDEnableItem, uEnable);
 }
 
-/*
- * @implemented
- */
-BOOL WINAPI
-EndMenu(VOID)
-{
-  GUITHREADINFO guii;
-  guii.cbSize = sizeof(GUITHREADINFO);
-  if(GetGUIThreadInfo(GetCurrentThreadId(), &guii) && guii.hwndMenuOwner)
-  {
-    if (!fEndMenu &&
-         top_popup &&
-         guii.hwndMenuOwner != top_popup )
-    {
-       ERR("Capture GUI pti hWnd does not match top_popup!\n");
-    }
-  }
-
-  /* if we are in the menu code, and it is active */
-  if (!fEndMenu && top_popup)
-  {
-      /* terminate the menu handling code */
-      fEndMenu = TRUE;
-
-      /* needs to be posted to wakeup the internal menu handler */
-      /* which will now terminate the menu, in the event that */
-      /* the main window was minimized, or lost focus, so we */
-      /* don't end up with an orphaned menu */
-      PostMessageW( top_popup, WM_CANCELMODE, 0, 0);
-  }
-  return TRUE;
-}
-
-// So this one maybe one day it will be a callback!
-BOOL WINAPI HiliteMenuItem( HWND hWnd, HMENU hMenu, UINT wItemID,
-                                UINT wHilite )
-{
-    ROSMENUINFO MenuInfo;
-    ROSMENUITEMINFO mii;
-    TRACE("(%p, %p, %04x, %04x);\n", hWnd, hMenu, wItemID, wHilite);
-    if (!hWnd)
-    {
-       SetLastError(ERROR_INVALID_WINDOW_HANDLE);
-       return FALSE;
-    }
-    if (!NtUserMenuItemInfo(hMenu, wItemID, wHilite, &mii, FALSE)) return FALSE;
-    if (!NtUserMenuInfo(hMenu, &MenuInfo, FALSE)) return FALSE;
-    if (MenuInfo.FocusedItem == wItemID) return TRUE;
-    MenuHideSubPopups( hWnd, &MenuInfo, FALSE, 0 );
-    MenuSelectItem( hWnd, &MenuInfo, wItemID, TRUE, 0 );
-    return TRUE;
-}
-
 /*
  * @implemented
  */
@@ -4215,38 +883,29 @@ GetMenu(HWND hWnd)
        return UlongToHandle(Wnd->IDMenu);
 }
 
-
 /*
  * @implemented
  */
-BOOL WINAPI GetMenuBarInfo( HWND hwnd, LONG idObject, LONG idItem, PMENUBARINFO pmbi )
+LONG WINAPI
+GetMenuCheckMarkDimensions(VOID)
 {
-    BOOL Ret;
-    Ret = NtUserGetMenuBarInfo( hwnd, idObject, idItem, pmbi);
-    // Reason to move to server side!!!!!
-    if (!Ret) return Ret;
-    // EL HAXZO!!!
-    pmbi->fBarFocused = top_popup_hmenu == pmbi->hMenu;
-    if (!idItem)
-    {
-        pmbi->fFocused = pmbi->fBarFocused;
-    }
-
-    return TRUE;
+  return(MAKELONG(GetSystemMetrics(SM_CXMENUCHECK),
+                 GetSystemMetrics(SM_CYMENUCHECK)));
 }
 
-
 /*
  * @implemented
  */
-LONG WINAPI
-GetMenuCheckMarkDimensions(VOID)
+DWORD
+WINAPI
+GetMenuContextHelpId(HMENU hmenu)
 {
-  return(MAKELONG(GetSystemMetrics(SM_CXMENUCHECK),
-                 GetSystemMetrics(SM_CYMENUCHECK)));
+  PMENU pMenu;
+  if ((pMenu = ValidateHandle(hmenu, TYPE_MENU)))
+     return pMenu->dwContextHelpId;
+  return 0;
 }
 
-
 /*
  * @implemented
  */
@@ -4255,9 +914,13 @@ GetMenuDefaultItem(HMENU hMenu,
                   UINT fByPos,
                   UINT gmdiFlags)
 {
-  return NtUserGetMenuDefaultItem(hMenu, fByPos, gmdiFlags);
-}
+  PMENU pMenu;
+  DWORD gismc = 0;
+  if (!(pMenu = ValidateHandle(hMenu, TYPE_MENU)))
+       return (UINT)-1;
 
+  return IntGetMenuDefaultItem( pMenu, (BOOL)fByPos, gmdiFlags, &gismc);
+}
 
 /*
  * @implemented
@@ -4266,216 +929,116 @@ BOOL WINAPI
 GetMenuInfo(HMENU hmenu,
            LPMENUINFO lpcmi)
 {
-  ROSMENUINFO mi;
-  BOOL res = FALSE;
-
-  if(!lpcmi || (lpcmi->cbSize != sizeof(MENUINFO)))
-    return FALSE;
-
-  RtlZeroMemory(&mi, sizeof(MENUINFO));
-  mi.cbSize = sizeof(MENUINFO);
-  mi.fMask = lpcmi->fMask;
-
-  res = NtUserMenuInfo(hmenu, &mi, FALSE);
-
-  memcpy(lpcmi, &mi, sizeof(MENUINFO));
-  return res;
-}
+  PMENU pMenu;
 
+  if (!lpcmi || (lpcmi->cbSize != sizeof(MENUINFO)))
+  {
+     SetLastError(ERROR_INVALID_PARAMETER);
+     return FALSE;
+  }
 
-/*
- * @implemented
- */
-int WINAPI
-GetMenuItemCount(HMENU Menu)
-{
-  ROSMENUINFO MenuInfo;
-
-  return MenuGetRosMenuInfo(&MenuInfo, Menu) ? MenuInfo.MenuItemCount : 0;
-}
+  if (!(pMenu = ValidateHandle(hmenu, TYPE_MENU)))
+     return FALSE;
 
+  if (lpcmi->fMask & MIM_BACKGROUND)
+      lpcmi->hbrBack = pMenu->hbrBack;
 
-/*
- * @implemented
- */
-UINT WINAPI
-GetMenuItemID(HMENU hMenu,
-             int nPos)
-{
-  ROSMENUITEMINFO mii;
+  if (lpcmi->fMask & MIM_HELPID)
+      lpcmi->dwContextHelpID = pMenu->dwContextHelpId;
 
-  mii.cbSize = sizeof(MENUITEMINFOW);
-  mii.fMask = MIIM_ID | MIIM_SUBMENU;
+  if (lpcmi->fMask & MIM_MAXHEIGHT)
+      lpcmi->cyMax = pMenu->cyMax;
 
-  if (! NtUserMenuItemInfo(hMenu, nPos, MF_BYPOSITION, &mii, FALSE))
-    {
-      return -1;
-    }
+  if (lpcmi->fMask & MIM_MENUDATA)
+      lpcmi->dwMenuData = pMenu->dwMenuData;
 
-  if (NULL != mii.hSubMenu)
-    {
-      return -1;
-    }
-  if (0 == mii.wID)
-    {
-      return -1;
-    }
+  if (lpcmi->fMask & MIM_STYLE)
+      lpcmi->dwStyle = pMenu->fFlags & MNS_STYLE_MASK;
 
-  return mii.wID;
+  return TRUE;
 }
 
-
 /*
  * @implemented
  */
-BOOL WINAPI
-GetMenuItemInfoA(
-   HMENU Menu,
-   UINT Item,
-   BOOL ByPosition,
-   LPMENUITEMINFOA mii)
-{
-   MENUITEMINFOW miiW;
-   LPSTR AnsiBuffer;
-   INT Count;
-
-   if (mii->cbSize != sizeof(MENUITEMINFOA) &&
-       mii->cbSize != sizeof(MENUITEMINFOA) - sizeof(HBITMAP))
-   {
-      SetLastError(ERROR_INVALID_PARAMETER);
-      return FALSE;
-   }
-
-   if(!(mii->fMask & (MIIM_TYPE | MIIM_STRING)))
-   {
-      /* No text requested, just pass on */
-      return NtUserMenuItemInfo(Menu, Item, ByPosition, (PROSMENUITEMINFO) mii, FALSE);
-   }
-
-   AnsiBuffer = mii->dwTypeData;
-   Count = miiW.cch = mii->cch;
-   RtlCopyMemory(&miiW, mii, mii->cbSize);
-   miiW.dwTypeData = 0;
-
-   if (AnsiBuffer)
-   {
-      miiW.dwTypeData = RtlAllocateHeap(GetProcessHeap(), 0,
-                                        miiW.cch * sizeof(WCHAR));
-      if (miiW.dwTypeData == NULL) return FALSE;
-      miiW.dwTypeData[0] = 0;
-   }
-
-   if (!NtUserMenuItemInfo(Menu, Item, ByPosition, (PROSMENUITEMINFO)&miiW, FALSE))
-   {
-      if (miiW.dwTypeData) RtlFreeHeap(GetProcessHeap(), 0, miiW.dwTypeData);
-      return FALSE;
-   }
-
-   RtlCopyMemory(mii, &miiW, miiW.cbSize);
-
-   if (!AnsiBuffer || !Count)
-   {
-      if (miiW.dwTypeData) RtlFreeHeap(GetProcessHeap(), 0, miiW.dwTypeData);
-      mii->dwTypeData = AnsiBuffer;
-      mii->cch = miiW.cch;
-      return TRUE;
-   }
-
-   if ((miiW.fMask & MIIM_STRING) || (IS_STRING_ITEM(miiW.fType)))
-   {
-      if (miiW.cch)
-      {
-         if (!WideCharToMultiByte(CP_ACP, 0, miiW.dwTypeData, miiW.cch, AnsiBuffer, mii->cch, NULL, NULL))
-         {
-            AnsiBuffer[0] = 0;
-         }
-         if (Count > miiW.cch)
-         {
-            AnsiBuffer[miiW.cch] = 0;
-         }
-         mii->cch = miiW.cch;
-      }
-   }
-   else
-   {
-      AnsiBuffer[0] = 0;
-   }
-
-   RtlFreeHeap(GetProcessHeap(), 0, miiW.dwTypeData);
-   mii->dwTypeData = AnsiBuffer;
+int WINAPI
+GetMenuItemCount(HMENU hmenu)
+{
+  PMENU pMenu;
+  if ((pMenu = ValidateHandle(hmenu, TYPE_MENU)))
+     return pMenu->cItems;
+  return -1;
+}
 
-   return TRUE;
+/*
+ * @implemented
+ */
+UINT WINAPI
+GetMenuItemID(HMENU hMenu,
+             int nPos)
+{
+  ITEM * lpmi;
+  if (!(lpmi = MENU_FindItem(&hMenu,(UINT*)&nPos,MF_BYPOSITION))) return -1;
+  if (lpmi->spSubMenu) return -1;
+  return lpmi->wID;
 }
 
+/*
+ * @implemented
+ */
+BOOL WINAPI
+GetMenuItemInfoA(
+   HMENU hmenu,
+   UINT item,
+   BOOL bypos,
+   LPMENUITEMINFOA lpmii)
+{
+    BOOL ret;
+    MENUITEMINFOA mii;
+
+    if( lpmii->cbSize != sizeof( mii) &&
+        lpmii->cbSize != sizeof( mii) - sizeof ( mii.hbmpItem))
+    {
+        SetLastError( ERROR_INVALID_PARAMETER);
+        return FALSE;
+    }
+    memcpy( &mii, lpmii, lpmii->cbSize);
+    mii.cbSize = sizeof( mii);
+    ret = GetMenuItemInfo_common (hmenu,
+                                  item,
+                                  bypos,
+                                  (LPMENUITEMINFOW)&mii,
+                                  FALSE);
+    mii.cbSize = lpmii->cbSize;
+    memcpy( lpmii, &mii, mii.cbSize);
+    return ret;
+}
 
 /*
  * @implemented
  */
 BOOL WINAPI
 GetMenuItemInfoW(
-   HMENU Menu,
+   HMENU hMenu,
    UINT Item,
-   BOOL ByPosition,
-   LPMENUITEMINFOW mii)
+   BOOL bypos,
+   LPMENUITEMINFOW lpmii)
 {
-   MENUITEMINFOW miiW;
-   LPWSTR String;
-   INT Count;
-
-   if (mii->cbSize != sizeof(MENUITEMINFOW) &&
-       mii->cbSize != sizeof(MENUITEMINFOW) - sizeof(HBITMAP))
-   {
-      SetLastError(ERROR_INVALID_PARAMETER);
-      return FALSE;
-   }
-
-   if(!(mii->fMask & (MIIM_TYPE | MIIM_STRING)))
-   {
-      /* No text requested, just pass on */
-      return NtUserMenuItemInfo(Menu, Item, ByPosition, (PROSMENUITEMINFO) mii, FALSE);
-   }
-
-   String = mii->dwTypeData;
-   Count = mii->cch;
-   RtlCopyMemory(&miiW, mii, mii->cbSize);
-   miiW.dwTypeData = 0;
-
-   if (String)
+   BOOL ret;
+   MENUITEMINFOW mii;
+   if( lpmii->cbSize != sizeof( mii) && lpmii->cbSize != sizeof( mii) - sizeof ( mii.hbmpItem))
    {
-      miiW.dwTypeData = RtlAllocateHeap(GetProcessHeap(), 0,
-                                        miiW.cch * sizeof(WCHAR));
-      if (miiW.dwTypeData == NULL) return FALSE;
-      miiW.dwTypeData[0] = 0;
-   }
-
-   if (!NtUserMenuItemInfo(Menu, Item, ByPosition, (PROSMENUITEMINFO) &miiW, FALSE))
-   {
-      if (miiW.dwTypeData) RtlFreeHeap(GetProcessHeap(), 0, miiW.dwTypeData);
+      SetLastError( ERROR_INVALID_PARAMETER);
       return FALSE;
    }
-
-   RtlCopyMemory(mii, &miiW, miiW.cbSize); // Okay to over write user data.
-
-   if (!String || !Count)
-   {
-      if (miiW.dwTypeData) RtlFreeHeap(GetProcessHeap(), 0, miiW.dwTypeData);
-      mii->dwTypeData = String; // may not be zero.
-      mii->cch = miiW.cch;
-      return TRUE;
-   }
-
-   if ((miiW.fMask & MIIM_STRING) || (IS_STRING_ITEM(miiW.fType)))
-   {
-      lstrcpynW( String, miiW.dwTypeData, Count );
-   }
-
-   RtlFreeHeap(GetProcessHeap(), 0, miiW.dwTypeData);
-   mii->dwTypeData = String;
-   mii->cch = strlenW(String);
-   return TRUE;
+   memcpy( &mii, lpmii, lpmii->cbSize);
+   mii.cbSize = sizeof( mii);
+   ret = GetMenuItemInfo_common (hMenu, Item, bypos, &mii, TRUE);
+   mii.cbSize = lpmii->cbSize;
+   memcpy( lpmii, &mii, mii.cbSize);
+   return ret;
 }
 
-
 /*
  * @implemented
  */
@@ -4486,39 +1049,24 @@ GetMenuState(
   UINT uId,
   UINT uFlags)
 {
-  ROSMENUINFO MenuInfo;
-  ROSMENUITEMINFO mii;
-  memset( &mii, 0, sizeof(mii) );
-  mii.cbSize = sizeof(MENUITEMINFOW);
-  mii.fMask = MIIM_STATE | MIIM_FTYPE | MIIM_SUBMENU;
-
-  SetLastError(0);
-  if(NtUserMenuItemInfo(hMenu, uId, uFlags, &mii, FALSE))
-    {
-      UINT nSubItems = 0;
-      if(mii.hSubMenu)
-        {
-          if (! MenuGetRosMenuInfo(&MenuInfo, mii.hSubMenu))
-            {
-              return (UINT) -1;
-            }
-          nSubItems = MenuInfo.MenuItemCount;
+  PITEM pItem;
+  UINT Type = 0;
+  TRACE("(menu=%p, id=%04x, flags=%04x);\n", hMenu, uId, uFlags);
+  if (!(pItem = MENU_FindItem( &hMenu, &uId, uFlags ))) return -1;
 
-          /* FIXME - ported from wine, does that work (0xff)? */
-          if(GetLastError() != ERROR_INVALID_MENU_HANDLE)
-            return (nSubItems << 8) | ((mii.fState | mii.fType) & 0xff);
+  if (!pItem->Xlpstr && pItem->hbmp) Type = MFT_BITMAP;
 
-          return (UINT)-1; /* Invalid submenu */
-        }
-
-      /* FIXME - ported from wine, does that work? */
-      return (mii.fType | mii.fState);
-    }
-
-  return (UINT)-1;
+  if (pItem->spSubMenu)
+  {
+     PMENU pSubMenu = DesktopPtrToUser(pItem->spSubMenu);
+     HMENU hsubmenu = UserHMGetHandle(pSubMenu);
+     if (!IsMenu(hsubmenu)) return (UINT)-1;
+     else return (pSubMenu->cItems << 8) | ((pItem->fState|pItem->fType|Type) & 0xff);
+  }
+  else
+     return (pItem->fType | pItem->fState | Type);  
 }
 
-
 /*
  * @implemented
  */
@@ -4531,20 +1079,28 @@ GetMenuStringA(
   int nMaxCount,
   UINT uFlag)
 {
-  MENUITEMINFOA mii;
-  memset( &mii, 0, sizeof(mii) );
-  mii.dwTypeData = lpString;
-  mii.fMask = MIIM_STRING | MIIM_FTYPE;
-  mii.fType = MFT_STRING;
-  mii.cbSize = sizeof(MENUITEMINFOA);
-  mii.cch = nMaxCount;
-
-  if(!(GetMenuItemInfoA( hMenu, uIDItem, (BOOL)(MF_BYPOSITION & uFlag),&mii)))
-     return 0;
-  else
-     return mii.cch;
-}
+  ITEM *item;
+  LPWSTR text;
+  ////// wine Code, seems to be faster.
+  TRACE("menu=%p item=%04x ptr=%p len=%d flags=%04x\n", hMenu, uIDItem, lpString, nMaxCount, uFlag );
+
+  if (lpString && nMaxCount) lpString[0] = '\0';
+
+  if (!(item = MENU_FindItem( &hMenu, &uIDItem, uFlag )))
+  {
+      SetLastError( ERROR_MENU_ITEM_NOT_FOUND);
+      return 0;
+  }
+
+  text = item->Xlpstr ? DesktopPtrToUser(item->Xlpstr) : NULL;
 
+  if (!text) return 0;
+  if (!lpString || !nMaxCount) return WideCharToMultiByte( CP_ACP, 0, text, -1, NULL, 0, NULL, NULL );
+  if (!WideCharToMultiByte( CP_ACP, 0, text, -1, lpString, nMaxCount, NULL, NULL ))
+      lpString[nMaxCount-1] = 0;
+  TRACE("A returning %s\n", lpString);
+  return strlen(lpString);
+}
 
 /*
  * @implemented
@@ -4558,20 +1114,31 @@ GetMenuStringW(
   int nMaxCount,
   UINT uFlag)
 {
-  MENUITEMINFOW miiW;
-  memset( &miiW, 0, sizeof(miiW) );
-  miiW.dwTypeData = lpString;
-  miiW.fMask = MIIM_STRING | MIIM_FTYPE;
-  miiW.fType = MFT_STRING;
-  miiW.cbSize = sizeof(MENUITEMINFOW);
-  miiW.cch = nMaxCount;
-
-  if(!(GetMenuItemInfoW( hMenu, uIDItem, (BOOL)(MF_BYPOSITION & uFlag),&miiW)))
-     return 0;
-  else
-     return miiW.cch;
-}
+  ITEM *item;
+  LPWSTR text;
+
+  TRACE("menu=%p item=%04x ptr=%p len=%d flags=%04x\n", hMenu, uIDItem, lpString, nMaxCount, uFlag );
+
+  if (lpString && nMaxCount) lpString[0] = '\0';
+
+  if (!(item = MENU_FindItem( &hMenu, &uIDItem, uFlag )))
+  {
+      SetLastError( ERROR_MENU_ITEM_NOT_FOUND);
+      return 0;
+  }
+
+  text = item->Xlpstr ? DesktopPtrToUser(item->Xlpstr) : NULL;
 
+  if (!lpString || !nMaxCount) return text ? strlenW(text) : 0;
+  if( !(text))
+  {
+      lpString[0] = 0;
+      return 0;
+  }
+  lstrcpynW( lpString, text, nMaxCount );
+  TRACE("W returning %S\n", lpString);
+  return strlenW(lpString);
+}
 
 /*
  * @implemented
@@ -4582,16 +1149,15 @@ GetSubMenu(
   HMENU hMenu,
   int nPos)
 {
-  ROSMENUITEMINFO mi;
-
-  mi.cbSize = sizeof(MENUITEMINFOW);
-  mi.fMask = MIIM_SUBMENU;
-
-  if (NtUserMenuItemInfo(hMenu, (UINT)nPos, MF_BYPOSITION, &mi, FALSE))
-    {
-      return IsMenu(mi.hSubMenu) ? mi.hSubMenu : NULL;
-    }
+  PITEM pItem;
+  if (!(pItem = MENU_FindItem( &hMenu, (UINT*)&nPos, MF_BYPOSITION ))) return NULL;
 
+  if (pItem->spSubMenu)
+  {
+     PMENU pSubMenu = DesktopPtrToUser(pItem->spSubMenu);
+     HMENU hsubmenu = UserHMGetHandle(pSubMenu);
+     if (IsMenu(hsubmenu)) return hsubmenu;
+  }
   return NULL;
 }
 
@@ -4600,18 +1166,11 @@ GetSubMenu(
  */
 HMENU
 WINAPI
-GetSystemMenu(
-  HWND hWnd,
-  BOOL bRevert)
+GetSystemMenu(HWND hWnd, BOOL bRevert)
 {
-  HMENU TopMenu;
-
-  TopMenu = NtUserGetSystemMenu(hWnd, bRevert);
-
-  return NULL == TopMenu ? NULL : GetSubMenu(TopMenu, 0);
+    return NtUserGetSystemMenu(hWnd, bRevert);
 }
 
-
 /*
  * @implemented
  */
@@ -4624,21 +1183,33 @@ InsertMenuA(
   UINT_PTR uIDNewItem,
   LPCSTR lpNewItem)
 {
-  MENUITEMINFOA mii;
-  memset( &mii, 0, sizeof(mii) );
-  mii.cbSize = sizeof(MENUITEMINFOA);
-  mii.fMask = MIIM_FTYPE;
-
-  MenuSetItemData((LPMENUITEMINFOW) &mii,
-                  uFlags,
-                  uIDNewItem,
-                 (LPCWSTR) lpNewItem,
-                  FALSE);
-
-  return InsertMenuItemA(hMenu, uPosition, (BOOL)((MF_BYPOSITION & uFlags) > 0), &mii);
-}
+  MENUITEMINFOW mii;
+  UNICODE_STRING UnicodeString;
+  BOOL res;
 
+  RtlInitUnicodeString(&UnicodeString, 0);
 
+  MENU_mnu2mnuii( uFlags, uIDNewItem, (LPCWSTR)lpNewItem, &mii, FALSE);
+
+  /* copy the text string, it will be one or the other */
+  if (lpNewItem && mii.fMask & MIIM_STRING && !mii.hbmpItem && mii.dwTypeData)
+  {
+      if (!RtlCreateUnicodeStringFromAsciiz(&UnicodeString, (LPSTR)mii.dwTypeData))
+      {
+        SetLastError (ERROR_NOT_ENOUGH_MEMORY);
+        return FALSE;
+      }
+      mii.dwTypeData = UnicodeString.Buffer;
+      mii.cch = UnicodeString.Length / sizeof(WCHAR);
+  }
+  else
+  {
+      TRACE("Handle bitmaps\n");
+  }
+  res = NtUserThunkedMenuItemInfo(hMenu, uPosition, (BOOL)(uFlags & MF_BYPOSITION), TRUE, &mii, &UnicodeString);
+  if ( UnicodeString.Buffer ) RtlFreeUnicodeString ( &UnicodeString );
+  return res;
+}
 
 /*
  * @implemented
@@ -4651,43 +1222,38 @@ InsertMenuItemA(
   BOOL fByPosition,
   LPCMENUITEMINFOA lpmii)
 {
-  MENUITEMINFOW mi;
-  UNICODE_STRING MenuText;
-  BOOL res = FALSE;
-  BOOL CleanHeap = FALSE;
+  MENUITEMINFOW mii;
+  UNICODE_STRING UnicodeString;
+  BOOL res;
 
-  if((lpmii->cbSize == sizeof(MENUITEMINFOA)) ||
-     (lpmii->cbSize == sizeof(MENUITEMINFOA) - sizeof(HBITMAP)))
-  {
-    RtlCopyMemory ( &mi, lpmii, lpmii->cbSize );
+  TRACE("hmenu %p, item %04x, by pos %d, info %p\n", hMenu, uItem, fByPosition, lpmii);
 
-    if( lpmii->cbSize != sizeof( MENUITEMINFOW))
-    {
-       mi.cbSize = sizeof( MENUITEMINFOW);
-       mi.hbmpItem = NULL;
-    }
-    /* copy the text string */
-    if (((mi.fMask & MIIM_STRING) ||
-        ((mi.fMask & MIIM_TYPE) && (MENU_ITEM_TYPE(mi.fType) == MF_STRING)))
-          && mi.dwTypeData != NULL)
-    {
-      if (!RtlCreateUnicodeStringFromAsciiz(&MenuText, (LPSTR)mi.dwTypeData))
+  RtlInitUnicodeString(&UnicodeString, 0);
+
+  if (!MENU_NormalizeMenuItemInfoStruct( (const MENUITEMINFOW *)lpmii, &mii )) return FALSE;
+
+  /* copy the text string */
+  if (((mii.fMask & MIIM_STRING) ||
+      ((mii.fMask & MIIM_TYPE) && (MENU_ITEM_TYPE(mii.fType) == MF_STRING)))
+        && mii.dwTypeData && !(GdiValidateHandle((HGDIOBJ)mii.dwTypeData)) )
+  {
+      if (!RtlCreateUnicodeStringFromAsciiz(&UnicodeString, (LPSTR)mii.dwTypeData))
       {
         SetLastError (ERROR_NOT_ENOUGH_MEMORY);
         return FALSE;
       }
-      mi.dwTypeData = MenuText.Buffer;
-      mi.cch = MenuText.Length / sizeof(WCHAR);
-      CleanHeap = TRUE;
-    }
-    res = NtUserThunkedMenuItemInfo(hMenu, uItem, fByPosition, TRUE, &mi, NULL);
-
-    if ( CleanHeap ) RtlFreeUnicodeString ( &MenuText );
+      mii.dwTypeData = UnicodeString.Buffer;
+      mii.cch = UnicodeString.Length / sizeof(WCHAR);
   }
+  else
+  {
+      TRACE("Handle bitmaps\n");
+  }
+  res = NtUserThunkedMenuItemInfo(hMenu, uItem, fByPosition, TRUE, &mii, &UnicodeString);
+  if ( UnicodeString.Buffer ) RtlFreeUnicodeString ( &UnicodeString );
   return res;
 }
 
-
 /*
  * @implemented
  */
@@ -4699,7 +1265,7 @@ InsertMenuItemW(
   BOOL fByPosition,
   LPCMENUITEMINFOW lpmii)
 {
-  MENUITEMINFOW mi;
+  MENUITEMINFOW mii;
   UNICODE_STRING MenuText;
   BOOL res = FALSE;
 
@@ -4707,31 +1273,25 @@ InsertMenuItemW(
      if a bad user passes bad data, we crash his process instead of the
      entire kernel */
 
-  if((lpmii->cbSize == sizeof(MENUITEMINFOW)) ||
-     (lpmii->cbSize == sizeof(MENUITEMINFOW) - sizeof(HBITMAP)))
-  {
-    RtlCopyMemory(&mi, lpmii, lpmii->cbSize);
+  TRACE("hmenu %p, item %04x, by pos %d, info %p\n", hMenu, uItem, fByPosition, lpmii);
 
-    if( lpmii->cbSize != sizeof( MENUITEMINFOW))
-    {
-       mi.cbSize = sizeof( MENUITEMINFOW);
-       mi.hbmpItem = NULL;
-    }
-    /* copy the text string */
-    if (((mi.fMask & MIIM_STRING) ||
-        ((mi.fMask & MIIM_TYPE) && (MENU_ITEM_TYPE(mi.fType) == MF_STRING)))
-          && mi.dwTypeData != NULL)
-    {
-      RtlInitUnicodeString(&MenuText, (PWSTR)lpmii->dwTypeData);
-      mi.dwTypeData = MenuText.Buffer;
-      mi.cch = MenuText.Length / sizeof(WCHAR);
-    }
-    res = NtUserThunkedMenuItemInfo(hMenu, uItem, fByPosition, TRUE, &mi, NULL);
+  RtlInitUnicodeString(&MenuText, 0);    
+
+  if (!MENU_NormalizeMenuItemInfoStruct( (const MENUITEMINFOW *)lpmii, &mii )) return FALSE;
+
+  /* copy the text string */
+  if (((mii.fMask & MIIM_STRING) ||
+      ((mii.fMask & MIIM_TYPE) && (MENU_ITEM_TYPE(mii.fType) == MF_STRING)))
+        && mii.dwTypeData && !(GdiValidateHandle((HGDIOBJ)mii.dwTypeData)) )
+  {
+    RtlInitUnicodeString(&MenuText, (PWSTR)lpmii->dwTypeData);
+    mii.dwTypeData = MenuText.Buffer;
+    mii.cch = MenuText.Length / sizeof(WCHAR);
   }
+  res = NtUserThunkedMenuItemInfo(hMenu, uItem, fByPosition, TRUE, &mii, &MenuText);
   return res;
 }
 
-
 /*
  * @implemented
  */
@@ -4745,19 +1305,23 @@ InsertMenuW(
   LPCWSTR lpNewItem)
 {
   MENUITEMINFOW mii;
-  memset( &mii, 0, sizeof(mii) );
-  mii.cbSize = sizeof(MENUITEMINFOW);
-  mii.fMask = MIIM_FTYPE;
+  UNICODE_STRING MenuText;
+  BOOL res;
 
-  MenuSetItemData( &mii,
-                  uFlags,
-                  uIDNewItem,
-                  lpNewItem,
-                  TRUE);
+  RtlInitUnicodeString(&MenuText, 0);    
 
-  return InsertMenuItemW(hMenu, uPosition, (BOOL)((MF_BYPOSITION & uFlags) > 0), &mii);
-}
+  MENU_mnu2mnuii( uFlags, uIDNewItem, lpNewItem, &mii, TRUE);
 
+  /* copy the text string, it will be one or the other */
+  if (lpNewItem && mii.fMask & MIIM_STRING && !mii.hbmpItem && mii.dwTypeData)
+  {
+    RtlInitUnicodeString(&MenuText, (PWSTR)mii.dwTypeData);
+    mii.dwTypeData = MenuText.Buffer;
+    mii.cch = MenuText.Length / sizeof(WCHAR);
+  }
+  res = NtUserThunkedMenuItemInfo(hMenu, uPosition, (BOOL)(uFlags & MF_BYPOSITION), TRUE, &mii, &MenuText);
+  return res;
+}
 
 /*
  * @implemented
@@ -4771,7 +1335,6 @@ IsMenu(
   return FALSE;
 }
 
-
 /*
  * @implemented
  */
@@ -4787,7 +1350,6 @@ LoadMenuA(HINSTANCE hInstance,
   return(LoadMenuIndirectA((PVOID)LoadResource(hInstance, Resource)));
 }
 
-
 /*
  * @implemented
  */
@@ -4797,7 +1359,6 @@ LoadMenuIndirectA(CONST MENUTEMPLATE *lpMenuTemplate)
   return(LoadMenuIndirectW(lpMenuTemplate));
 }
 
-
 /*
  * @implemented
  */
@@ -4817,7 +1378,7 @@ LoadMenuIndirectW(CONST MENUTEMPLATE *lpMenuTemplate)
       offset = GET_WORD(p);
       p += sizeof(WORD) + offset;
       if (!(hMenu = CreateMenu())) return 0;
-      if (!MENU_ParseResource(p, hMenu, TRUE))
+      if (!MENU_ParseResource(p, hMenu))
       {
         DestroyMenu(hMenu);
         return 0;
@@ -4839,7 +1400,6 @@ LoadMenuIndirectW(CONST MENUTEMPLATE *lpMenuTemplate)
   }
 }
 
-
 /*
  * @implemented
  */
@@ -4855,117 +1415,81 @@ LoadMenuW(HINSTANCE hInstance,
   return(LoadMenuIndirectW((PVOID)LoadResource(hInstance, Resource)));
 }
 
-
-/*
- * @implemented
- */
-int
-WINAPI
-MenuItemFromPoint(
-  HWND hWnd,
-  HMENU hMenu,
-  POINT ptScreen)
-{
-  return NtUserMenuItemFromPoint(hWnd, hMenu, ptScreen.x, ptScreen.y);
-}
-
-
 /*
  * @implemented
  */
 BOOL
 WINAPI
 ModifyMenuA(
-  HMENU hMnu,
+  HMENU hMenu,
   UINT uPosition,
   UINT uFlags,
   UINT_PTR uIDNewItem,
   LPCSTR lpNewItem)
 {
-  ROSMENUINFO mi;
-  ROSMENUITEMINFO rmii;
-  MENUITEMINFOA mii;
-  memset( &mii, 0, sizeof(mii) );
-  mii.cbSize = sizeof(MENUITEMINFOA);
-  mii.fMask = MIIM_FTYPE;
-
-  if (!MenuGetRosMenuInfo( &mi, hMnu )) return FALSE;
-
-  mi.Height = 0;
-
-  if (!MenuSetRosMenuInfo( &mi )) return FALSE;
-
-  MenuInitRosMenuItemInfo( &rmii );
-
-  if(!MenuGetRosMenuItemInfo( hMnu, uPosition, &rmii)) return FALSE;
-
-  if ((rmii.fType & MF_POPUP) && (uFlags & MF_POPUP) && (rmii.hSubMenu != (HMENU)uIDNewItem))
-    NtUserDestroyMenu( rmii.hSubMenu );   /* ModifyMenu() spec */
+  MENUITEMINFOW mii;
+  UNICODE_STRING UnicodeString;
+  BOOL res;
 
-  MenuCleanupRosMenuItemInfo( &rmii );
+  RtlInitUnicodeString(&UnicodeString, 0);
 
-  MenuSetItemData((LPMENUITEMINFOW) &mii,
-                  uFlags,
-                  uIDNewItem,
-                 (LPCWSTR) lpNewItem,
-                  FALSE);
+  MENU_mnu2mnuii( uFlags, uIDNewItem, (LPCWSTR)lpNewItem, &mii, FALSE);
 
-  return SetMenuItemInfoA( hMnu,
-                           uPosition,
-                          (BOOL)(MF_BYPOSITION & uFlags),
-                           &mii);
+  /* copy the text string, it will be one or the other */
+  if (lpNewItem && mii.fMask & MIIM_STRING && !mii.hbmpItem && mii.dwTypeData)
+  {
+      if (!RtlCreateUnicodeStringFromAsciiz(&UnicodeString, (LPSTR)mii.dwTypeData))
+      {
+        SetLastError (ERROR_NOT_ENOUGH_MEMORY);
+        return FALSE;
+      }
+      mii.dwTypeData = UnicodeString.Buffer;
+      mii.cch = UnicodeString.Length / sizeof(WCHAR);
+  }
+  else
+  {
+      TRACE("Handle bitmaps\n");
+  }
+  res = NtUserThunkedMenuItemInfo(hMenu, uPosition, (BOOL)(uFlags & MF_BYPOSITION), FALSE, &mii, &UnicodeString);
+  if ( UnicodeString.Buffer ) RtlFreeUnicodeString ( &UnicodeString );
+  return res;
 }
 
-
 /*
  * @implemented
  */
 BOOL
 WINAPI
 ModifyMenuW(
-  HMENU hMnu,
+  HMENU hMenu,
   UINT uPosition,
   UINT uFlags,
   UINT_PTR uIDNewItem,
   LPCWSTR lpNewItem)
 {
-  ROSMENUINFO mi;
-  ROSMENUITEMINFO rmii;
   MENUITEMINFOW mii;
-  memset ( &mii, 0, sizeof(mii) );
-  mii.cbSize = sizeof(MENUITEMINFOW);
-  mii.fMask = MIIM_FTYPE;
-
-  if (!MenuGetRosMenuInfo( &mi, hMnu )) return FALSE;
-
-  mi.Height = 0; // Force size recalculation.
-
-  if (!MenuSetRosMenuInfo( &mi )) return FALSE;
-
-  MenuInitRosMenuItemInfo( &rmii );
-
-  if(!MenuGetRosMenuItemInfo( hMnu, uPosition, &rmii)) return FALSE;
-
-  if ((rmii.fType & MF_POPUP) && (uFlags & MF_POPUP) && (rmii.hSubMenu != (HMENU)uIDNewItem))
-    NtUserDestroyMenu( rmii.hSubMenu );   /* ModifyMenu() spec */
+  UNICODE_STRING MenuText;
+  BOOL res;
 
-  MenuCleanupRosMenuItemInfo( &rmii );
+  RtlInitUnicodeString(&MenuText, 0);    
 
-  /* Init new data for this menu item */
-  MenuSetItemData( &mii,
-                  uFlags,
-                  uIDNewItem,
-                  lpNewItem,
-                  TRUE);
+  MENU_mnu2mnuii( uFlags, uIDNewItem, lpNewItem, &mii, TRUE);
 
-  /* Now, make Win32k IntSetMenuItemInfo handle the changes to this menu item. */
-  return SetMenuItemInfoW( hMnu,
-                           uPosition,
-                           (BOOL)(MF_BYPOSITION & uFlags),
-                           &mii);
+  /* copy the text string, it will be one or the other */
+  if (lpNewItem && mii.fMask & MIIM_STRING && !mii.hbmpItem && mii.dwTypeData)
+  {
+    RtlInitUnicodeString(&MenuText, (PWSTR)mii.dwTypeData);
+    mii.dwTypeData = MenuText.Buffer;
+    mii.cch = MenuText.Length / sizeof(WCHAR);
+  }
+  else
+  {
+      TRACE("Handle bitmaps\n");
+  }
+  res = NtUserThunkedMenuItemInfo(hMenu, uPosition, (BOOL)(uFlags & MF_BYPOSITION), FALSE, &mii, &MenuText);
+  return res;
 }
 
-
 /*
  * @implemented
  */
@@ -4976,7 +1500,6 @@ SetMenu(HWND hWnd,
   return NtUserSetMenu(hWnd, hMenu, TRUE);
 }
 
-
 /*
  * @implemented
  */
@@ -4986,7 +1509,7 @@ SetMenuInfo(
   HMENU hmenu,
   LPCMENUINFO lpcmi)
 {
-  ROSMENUINFO mi;
+  MENUINFO mi;
   BOOL res = FALSE;
 
   if (!lpcmi || (lpcmi->cbSize != sizeof(MENUINFO)))
@@ -4996,10 +1519,9 @@ SetMenuInfo(
   }
 
   memcpy(&mi, lpcmi, sizeof(MENUINFO));
-  return NtUserMenuInfo(hmenu, &mi, TRUE);
+  return NtUserThunkedMenuInfo(hmenu, (LPCMENUINFO)&mi);
 }
 
-
 /*
  * @implemented
  */
@@ -5012,88 +1534,63 @@ SetMenuItemBitmaps(
   HBITMAP hBitmapUnchecked,
   HBITMAP hBitmapChecked)
 {
-  ROSMENUITEMINFO uItem;
+  MENUITEMINFOW uItem;
   memset ( &uItem, 0, sizeof(uItem) );
-  uItem.fMask = MIIM_STATE | MIIM_BITMAP;
-
-  if(!(NtUserMenuItemInfo(hMenu, uPosition,
-                 (BOOL)(MF_BYPOSITION & uFlags), &uItem, FALSE))) return FALSE;
-
-  if (!hBitmapChecked && !hBitmapUnchecked)
-  {
-    uItem.fState &= ~MF_USECHECKBITMAPS;
-  }
-  else  /* Install new bitmaps */
-  {
-    uItem.hbmpChecked = hBitmapChecked;
-    uItem.hbmpUnchecked = hBitmapUnchecked;
-    uItem.fState |= MF_USECHECKBITMAPS;
-  }
- return NtUserMenuItemInfo(hMenu, uPosition,
-                                 (BOOL)(MF_BYPOSITION & uFlags), &uItem, TRUE);
+  uItem.cbSize = sizeof(MENUITEMINFOW);
+  uItem.fMask = MIIM_CHECKMARKS;
+  uItem.hbmpUnchecked = hBitmapUnchecked;
+  uItem.hbmpChecked = hBitmapChecked;
+  return SetMenuItemInfoW(hMenu, uPosition, (BOOL)(uFlags & MF_BYPOSITION), &uItem);
 }
 
-
 /*
  * @implemented
  */
 BOOL
 WINAPI
 SetMenuItemInfoA(
-  HMENU hMenu,
-  UINT uItem,
-  BOOL fByPosition,
+  HMENU hmenu,
+  UINT item,
+  BOOL bypos,
   LPCMENUITEMINFOA lpmii)
 {
-  MENUITEMINFOW MenuItemInfoW;
+  MENUITEMINFOW mii;
   UNICODE_STRING UnicodeString;
-  ULONG Result = FALSE;
+  BOOL Ret;
 
-  RtlCopyMemory(&MenuItemInfoW, lpmii, min(lpmii->cbSize, sizeof(MENUITEMINFOW)));
+  TRACE("hmenu %p, item %u, by pos %d, info %p\n", hmenu, item, bypos, lpmii);
 
-  if( lpmii->cbSize != sizeof( MENUITEMINFOW))
-  {
-     MenuItemInfoW.cbSize = sizeof( MENUITEMINFOW);
-     MenuItemInfoW.hbmpItem = NULL;
-  }
+  RtlInitUnicodeString(&UnicodeString, 0);
+
+  if (!MENU_NormalizeMenuItemInfoStruct( (const MENUITEMINFOW *)lpmii, &mii )) return FALSE;
 /*
  *  MIIM_STRING              == good
  *  MIIM_TYPE & MFT_STRING   == good
  *  MIIM_STRING & MFT_STRING == good
- *  MIIM_STRING & MFT_OWNERSRAW == good
+ *  MIIM_STRING & MFT_OWNERDRAW == good
  */
-  if (((MenuItemInfoW.fMask & MIIM_STRING) ||
-      ((MenuItemInfoW.fMask & MIIM_TYPE) &&
-                           (MENU_ITEM_TYPE(MenuItemInfoW.fType) == MF_STRING)))
-        && MenuItemInfoW.dwTypeData != NULL)
+  if (((mii.fMask & MIIM_STRING) ||
+      ((mii.fMask & MIIM_TYPE) && (MENU_ITEM_TYPE(mii.fType) == MF_STRING)))
+        && mii.dwTypeData && !(GdiValidateHandle((HGDIOBJ)mii.dwTypeData)) )
   {
-/* cch is ignored when the content of a menu item is set by calling SetMenuItemInfo. */
-    if (!RtlCreateUnicodeStringFromAsciiz(&UnicodeString,
-                                          (LPSTR)MenuItemInfoW.dwTypeData))
-    {
+    /* cch is ignored when the content of a menu item is set by calling SetMenuItemInfo. */
+     if (!RtlCreateUnicodeStringFromAsciiz(&UnicodeString, (LPSTR)mii.dwTypeData))
+     {
         SetLastError (ERROR_NOT_ENOUGH_MEMORY);
         return FALSE;
-    }
-    MenuItemInfoW.dwTypeData = UnicodeString.Buffer;
-    MenuItemInfoW.cch = UnicodeString.Length / sizeof(WCHAR);
+     }
+     mii.dwTypeData = UnicodeString.Buffer;
+     mii.cch = UnicodeString.Length / sizeof(WCHAR);
   }
   else
   {
-    UnicodeString.Buffer = NULL;
-  }
-
-  Result = NtUserMenuItemInfo(hMenu, uItem, fByPosition,
-                              (PROSMENUITEMINFO)&MenuItemInfoW, TRUE);
-
-  if (UnicodeString.Buffer != NULL)
-  {
-    RtlFreeUnicodeString(&UnicodeString);
+     UnicodeString.Buffer = NULL;
   }
-
-  return Result;
+  Ret = NtUserThunkedMenuItemInfo(hmenu, item, bypos, FALSE, &mii, &UnicodeString);
+  if (UnicodeString.Buffer != NULL) RtlFreeUnicodeString(&UnicodeString);
+  return Ret;
 }
 
-
 /*
  * @implemented
  */
@@ -5106,27 +1603,26 @@ SetMenuItemInfoW(
   LPCMENUITEMINFOW lpmii)
 {
   MENUITEMINFOW MenuItemInfoW;
-  ULONG Result;
+  UNICODE_STRING UnicodeString;
+  BOOL Ret;
 
-  RtlCopyMemory(&MenuItemInfoW, lpmii, min(lpmii->cbSize, sizeof(MENUITEMINFOW)));
+  TRACE("hmenu %p, item %u, by pos %d, info %p\n", hMenu, uItem, fByPosition, lpmii);
 
-  if( lpmii->cbSize != sizeof( MENUITEMINFOW))
-  {
-     MenuItemInfoW.cbSize = sizeof( MENUITEMINFOW);
-     MenuItemInfoW.hbmpItem = NULL;
-  }
+  RtlInitUnicodeString(&UnicodeString, 0);
+
+  if (!MENU_NormalizeMenuItemInfoStruct( (const MENUITEMINFOW *)lpmii, &MenuItemInfoW )) return FALSE;
 
   if (((MenuItemInfoW.fMask & MIIM_STRING) ||
       ((MenuItemInfoW.fMask & MIIM_TYPE) &&
                            (MENU_ITEM_TYPE(MenuItemInfoW.fType) == MF_STRING)))
-        && MenuItemInfoW.dwTypeData != NULL)
+        && MenuItemInfoW.dwTypeData && !(GdiValidateHandle((HGDIOBJ)MenuItemInfoW.dwTypeData)) )
   {
-     MenuItemInfoW.cch = strlenW(MenuItemInfoW.dwTypeData);
+      RtlInitUnicodeString(&UnicodeString, (PCWSTR)MenuItemInfoW.dwTypeData);
+      MenuItemInfoW.cch = strlenW(MenuItemInfoW.dwTypeData);
   }
-  Result = NtUserMenuItemInfo(hMenu, uItem, fByPosition,
-                            (PROSMENUITEMINFO)&MenuItemInfoW, TRUE);
+  Ret = NtUserThunkedMenuItemInfo(hMenu, uItem, fByPosition, FALSE, &MenuItemInfoW, &UnicodeString);
 
-  return Result;
+  return Ret;
 }
 
 /*
@@ -5151,14 +1647,9 @@ SetSystemMenu (
   return NtUserSetSystemMenu(hwnd, hMenu);
 }
 
-//
-// Example for the Win32/User32 rewrite.
-// Def = TrackPopupMenuEx@24=NtUserTrackPopupMenuEx@24
-//
-//
 BOOL
 WINAPI
-NEWTrackPopupMenu(
+TrackPopupMenu(
   HMENU Menu,
   UINT Flags,
   int x,
@@ -5175,29 +1666,10 @@ NEWTrackPopupMenu(
                                  NULL); // LPTPMPARAMS is null
 }
 
-
-/*
- * @implemented
- */
-DWORD
-WINAPI
-GetMenuContextHelpId(HMENU hmenu)
-{
-  ROSMENUINFO mi;
-  mi.cbSize = sizeof(ROSMENUINFO);
-  mi.fMask = MIM_HELPID;
-
-  if(NtUserMenuInfo(hmenu, &mi, FALSE))
-  {
-    return mi.dwContextHelpID;
-  }
-  return 0;
-}
-
 /*
  * @unimplemented
  */
-BOOL
+LRESULT
 WINAPI
 MenuWindowProcA(
                HWND   hWnd,
@@ -5209,23 +1681,15 @@ MenuWindowProcA(
 {
   if ( Msg < WM_USER)
   {
-     LRESULT lResult;
-     lResult = PopupMenuWndProcA(hWnd, Msg, wParam, lParam );
-     if (Result)
-     {
-        Result = (ULONG_PTR)lResult;
-        return TRUE;
-     }
-     return FALSE;
+     return PopupMenuWndProcA(hWnd, Msg, wParam, lParam );
   }
   return NtUserMessageCall(hWnd, Msg, wParam, lParam, Result, FNID_MENU, TRUE);
-
 }
 
 /*
  * @unimplemented
  */
-BOOL
+LRESULT
 WINAPI
 MenuWindowProcW(
                HWND   hWnd,
@@ -5237,14 +1701,7 @@ MenuWindowProcW(
 {
   if ( Msg < WM_USER)
   {
-     LRESULT lResult;
-     lResult = PopupMenuWndProcW(hWnd, Msg, wParam, lParam );
-     if (Result)
-     {
-        Result = (ULONG_PTR)lResult;
-        return TRUE;
-     }
-     return FALSE;
+     return PopupMenuWndProcW(hWnd, Msg, wParam, lParam );
   }
   return NtUserMessageCall(hWnd, Msg, wParam, lParam, Result, FNID_MENU, FALSE);
 }
@@ -5325,7 +1782,3 @@ ChangeMenuA(
     };
 }
 
-
-
-
-