Changeset 10190 for trunk/src/user32/menu.c

Timestamp:

Jul 31, 2003, 5:58:58 PM (22 years ago)

Author:

sandervl

Message:

KOMH: DBCS updates/fixes

File:

• trunk/src/user32/menu.c (modified) (146 diffs)

trunk/src/user32/menu.c

@@ -44 +44 @@
 #include "oslibmsg.h"
 #include "oslibwin.h"
+#include "heapstring.h"
+
 
 HBRUSH WIN32API GetOS2ColorBrush(int nIndex);
@@ -55 +57 @@
 DECLARE_DEBUG_CHANNEL(accel);
 
-#define HACCEL_16(h32)          (LOWORD(h32))
+#define HACCEL_16(h32)      (LOWORD(h32))
 
 /* internal popup menu window messages */
 
-#define MM_SETMENUHANDLE        (WM_USER + 0)
-#define MM_GETMENUHANDLE        (WM_USER + 1)
+#define MM_SETMENUHANDLE    (WM_USER + 0)
+#define MM_GETMENUHANDLE    (WM_USER + 1)
 
 /* Menu item structure */
 typedef struct {
     /* ----------- MENUITEMINFO Stuff ----------- */
-    UINT fType;                 /* Item type. */
-    UINT fState;                /* Item state.  */
-    UINT wID;                   /* Item id.  */
-    HMENU hSubMenu;             /* Pop-up menu.  */
-    HBITMAP hCheckBit;          /* Bitmap when checked.  */
-    HBITMAP hUnCheckBit;        /* Bitmap when unchecked.  */
-    LPWSTR text;                        /* Item text or bitmap handle.  */
-    DWORD dwItemData;           /* Application defined.  */
-    DWORD dwTypeData;           /* depends on fMask */
-    HBITMAP hbmpItem;           /* bitmap in win98 style menus */
+    UINT fType;         /* Item type. */
+    UINT fState;        /* Item state.  */
+    UINT wID;           /* Item id.  */
+    HMENU hSubMenu;     /* Pop-up menu.  */
+    HBITMAP hCheckBit;      /* Bitmap when checked.  */
+    HBITMAP hUnCheckBit;    /* Bitmap when unchecked.  */
+    LPWSTR text;            /* Item text or bitmap handle.  */
+    DWORD dwItemData;       /* Application defined.  */
+    DWORD dwTypeData;       /* depends on fMask */
+    HBITMAP hbmpItem;       /* bitmap in win98 style menus */
     /* ----------- Wine stuff ----------- */
-    RECT      rect;             /* Item area (relative to menu window) */
-    UINT      xTab;             /* X position of text after Tab */
+    RECT      rect;     /* Item area (relative to menu window) */
+    UINT      xTab;     /* X position of text after Tab */
 } MENUITEM;
 
@@ -84 +86 @@
     WORD        wFlags;       /* Menu flags (MF_POPUP, MF_SYSMENU) */
     WORD        wMagic;       /* Magic number */
-    WORD        Width;        /* Width of the whole menu */
-    WORD        Height;       /* Height of the whole menu */
+    WORD    Width;        /* Width of the whole menu */
+    WORD    Height;       /* Height of the whole menu */
     UINT        nItems;       /* Number of items in the menu */
     HWND        hWnd;         /* Window containing the menu */
     MENUITEM    *items;       /* Array of menu items */
     UINT        FocusedItem;  /* Currently focused item */
-    HWND        hwndOwner;    /* window receiving the messages for ownerdraw */
+    HWND    hwndOwner;    /* window receiving the messages for ownerdraw */
     BOOL        bTimeToHide;  /* Request hiding when receiving a second click in the top-level menu item */
     /* ------------ MENUINFO members ------ */
-    DWORD       dwStyle;        /* Extended mennu style */
-    UINT        cyMax;          /* max hight of the whole menu, 0 is screen hight */
-    HBRUSH      hbrBack;        /* brush for menu background */
-    DWORD       dwContextHelpID;
-    DWORD       dwMenuData;     /* application defined value */
+    DWORD   dwStyle;    /* Extended mennu style */
+    UINT    cyMax;      /* max hight of the whole menu, 0 is screen hight */
+    HBRUSH  hbrBack;    /* brush for menu background */
+    DWORD   dwContextHelpID;
+    DWORD   dwMenuData; /* application defined value */
     HMENU       hSysMenuOwner;  /* Handle to the dummy sys menu holder */
 } POPUPMENU, *LPPOPUPMENU;
@@ -105 +107 @@
 #define TF_ENDMENU              0x0001
 #define TF_SUSPENDPOPUP         0x0002
-#define TF_SKIPREMOVE           0x0004
+#define TF_SKIPREMOVE       0x0004
 
 typedef struct
 {
-    UINT        trackFlags;
-    HMENU       hCurrentMenu; /* current submenu (can be equal to hTopMenu)*/
-    HMENU       hTopMenu;     /* initial menu */
-    HWND        hOwnerWnd;    /* where notifications are sent */
-    POINT       pt;
+    UINT    trackFlags;
+    HMENU   hCurrentMenu; /* current submenu (can be equal to hTopMenu)*/
+    HMENU   hTopMenu;     /* initial menu */
+    HWND    hOwnerWnd;    /* where notifications are sent */
+    POINT   pt;
 } MTRACKER;
 
 #define MENU_MAGIC   0x554d  /* 'MU' */
 
-#define ITEM_PREV               -1
-#define ITEM_NEXT                1
+#define ITEM_PREV       -1
+#define ITEM_NEXT        1
 
   /* Internal MENU_TrackMenu() flags */
-#define TPM_INTERNAL            0xF0000000
-#define TPM_ENTERIDLEEX         0x80000000              /* set owner window for WM_ENTERIDLE */
-#define TPM_BUTTONDOWN          0x40000000              /* menu was clicked before tracking */
+#define TPM_INTERNAL        0xF0000000
+#define TPM_ENTERIDLEEX     0x80000000      /* set owner window for WM_ENTERIDLE */
+#define TPM_BUTTONDOWN      0x40000000      /* menu was clicked before tracking */
 #define TPM_POPUPMENU           0x20000000              /* menu is a popup menu */
 
   /* popup menu shade thickness */
-#define POPUP_XSHADE            4
-#define POPUP_YSHADE            4
+#define POPUP_XSHADE        4
+#define POPUP_YSHADE        4
 
   /* Space between 2 menu bar items */
@@ -151 +153 @@
 
 #define IS_SYSTEM_MENU(menu)  \
-        (!((menu)->wFlags & MF_POPUP) && (menu)->wFlags & MF_SYSMENU)
+    (!((menu)->wFlags & MF_POPUP) && (menu)->wFlags & MF_SYSMENU)
 
 #define IS_SYSTEM_POPUP(menu) \
-        ((menu)->wFlags & MF_POPUP && (menu)->wFlags & MF_SYSMENU)
+    ((menu)->wFlags & MF_POPUP && (menu)->wFlags & MF_SYSMENU)
 
 #define TYPE_MASK (MFT_STRING | MFT_BITMAP | MFT_OWNERDRAW | MFT_SEPARATOR | \
-                   MFT_MENUBARBREAK | MFT_MENUBREAK | MFT_RADIOCHECK | \
-                   MFT_RIGHTORDER | MFT_RIGHTJUSTIFY | \
-                   MF_POPUP | MF_SYSMENU | MF_HELP)
+           MFT_MENUBARBREAK | MFT_MENUBREAK | MFT_RADIOCHECK | \
+           MFT_RIGHTORDER | MFT_RIGHTJUSTIFY | \
+           MF_POPUP | MF_SYSMENU | MF_HELP)
 #define STATE_MASK (~TYPE_MASK)
 
@@ -168 +170 @@
 static HBITMAP hBmpSysMenu = 0;
 
-static HBRUSH   hShadeBrush = 0;
-static HFONT    hMenuFont = 0;
-static HFONT    hMenuFontBold = 0;
+static HBRUSH   hShadeBrush = 0;
+static HFONT    hMenuFont = 0;
+static HFONT    hMenuFontBold = 0;
 
 static HMENU MENU_DefSysPopup = 0;  /* Default system menu popup */
@@ -217 +219 @@
 
 static void do_debug_print_menuitem(const char *prefix, MENUITEM * mp,
-                                    const char *postfix)
+                    const char *postfix)
 {
     TRACE("%s ", prefix);
     if (mp) {
-        UINT flags = mp->fType;
-        int typ = MENU_ITEM_TYPE(flags);
-        DPRINTF( "{ ID=0x%x", mp->wID);
-        if (flags & MF_POPUP)
-            DPRINTF( ", Sub=0x%x", mp->hSubMenu);
-        if (flags) {
-            int count = 0;
-            DPRINTF( ", Typ=");
-            if (typ == MFT_STRING)
-                /* Nothing */ ;
-            else if (typ == MFT_SEPARATOR)
-                MENUOUT("sep");
-            else if (typ == MFT_OWNERDRAW)
-                MENUOUT("own");
-            else if (typ == MFT_BITMAP)
-                MENUOUT("bit");
-            else
-                MENUOUT("???");
-            flags -= typ;
-
-            MENUFLAG(MF_POPUP, "pop");
-            MENUFLAG(MFT_MENUBARBREAK, "barbrk");
-            MENUFLAG(MFT_MENUBREAK, "brk");
-            MENUFLAG(MFT_RADIOCHECK, "radio");
-            MENUFLAG(MFT_RIGHTORDER, "rorder");
-            MENUFLAG(MF_SYSMENU, "sys");
-            MENUFLAG(MFT_RIGHTJUSTIFY, "right");        /* same as MF_HELP */
-
-            if (flags)
-                DPRINTF( "+0x%x", flags);
-        }
-        flags = mp->fState;
-        if (flags) {
-            int count = 0;
-            DPRINTF( ", State=");
-            MENUFLAG(MFS_GRAYED, "grey");
-            MENUFLAG(MFS_DEFAULT, "default");
-            MENUFLAG(MFS_DISABLED, "dis");
-            MENUFLAG(MFS_CHECKED, "check");
-            MENUFLAG(MFS_HILITE, "hi");
-            MENUFLAG(MF_USECHECKBITMAPS, "usebit");
-            MENUFLAG(MF_MOUSESELECT, "mouse");
-            if (flags)
-                DPRINTF( "+0x%x", flags);
-        }
-        if (mp->hCheckBit)
-            DPRINTF( ", Chk=0x%x", mp->hCheckBit);
-        if (mp->hUnCheckBit)
-            DPRINTF( ", Unc=0x%x", mp->hUnCheckBit);
-
-        if (typ == MFT_STRING) {
-            if (mp->text)
-                DPRINTF( ", Text=%s", debugstr_w(mp->text));
-            else
-                DPRINTF( ", Text=Null");
-        } else if (mp->text == NULL)
-            /* Nothing */ ;
-        else
-            DPRINTF( ", Text=%p", mp->text);
-        if (mp->dwItemData)
-            DPRINTF( ", ItemData=0x%08lx", mp->dwItemData);
-        DPRINTF( " }");
+    UINT flags = mp->fType;
+    int typ = MENU_ITEM_TYPE(flags);
+    DPRINTF( "{ ID=0x%x", mp->wID);
+    if (flags & MF_POPUP)
+        DPRINTF( ", Sub=0x%x", mp->hSubMenu);
+    if (flags) {
+        int count = 0;
+        DPRINTF( ", Typ=");
+        if (typ == MFT_STRING)
+        /* Nothing */ ;
+        else if (typ == MFT_SEPARATOR)
+        MENUOUT("sep");
+        else if (typ == MFT_OWNERDRAW)
+        MENUOUT("own");
+        else if (typ == MFT_BITMAP)
+        MENUOUT("bit");
+        else
+        MENUOUT("???");
+        flags -= typ;
+
+        MENUFLAG(MF_POPUP, "pop");
+        MENUFLAG(MFT_MENUBARBREAK, "barbrk");
+        MENUFLAG(MFT_MENUBREAK, "brk");
+        MENUFLAG(MFT_RADIOCHECK, "radio");
+        MENUFLAG(MFT_RIGHTORDER, "rorder");
+        MENUFLAG(MF_SYSMENU, "sys");
+        MENUFLAG(MFT_RIGHTJUSTIFY, "right");    /* same as MF_HELP */
+
+        if (flags)
+        DPRINTF( "+0x%x", flags);
+    }
+    flags = mp->fState;
+    if (flags) {
+        int count = 0;
+        DPRINTF( ", State=");
+        MENUFLAG(MFS_GRAYED, "grey");
+        MENUFLAG(MFS_DEFAULT, "default");
+        MENUFLAG(MFS_DISABLED, "dis");
+        MENUFLAG(MFS_CHECKED, "check");
+        MENUFLAG(MFS_HILITE, "hi");
+        MENUFLAG(MF_USECHECKBITMAPS, "usebit");
+        MENUFLAG(MF_MOUSESELECT, "mouse");
+        if (flags)
+        DPRINTF( "+0x%x", flags);
+    }
+    if (mp->hCheckBit)
+        DPRINTF( ", Chk=0x%x", mp->hCheckBit);
+    if (mp->hUnCheckBit)
+        DPRINTF( ", Unc=0x%x", mp->hUnCheckBit);
+
+    if (typ == MFT_STRING) {
+        if (mp->text)
+        DPRINTF( ", Text=%s", debugstr_w(mp->text));
+        else
+        DPRINTF( ", Text=Null");
+    } else if (mp->text == NULL)
+        /* Nothing */ ;
+    else
+        DPRINTF( ", Text=%p", mp->text);
+    if (mp->dwItemData)
+        DPRINTF( ", ItemData=0x%08lx", mp->dwItemData);
+    DPRINTF( " }");
     } else {
-        DPRINTF( "NULL");
+    DPRINTF( "NULL");
     }
 
@@ -355 +357 @@
         POPUPMENU* menu = MENU_GetMenu(hMenu);
         menu->wFlags |= MF_SYSMENU | MF_POPUP;
-        SetMenuDefaultItem(hMenu, SC_CLOSE, FALSE);
+    SetMenuDefaultItem(hMenu, SC_CLOSE, FALSE);
 #ifdef __WIN32OS2__
         if(!fDisableOdinSysMenuItems) {
@@ -372 +374 @@
     }
     else
-        ERR("Unable to load default system menu\n" );
+    ERR("Unable to load default system menu\n" );
 
     TRACE("returning %x.\n", hMenu );
@@ -395 +397 @@
     if ((hMenu = CreateMenu()))
     {
-        POPUPMENU *menu = MENU_GetMenu(hMenu);
-        menu->wFlags = MF_SYSMENU;
-        menu->hWnd = WIN_GetFullHandle( hWnd );
-
-        if (hPopupMenu == (HMENU)(-1))
-            hPopupMenu = MENU_CopySysPopup();
-        else if( !hPopupMenu ) hPopupMenu = MENU_DefSysPopup;
-
-        if (hPopupMenu)
-        {
-            InsertMenuA( hMenu, -1, MF_SYSMENU | MF_POPUP | MF_BYPOSITION, hPopupMenu, NULL );
+    POPUPMENU *menu = MENU_GetMenu(hMenu);
+    menu->wFlags = MF_SYSMENU;
+    menu->hWnd = WIN_GetFullHandle( hWnd );
+
+    if (hPopupMenu == (HMENU)(-1))
+        hPopupMenu = MENU_CopySysPopup();
+    else if( !hPopupMenu ) hPopupMenu = MENU_DefSysPopup;
+
+    if (hPopupMenu)
+    {
+        InsertMenuA( hMenu, -1, MF_SYSMENU | MF_POPUP | MF_BYPOSITION, hPopupMenu, NULL );
 
             menu->items[0].fType = MF_SYSMENU | MF_POPUP;
@@ -411 +413 @@
             if ((menu = MENU_GetMenu(hPopupMenu))) menu->wFlags |= MF_SYSMENU;
 
-            TRACE("GetSysMenu hMenu=%04x (%04x)\n", hMenu, hPopupMenu );
-            return hMenu;
-        }
-        DestroyMenu( hMenu );
+        TRACE("GetSysMenu hMenu=%04x (%04x)\n", hMenu, hPopupMenu );
+        return hMenu;
+    }
+    DestroyMenu( hMenu );
     }
     ERR("failed to load system menu!\n");
@@ -432 +434 @@
 
     static unsigned char shade_bits[16] = { 0x55, 0, 0xAA, 0,
-                                            0x55, 0, 0xAA, 0,
-                                            0x55, 0, 0xAA, 0,
-                                            0x55, 0, 0xAA, 0 };
+                        0x55, 0, 0xAA, 0,
+                        0x55, 0, 0xAA, 0,
+                        0x55, 0, 0xAA, 0 };
 
     /* Load menu bitmaps */
@@ -443 +445 @@
     if (hStdMnArrow)
     {
-        BITMAP bm;
-        GetObjectA( hStdMnArrow, sizeof(bm), &bm );
-        arrow_bitmap_width = bm.bmWidth;
-        arrow_bitmap_height = bm.bmHeight;
+    BITMAP bm;
+    GetObjectA( hStdMnArrow, sizeof(bm), &bm );
+    arrow_bitmap_width = bm.bmWidth;
+    arrow_bitmap_height = bm.bmHeight;
     } else
-        return FALSE;
+    return FALSE;
 
     if (! (hBitmap = CreateBitmap( 8, 8, 1, 1, shade_bits)))
-        return FALSE;
+    return FALSE;
 
     if(!(hShadeBrush = CreatePatternBrush( hBitmap )))
-        return FALSE;
+    return FALSE;
 
     DeleteObject( hBitmap );
     if (!(MENU_DefSysPopup = MENU_CopySysPopup()))
-        return FALSE;
+    return FALSE;
 
     ncm.cbSize = sizeof (NONCLIENTMETRICSA);
     if (!(SystemParametersInfoA(SPI_GETNONCLIENTMETRICS, sizeof(NONCLIENTMETRICSA), &ncm, 0)))
-        return FALSE;
+    return FALSE;
 
     if (!(hMenuFont = CreateFontIndirectA( &ncm.lfMenuFont )))
-        return FALSE;
+    return FALSE;
 
     ncm.lfMenuFont.lfWeight += 300;
     if ( ncm.lfMenuFont.lfWeight > 1000)
-        ncm.lfMenuFont.lfWeight = 1000;
+    ncm.lfMenuFont.lfWeight = 1000;
 
     if (!(hMenuFontBold = CreateFontIndirectA( &ncm.lfMenuFont )))
-        return FALSE;
+    return FALSE;
 
     return TRUE;
@@ -500 +502 @@
     /* The menu item must keep its state if it's disabled */
     if(gray)
-        EnableMenuItem( hmenu, SC_CLOSE, MF_GRAYED);
+    EnableMenuItem( hmenu, SC_CLOSE, MF_GRAYED);
 }
 
@@ -518 +520 @@
 
     if(!menu)
-        return NO_SELECTED_ITEM;
+    return NO_SELECTED_ITEM;
 
     i = menu->FocusedItem + 1;
     if( i == NO_SELECTED_ITEM )
-        return i;
+    return i;
 
     for( ; i < menu->nItems; ++i ) {
-        if (menu->items[i].fType & MF_MENUBARBREAK)
-            return i;
+    if (menu->items[i].fType & MF_MENUBARBREAK)
+        return i;
     }
 
@@ -547 +549 @@
 
     if( !menu )
-        return NO_SELECTED_ITEM;
+    return NO_SELECTED_ITEM;
 
     if( menu->FocusedItem == 0 || menu->FocusedItem == NO_SELECTED_ITEM )
-        return NO_SELECTED_ITEM;
+    return NO_SELECTED_ITEM;
 
     /* Find the start of the column */
 
     for(i = menu->FocusedItem; i != 0 &&
-         !(menu->items[i].fType & MF_MENUBARBREAK);
-        --i); /* empty */
+     !(menu->items[i].fType & MF_MENUBARBREAK);
+    --i); /* empty */
 
     if(i == 0)
-        return NO_SELECTED_ITEM;
+    return NO_SELECTED_ITEM;
 
     for(--i; i != 0; --i) {
-        if (menu->items[i].fType & MF_MENUBARBREAK)
-            break;
+    if (menu->items[i].fType & MF_MENUBARBREAK)
+        break;
     }
 
@@ -587 +589 @@
     if (wFlags & MF_BYPOSITION)
     {
-        if (*nPos >= menu->nItems) return NULL;
-        return &menu->items[*nPos];
+    if (*nPos >= menu->nItems) return NULL;
+    return &menu->items[*nPos];
     }
     else
     {
         MENUITEM *item = menu->items;
-        for (i = 0; i < menu->nItems; i++, item++)
-        {
-            if (item->wID == *nPos)
-            {
-                *nPos = i;
-                return item;
-            }
-            else if (item->fType & MF_POPUP)
-            {
-                HMENU hsubmenu = item->hSubMenu;
-                MENUITEM *subitem = MENU_FindItem( &hsubmenu, nPos, wFlags );
+    for (i = 0; i < menu->nItems; i++, item++)
+    {
+        if (item->wID == *nPos)
+        {
+        *nPos = i;
+        return item;
+        }
+        else if (item->fType & MF_POPUP)
+        {
+        HMENU hsubmenu = item->hSubMenu;
+        MENUITEM *subitem = MENU_FindItem( &hsubmenu, nPos, wFlags );
 #ifdef __WIN32OS2__
                 //YD: 2000-12-01: extra check added for loops in menus
                 if (subitem && subitem!=item)
 #else
-                if (subitem)
+        if (subitem)
 #endif
-                {
-                    *hmenu = hsubmenu;
-                    return subitem;
-                }
-            }
-        }
+        {
+            *hmenu = hsubmenu;
+            return subitem;
+        }
+        }
+    }
     }
     return NULL;
@@ -671 +673 @@
  */
 static MENUITEM *MENU_FindItemByCoords( POPUPMENU *menu,
-                                        POINT pt, UINT *pos )
+                    POINT pt, UINT *pos )
 {
     MENUITEM *item;
@@ -682 +684 @@
     for (i = 0; i < menu->nItems; i++, item++)
     {
-        if ((pt.x >= item->rect.left) && (pt.x < item->rect.right) &&
-            (pt.y >= item->rect.top) && (pt.y < item->rect.bottom))
-        {
-            if (pos) *pos = i;
-            return item;
-        }
+    if ((pt.x >= item->rect.left) && (pt.x < item->rect.right) &&
+        (pt.y >= item->rect.top) && (pt.y < item->rect.bottom))
+    {
+        if (pos) *pos = i;
+        return item;
+    }
     }
     return NULL;
@@ -700 +702 @@
  */
 static UINT MENU_FindItemByKey( HWND hwndOwner, HMENU hmenu,
-                                  UINT key, BOOL forceMenuChar )
+                  UINT key, BOOL forceMenuChar )
 {
     TRACE("\tlooking for '%c' in [%04x]\n", (char)key, (UINT16)hmenu );
@@ -708 +710 @@
     if (hmenu)
     {
-        POPUPMENU *menu = MENU_GetMenu( hmenu );
-        MENUITEM *item = menu->items;
-        LONG menuchar;
-
-        if( !forceMenuChar )
-        {
-             UINT i;
-
-             key = toupper(key);
-             for (i = 0; i < menu->nItems; i++, item++)
-             {
-                if (item->text && (IS_STRING_ITEM(item->fType)))
-                {
-                    WCHAR *p = item->text - 2;
-                    do
-                    {
-                        p = strchrW (p + 2, '&');
-                    }
-                    while (p != NULL && p [1] == '&');
-                    if (p && (toupper(p[1]) == key)) return i;
-                }
-             }
-        }
-        menuchar = SendMessageA( hwndOwner, WM_MENUCHAR,
+    POPUPMENU *menu = MENU_GetMenu( hmenu );
+    MENUITEM *item = menu->items;
+    LONG menuchar;
+
+    if( !forceMenuChar )
+    {
+         UINT i;
+
+         key = toupper(key);
+         for (i = 0; i < menu->nItems; i++, item++)
+         {
+        if (item->text && (IS_STRING_ITEM(item->fType)))
+        {
+            WCHAR *p = item->text - 2;
+            do
+            {
+                p = strchrW (p + 2, '&');
+            }
+            while (p != NULL && p [1] == '&');
+            if (p && (toupper(p[1]) == key)) return i;
+        }
+         }
+    }
+    menuchar = SendMessageA( hwndOwner, WM_MENUCHAR,
                                    MAKEWPARAM( key, menu->wFlags ), hmenu );
-        if (HIWORD(menuchar) == 2) return LOWORD(menuchar);
-        if (HIWORD(menuchar) == 1) return (UINT)(-2);
+    if (HIWORD(menuchar) == 2) return LOWORD(menuchar);
+    if (HIWORD(menuchar) == 1) return (UINT)(-2);
     }
     return (UINT)(-1);
@@ -888 +890 @@
  */
 static void MENU_CalcItemSize( HDC hdc, MENUITEM *lpitem, HWND hwndOwner,
-                               INT orgX, INT orgY, BOOL menuBar )
+                   INT orgX, INT orgY, BOOL menuBar )
 {
     WCHAR *p;
@@ -895 +897 @@
     TRACE("dc=0x%04x owner=0x%04x (%d,%d)\n", hdc, hwndOwner, orgX, orgY);
     debug_print_menuitem("MENU_CalcItemSize: menuitem:", lpitem,
-                         (menuBar ? " (MenuBar)" : ""));
+             (menuBar ? " (MenuBar)" : ""));
 
     SetRect( &lpitem->rect, orgX, orgY, orgX, orgY );
@@ -918 +920 @@
         lpitem->rect.right  += mis.itemWidth;
 
-        if (menuBar)
-        {
-             lpitem->rect.right += MENU_BAR_ITEMS_SPACE;
+    if (menuBar)
+    {
+         lpitem->rect.right += MENU_BAR_ITEMS_SPACE;
 
 
@@ -926 +928 @@
                 height for the menu and the height value is ignored */
 
-             if (TWEAK_WineLook == WIN31_LOOK)
-                lpitem->rect.bottom += GetSystemMetrics(SM_CYMENU);
-             else
-                lpitem->rect.bottom += GetSystemMetrics(SM_CYMENU)-1;
+         if (TWEAK_WineLook == WIN31_LOOK)
+            lpitem->rect.bottom += GetSystemMetrics(SM_CYMENU);
+         else
+            lpitem->rect.bottom += GetSystemMetrics(SM_CYMENU)-1;
         }
         else
             lpitem->rect.bottom += mis.itemHeight;
 
-        TRACE("id=%04x size=%dx%d\n",
+    TRACE("id=%04x size=%dx%d\n",
                      lpitem->wID, mis.itemWidth, mis.itemHeight);
         /* Fall through to get check/arrow width calculation. */
@@ -941 +943 @@
     if (lpitem->fType & MF_SEPARATOR)
     {
-        lpitem->rect.bottom += SEPARATOR_HEIGHT;
-        return;
+    lpitem->rect.bottom += SEPARATOR_HEIGHT;
+    return;
     }
 
     if (!menuBar)
     {
-        lpitem->rect.right += 2 * check_bitmap_width;
-        if (lpitem->fType & MF_POPUP)
-            lpitem->rect.right += arrow_bitmap_width;
+    lpitem->rect.right += 2 * check_bitmap_width;
+    if (lpitem->fType & MF_POPUP)
+        lpitem->rect.right += arrow_bitmap_width;
     }
 
@@ -975 +977 @@
     {   SIZE size;
 
-        GetTextExtentPoint32W(hdc, lpitem->text,  strlenW(lpitem->text), &size);
-
-        lpitem->rect.right  += size.cx;
-        if (TWEAK_WineLook == WIN31_LOOK)
-            lpitem->rect.bottom += max( size.cy, GetSystemMetrics(SM_CYMENU) );
-        else
-            lpitem->rect.bottom += max(size.cy, GetSystemMetrics(SM_CYMENU)-1);
-        lpitem->xTab = 0;
-
-        if (menuBar)
-        {
-             lpitem->rect.right += MENU_BAR_ITEMS_SPACE;
-        }
-        else if ((p = strchrW( lpitem->text, '\t' )) != NULL)
-        {
-            /* Item contains a tab (only meaningful in popup menus) */
-            GetTextExtentPoint32W(hdc, lpitem->text, (int)(p - lpitem->text) , &size);
-            lpitem->xTab = check_bitmap_width + MENU_TAB_SPACE + size.cx;
-            lpitem->rect.right += MENU_TAB_SPACE;
-        }
-        else
-        {
-            if (strchrW( lpitem->text, '\b' ))
-                lpitem->rect.right += MENU_TAB_SPACE;
-            lpitem->xTab = lpitem->rect.right - check_bitmap_width
-                           - arrow_bitmap_width;
-        }
+    GetTextExtentPoint32W(hdc, lpitem->text,  strlenW(lpitem->text), &size);
+
+    lpitem->rect.right  += size.cx;
+    if (TWEAK_WineLook == WIN31_LOOK)
+        lpitem->rect.bottom += max( size.cy, GetSystemMetrics(SM_CYMENU) );
+    else
+        lpitem->rect.bottom += max(size.cy, GetSystemMetrics(SM_CYMENU)-1);
+    lpitem->xTab = 0;
+
+    if (menuBar)
+    {
+         lpitem->rect.right += MENU_BAR_ITEMS_SPACE;
+    }
+    else if ((p = strchrW( lpitem->text, '\t' )) != NULL)
+    {
+        /* Item contains a tab (only meaningful in popup menus) */
+        GetTextExtentPoint32W(hdc, lpitem->text, (int)(p - lpitem->text) , &size);
+        lpitem->xTab = check_bitmap_width + MENU_TAB_SPACE + size.cx;
+        lpitem->rect.right += MENU_TAB_SPACE;
+    }
+    else
+    {
+        if (strchrW( lpitem->text, '\b' ))
+            lpitem->rect.right += MENU_TAB_SPACE;
+        lpitem->xTab = lpitem->rect.right - check_bitmap_width
+                       - arrow_bitmap_width;
+    }
     }
     TRACE("(%d,%d)-(%d,%d)\n", lpitem->rect.left, lpitem->rect.top, lpitem->rect.right, lpitem->rect.bottom);
@@ -1034 +1036 @@
     while (start < lppop->nItems)
     {
-        lpitem = &lppop->items[start];
-        orgX = maxX;
-        orgY = (TWEAK_WineLook == WIN31_LOOK) ? GetSystemMetrics(SM_CYBORDER) : 2;
-
-        maxTab = maxTabWidth = 0;
-
-          /* Parse items until column break or end of menu */
-        for (i = start; i < lppop->nItems; i++, lpitem++)
-        {
-            if ((i != start) &&
-                (lpitem->fType & (MF_MENUBREAK | MF_MENUBARBREAK))) break;
-
-            MENU_CalcItemSize( hdc, lpitem, hwndOwner, orgX, orgY, FALSE );
-
-            if (lpitem->fType & MF_MENUBARBREAK) orgX++;
-            maxX = max( maxX, lpitem->rect.right );
-            orgY = lpitem->rect.bottom;
-            if (IS_STRING_ITEM(lpitem->fType) && lpitem->xTab)
-            {
-                maxTab = max( maxTab, lpitem->xTab );
-                maxTabWidth = max(maxTabWidth,lpitem->rect.right-lpitem->xTab);
-            }
-        }
-
-          /* Finish the column (set all items to the largest width found) */
-        maxX = max( maxX, maxTab + maxTabWidth );
-        for (lpitem = &lppop->items[start]; start < i; start++, lpitem++)
-        {
-            lpitem->rect.right = maxX;
-            if (IS_STRING_ITEM(lpitem->fType) && lpitem->xTab)
-                lpitem->xTab = maxTab;
-
-        }
-        lppop->Height = max( lppop->Height, orgY );
+    lpitem = &lppop->items[start];
+    orgX = maxX;
+    orgY = (TWEAK_WineLook == WIN31_LOOK) ? GetSystemMetrics(SM_CYBORDER) : 2;
+
+    maxTab = maxTabWidth = 0;
+
+      /* Parse items until column break or end of menu */
+    for (i = start; i < lppop->nItems; i++, lpitem++)
+    {
+        if ((i != start) &&
+        (lpitem->fType & (MF_MENUBREAK | MF_MENUBARBREAK))) break;
+
+        MENU_CalcItemSize( hdc, lpitem, hwndOwner, orgX, orgY, FALSE );
+
+        if (lpitem->fType & MF_MENUBARBREAK) orgX++;
+        maxX = max( maxX, lpitem->rect.right );
+        orgY = lpitem->rect.bottom;
+        if (IS_STRING_ITEM(lpitem->fType) && lpitem->xTab)
+        {
+        maxTab = max( maxTab, lpitem->xTab );
+        maxTabWidth = max(maxTabWidth,lpitem->rect.right-lpitem->xTab);
+        }
+    }
+
+      /* Finish the column (set all items to the largest width found) */
+    maxX = max( maxX, maxTab + maxTabWidth );
+    for (lpitem = &lppop->items[start]; start < i; start++, lpitem++)
+    {
+        lpitem->rect.right = maxX;
+        if (IS_STRING_ITEM(lpitem->fType) && lpitem->xTab)
+        lpitem->xTab = maxTab;
+
+    }
+    lppop->Height = max( lppop->Height, orgY );
     }
 
@@ -1075 +1077 @@
     if(TWEAK_WineLook > WIN31_LOOK)
     {
-        lppop->Height += 2;
-        lppop->Width += 2;
+    lppop->Height += 2;
+    lppop->Width += 2;
     }
 
@@ -1113 +1115 @@
     while (start < lppop->nItems)
     {
-        lpitem = &lppop->items[start];
-        orgX = lprect->left;
-        orgY = maxY;
-
-          /* Parse items until line break or end of menu */
-        for (i = start; i < lppop->nItems; i++, lpitem++)
-        {
-            if ((helpPos == -1) && (lpitem->fType & MF_RIGHTJUSTIFY)) helpPos = i;
-            if ((i != start) &&
-                (lpitem->fType & (MF_MENUBREAK | MF_MENUBARBREAK))) break;
-
-            TRACE("calling MENU_CalcItemSize org=(%d, %d)\n",
-                         orgX, orgY );
-            debug_print_menuitem ("  item: ", lpitem, "");
-            MENU_CalcItemSize( hdc, lpitem, hwndOwner, orgX, orgY, TRUE );
-
-            if (lpitem->rect.right > lprect->right)
-            {
-                if (i != start) break;
-                else lpitem->rect.right = lprect->right;
-            }
-            maxY = max( maxY, lpitem->rect.bottom );
-            orgX = lpitem->rect.right;
-        }
-
-          /* Finish the line (set all items to the largest height found) */
-        while (start < i) lppop->items[start++].rect.bottom = maxY;
+    lpitem = &lppop->items[start];
+    orgX = lprect->left;
+    orgY = maxY;
+
+      /* Parse items until line break or end of menu */
+    for (i = start; i < lppop->nItems; i++, lpitem++)
+    {
+        if ((helpPos == -1) && (lpitem->fType & MF_RIGHTJUSTIFY)) helpPos = i;
+        if ((i != start) &&
+        (lpitem->fType & (MF_MENUBREAK | MF_MENUBARBREAK))) break;
+
+        TRACE("calling MENU_CalcItemSize org=(%d, %d)\n",
+             orgX, orgY );
+        debug_print_menuitem ("  item: ", lpitem, "");
+        MENU_CalcItemSize( hdc, lpitem, hwndOwner, orgX, orgY, TRUE );
+
+        if (lpitem->rect.right > lprect->right)
+        {
+        if (i != start) break;
+        else lpitem->rect.right = lprect->right;
+        }
+        maxY = max( maxY, lpitem->rect.bottom );
+        orgX = lpitem->rect.right;
+    }
+
+      /* Finish the line (set all items to the largest height found) */
+    while (start < i) lppop->items[start++].rect.bottom = maxY;
     }
 
@@ -1152 +1154 @@
     for (i = lppop->nItems - 1; i >= helpPos; i--, lpitem--) {
         if ( (helpPos==-1) || (helpPos>i) )
-            break;                              /* done */
-        if (lpitem->rect.top != orgY) break;    /* Other line */
-        if (lpitem->rect.right >= orgX) break;  /* Too far right already */
+            break;              /* done */
+        if (lpitem->rect.top != orgY) break;    /* Other line */
+        if (lpitem->rect.right >= orgX) break;  /* Too far right already */
         lpitem->rect.left += orgX - lpitem->rect.right;
         lpitem->rect.right = orgX;
@@ -1167 +1169 @@
  */
 static void MENU_DrawMenuItem( HWND hwnd, HMENU hmenu, HWND hwndOwner, HDC hdc, MENUITEM *lpitem,
-                               UINT height, BOOL menuBar, UINT odaction )
+                   UINT height, BOOL menuBar, UINT odaction )
 {
     RECT rect;
@@ -1175 +1177 @@
     if (lpitem->fType & MF_SYSMENU)
     {
-        if( !IsIconic(hwnd) ) {
-            if (TWEAK_WineLook > WIN31_LOOK)
-                NC_DrawSysButton95( hwnd, hdc,
-                                    lpitem->fState &
-                                    (MF_HILITE | MF_MOUSESELECT) );
-            else
-                NC_DrawSysButton( hwnd, hdc,
-                                  lpitem->fState &
-                                  (MF_HILITE | MF_MOUSESELECT) );
-        }
-
-        return;
+    if( !IsIconic(hwnd) ) {
+        if (TWEAK_WineLook > WIN31_LOOK)
+        NC_DrawSysButton95( hwnd, hdc,
+                    lpitem->fState &
+                    (MF_HILITE | MF_MOUSESELECT) );
+        else
+        NC_DrawSysButton( hwnd, hdc,
+                  lpitem->fState &
+                  (MF_HILITE | MF_MOUSESELECT) );
+    }
+
+    return;
     }
 
@@ -1203 +1205 @@
 
         dis.CtlType   = ODT_MENU;
-        dis.CtlID     = 0;
+    dis.CtlID     = 0;
         dis.itemID    = lpitem->wID;
         dis.itemData  = (DWORD)lpitem->dwItemData;
@@ -1215 +1217 @@
         dis.rcItem     = lpitem->rect;
         TRACE("Ownerdraw: owner=%04x itemID=%d, itemState=%d, itemAction=%d, "
-              "hwndItem=%04x, hdc=%04x, rcItem={%d,%d,%d,%d}\n", hwndOwner,
-              dis.itemID, dis.itemState, dis.itemAction, dis.hwndItem,
-              dis.hDC, dis.rcItem.left, dis.rcItem.top, dis.rcItem.right,
-              dis.rcItem.bottom);
+          "hwndItem=%04x, hdc=%04x, rcItem={%d,%d,%d,%d}\n", hwndOwner,
+          dis.itemID, dis.itemState, dis.itemAction, dis.hwndItem,
+          dis.hDC, dis.rcItem.left, dis.rcItem.top, dis.rcItem.right,
+          dis.rcItem.bottom);
         SendMessageA( hwndOwner, WM_DRAWITEM, 0, (LPARAM)&dis );
         /* Fall through to draw popup-menu arrow */
@@ -1224 +1226 @@
 
     TRACE("rect={%d,%d,%d,%d}\n", lpitem->rect.left, lpitem->rect.top,
-                                        lpitem->rect.right,lpitem->rect.bottom);
+                    lpitem->rect.right,lpitem->rect.bottom);
 
     if (menuBar && (lpitem->fType & MF_SEPARATOR)) return;
@@ -1232 +1234 @@
     if (!(lpitem->fType & MF_OWNERDRAW))
     {
-        if (lpitem->fState & MF_HILITE)
-        {
+    if (lpitem->fState & MF_HILITE)
+    {
 #ifdef __WIN32OS2__
-            if(!fOS2Look) 
+            if(!fOS2Look)
 #else
-            if(TWEAK_WineLook == WIN98_LOOK)
+        if(TWEAK_WineLook == WIN98_LOOK)
 #endif
-            {
-                if(menuBar)
-                    DrawEdge(hdc, &rect, BDR_SUNKENOUTER, BF_RECT);
-                else
-                    FillRect(hdc, &rect, GetSysColorBrush(COLOR_HIGHLIGHT));
-            }
-            else /* Not Win98 Look */
-            {
-                if(!IS_BITMAP_ITEM(lpitem->fType))
+        {
+        if(menuBar)
+            DrawEdge(hdc, &rect, BDR_SUNKENOUTER, BF_RECT);
+        else
+            FillRect(hdc, &rect, GetSysColorBrush(COLOR_HIGHLIGHT));
+        }
+        else /* Not Win98 Look */
+        {
+        if(!IS_BITMAP_ITEM(lpitem->fType))
 #ifdef __WIN32OS2__
-                    FillRect(hdc, &rect, GetOS2ColorBrush(PMSYSCLR_MENUHILITEBGND));
+            FillRect(hdc, &rect, GetOS2ColorBrush(PMSYSCLR_MENUHILITEBGND));
 #else
-                    FillRect(hdc, &rect, GetSysColorBrush(COLOR_HIGHLIGHT));
+            FillRect(hdc, &rect, GetSysColorBrush(COLOR_HIGHLIGHT));
 #endif
-            }
-        }
+        }
+    }
         else
-            FillRect( hdc, &rect, GetSysColorBrush(COLOR_MENU) );
+        FillRect( hdc, &rect, GetSysColorBrush(COLOR_MENU) );
     }
 
@@ -1266 +1268 @@
         if (!menuBar && (lpitem->fType & MF_MENUBARBREAK))
         {
-            if (TWEAK_WineLook > WIN31_LOOK)
-            {
-                RECT rc = rect;
-                rc.top = 3;
-                rc.bottom = height - 3;
-                DrawEdge (hdc, &rc, EDGE_ETCHED, BF_LEFT);
-            }
-            else
-            {
-                SelectObject( hdc, GetSysColorPen(COLOR_WINDOWFRAME) );
-                MoveToEx( hdc, rect.left, 0, NULL );
-                LineTo( hdc, rect.left, height );
-            }
+        if (TWEAK_WineLook > WIN31_LOOK)
+        {
+            RECT rc = rect;
+            rc.top = 3;
+            rc.bottom = height - 3;
+            DrawEdge (hdc, &rc, EDGE_ETCHED, BF_LEFT);
+        }
+        else
+        {
+            SelectObject( hdc, GetSysColorPen(COLOR_WINDOWFRAME) );
+            MoveToEx( hdc, rect.left, 0, NULL );
+            LineTo( hdc, rect.left, height );
+        }
         }
 
@@ -1284 +1286 @@
         if (lpitem->fType & MF_SEPARATOR)
         {
-            if (TWEAK_WineLook > WIN31_LOOK)
-            {
-                RECT rc = rect;
-                rc.left++;
-                rc.right--;
-                rc.top += SEPARATOR_HEIGHT / 2;
-                DrawEdge (hdc, &rc, EDGE_ETCHED, BF_TOP);
-            }
-            else
-            {
-                SelectObject( hdc, GetSysColorPen(COLOR_WINDOWFRAME) );
-                MoveToEx( hdc, rect.left, rect.top + SEPARATOR_HEIGHT/2, NULL );
-                LineTo( hdc, rect.right, rect.top + SEPARATOR_HEIGHT/2 );
-            }
-            return;
+        if (TWEAK_WineLook > WIN31_LOOK)
+        {
+            RECT rc = rect;
+            rc.left++;
+            rc.right--;
+            rc.top += SEPARATOR_HEIGHT / 2;
+            DrawEdge (hdc, &rc, EDGE_ETCHED, BF_TOP);
+        }
+        else
+        {
+            SelectObject( hdc, GetSysColorPen(COLOR_WINDOWFRAME) );
+            MoveToEx( hdc, rect.left, rect.top + SEPARATOR_HEIGHT/2, NULL );
+            LineTo( hdc, rect.right, rect.top + SEPARATOR_HEIGHT/2 );
+        }
+        return;
         }
     }
@@ -1307 +1309 @@
     {
 #ifdef __WIN32OS2__
-        if(!fOS2Look) 
+        if(!fOS2Look)
 #else
-        if(TWEAK_WineLook == WIN98_LOOK)
+    if(TWEAK_WineLook == WIN98_LOOK)
 #endif
-        {
+    {
             if(menuBar) {
-                SetTextColor(hdc, GetSysColor(COLOR_MENUTEXT));
+        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));
+        } else {
+        if(lpitem->fState & MF_GRAYED)
+            SetTextColor(hdc, GetSysColor(COLOR_GRAYTEXT));
+        else
+            SetTextColor(hdc, GetSysColor(COLOR_HIGHLIGHTTEXT));
                 SetBkColor(hdc, GetSysColor(COLOR_HIGHLIGHT));
-            }
-        }
-        else /* Not Win98 Look */
-        {
-            SetTextColor(hdc, GetSysColor(COLOR_HIGHLIGHTTEXT));
-            if(!IS_BITMAP_ITEM(lpitem->fType))
+        }
+    }
+    else /* Not Win98 Look */
+    {
+        SetTextColor(hdc, GetSysColor(COLOR_HIGHLIGHTTEXT));
+        if(!IS_BITMAP_ITEM(lpitem->fType))
 #ifdef __WIN32OS2__
-                SetBkColor(hdc, GetOS2Color(PMSYSCLR_MENUHILITEBGND));
+        SetBkColor(hdc, GetOS2Color(PMSYSCLR_MENUHILITEBGND));
 #else
-                SetBkColor(hdc, GetSysColor(COLOR_HIGHLIGHT));
+        SetBkColor(hdc, GetSysColor(COLOR_HIGHLIGHT));
 #endif
-        }
+    }
     }
     else
     {
-        if (lpitem->fState & MF_GRAYED)
-            SetTextColor( hdc, GetSysColor( COLOR_GRAYTEXT ) );
-        else
-            SetTextColor( hdc, GetSysColor( COLOR_MENUTEXT ) );
-        SetBkColor( hdc, GetSysColor( COLOR_MENU ) );
-    }
-
-        /* helper lines for debugging */
-/*      FrameRect(hdc, &rect, GetStockObject(BLACK_BRUSH));
-        SelectObject( hdc, GetSysColorPen(COLOR_WINDOWFRAME) );
-        MoveToEx( hdc, rect.left, (rect.top + rect.bottom)/2, NULL );
-        LineTo( hdc, rect.right, (rect.top + rect.bottom)/2 );
+    if (lpitem->fState & MF_GRAYED)
+        SetTextColor( hdc, GetSysColor( COLOR_GRAYTEXT ) );
+    else
+        SetTextColor( hdc, GetSysColor( COLOR_MENUTEXT ) );
+    SetBkColor( hdc, GetSysColor( COLOR_MENU ) );
+    }
+
+    /* helper lines for debugging */
+/*  FrameRect(hdc, &rect, GetStockObject(BLACK_BRUSH));
+    SelectObject( hdc, GetSysColorPen(COLOR_WINDOWFRAME) );
+    MoveToEx( hdc, rect.left, (rect.top + rect.bottom)/2, NULL );
+    LineTo( hdc, rect.right, (rect.top + rect.bottom)/2 );
 */
 
     if (!menuBar)
     {
-        INT     y = rect.top + rect.bottom;
+    INT y = rect.top + rect.bottom;
         UINT check_bitmap_width = GetSystemMetrics( SM_CXMENUCHECK );
         UINT check_bitmap_height = GetSystemMetrics( SM_CYMENUCHECK );
@@ -1358 +1360 @@
         if (!(lpitem->fType & MF_OWNERDRAW))
         {
-              /* Draw the check mark
-               *
-               * FIXME:
-               * Custom checkmark bitmaps are monochrome but not always 1bpp.
-               */
+          /* Draw the check mark
+           *
+           * FIXME:
+           * Custom checkmark bitmaps are monochrome but not always 1bpp.
+           */
             HBITMAP bm = (lpitem->fState & MF_CHECKED) ? lpitem->hCheckBit : lpitem->hUnCheckBit;
             if (bm)  /* we have a custom bitmap */
@@ -1390 +1392 @@
         }
 
-          /* Draw the popup-menu arrow */
-        if (lpitem->fType & MF_POPUP)
-        {
-            HDC hdcMem = CreateCompatibleDC( hdc );
-            HBITMAP hOrigBitmap;
-
-            hOrigBitmap = SelectObject( hdcMem, hStdMnArrow );
-            BitBlt( hdc, rect.right - arrow_bitmap_width - 1,
-                      (y - arrow_bitmap_height) / 2,
-                      arrow_bitmap_width, arrow_bitmap_height,
-                      hdcMem, 0, 0, SRCCOPY );
+      /* Draw the popup-menu arrow */
+    if (lpitem->fType & MF_POPUP)
+    {
+        HDC hdcMem = CreateCompatibleDC( hdc );
+        HBITMAP hOrigBitmap;
+
+        hOrigBitmap = SelectObject( hdcMem, hStdMnArrow );
+        BitBlt( hdc, rect.right - arrow_bitmap_width - 1,
+              (y - arrow_bitmap_height) / 2,
+              arrow_bitmap_width, arrow_bitmap_height,
+              hdcMem, 0, 0, SRCCOPY );
             SelectObject( hdcMem, hOrigBitmap );
-            DeleteDC( hdcMem );
-        }
-
-        rect.left += check_bitmap_width;
-        rect.right -= arrow_bitmap_width;
+        DeleteDC( hdcMem );
+    }
+
+    rect.left += check_bitmap_width;
+    rect.right -= arrow_bitmap_width;
     }
 
@@ -1417 +1419 @@
     {
         MENU_DrawBitmapItem( hdc, lpitem, &rect, menuBar );
-        return;
+    return;
 
     }
@@ -1423 +1425 @@
     else if (IS_STRING_ITEM(lpitem->fType))
     {
-        register int i;
-        HFONT hfontOld = 0;
-
-        UINT uFormat = (menuBar) ?
-                        DT_CENTER | DT_VCENTER | DT_SINGLELINE :
-                        DT_LEFT | DT_VCENTER | DT_SINGLELINE;
+    register int i;
+    HFONT hfontOld = 0;
+
+    UINT uFormat = (menuBar) ?
+            DT_CENTER | DT_VCENTER | DT_SINGLELINE :
+            DT_LEFT | DT_VCENTER | DT_SINGLELINE;
 
 #ifdef __WIN32OS2__
-        if ( lpitem->fState & MFS_DEFAULT && !fOS2Look)
+    if ( lpitem->fState & MFS_DEFAULT && !fOS2Look)
 #else
-        if ( lpitem->fState & MFS_DEFAULT )
+    if ( lpitem->fState & MFS_DEFAULT )
 #endif
-        {
-             hfontOld = SelectObject( hdc, hMenuFontBold);
-        }
-
-        if (menuBar)
-        {
-            rect.left += MENU_BAR_ITEMS_SPACE / 2;
-            rect.right -= MENU_BAR_ITEMS_SPACE / 2;
-        }
-
-        for (i = 0; lpitem->text[i]; i++)
-            if ((lpitem->text[i] == '\t') || (lpitem->text[i] == '\b'))
-                break;
-
-        if( (TWEAK_WineLook != WIN31_LOOK) && (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, lpitem->text, i, &rect, uFormat );
-                --rect.left; --rect.top; --rect.right; --rect.bottom;
-            }
-            SetTextColor(hdc, RGB(0x80, 0x80, 0x80));
-        }
-
-        DrawTextW( hdc, lpitem->text, i, &rect, uFormat);
-
-        /* paint the shortcut text */
-        if (!menuBar && lpitem->text[i])  /* There's a tab or flush-right char */
-        {
-            if (lpitem->text[i] == '\t')
-            {
-                rect.left = lpitem->xTab;
-                uFormat = DT_LEFT | DT_VCENTER | DT_SINGLELINE;
-            }
-            else
-            {
-                uFormat = DT_RIGHT | DT_VCENTER | DT_SINGLELINE;
-            }
-
-            if( !(TWEAK_WineLook == WIN31_LOOK) && (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, lpitem->text + i + 1, -1, &rect, uFormat );
-                    --rect.left; --rect.top; --rect.right; --rect.bottom;
-                }
-                SetTextColor(hdc, RGB(0x80, 0x80, 0x80));
-            }
-            DrawTextW( hdc, lpitem->text + i + 1, -1, &rect, uFormat );
-        }
-
-        if (hfontOld)
-            SelectObject (hdc, hfontOld);
+    {
+         hfontOld = SelectObject( hdc, hMenuFontBold);
+    }
+
+    if (menuBar)
+    {
+        rect.left += MENU_BAR_ITEMS_SPACE / 2;
+        rect.right -= MENU_BAR_ITEMS_SPACE / 2;
+    }
+
+    for (i = 0; lpitem->text[i]; i++)
+        if ((lpitem->text[i] == '\t') || (lpitem->text[i] == '\b'))
+            break;
+
+    if( (TWEAK_WineLook != WIN31_LOOK) && (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, lpitem->text, i, &rect, uFormat );
+        --rect.left; --rect.top; --rect.right; --rect.bottom;
+        }
+        SetTextColor(hdc, RGB(0x80, 0x80, 0x80));
+    }
+
+    DrawTextW( hdc, lpitem->text, i, &rect, uFormat);
+
+    /* paint the shortcut text */
+    if (!menuBar && lpitem->text[i])  /* There's a tab or flush-right char */
+    {
+        if (lpitem->text[i] == '\t')
+        {
+        rect.left = lpitem->xTab;
+        uFormat = DT_LEFT | DT_VCENTER | DT_SINGLELINE;
+        }
+        else
+        {
+        uFormat = DT_RIGHT | DT_VCENTER | DT_SINGLELINE;
+        }
+
+        if( !(TWEAK_WineLook == WIN31_LOOK) && (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, lpitem->text + i + 1, -1, &rect, uFormat );
+            --rect.left; --rect.top; --rect.right; --rect.bottom;
+        }
+        SetTextColor(hdc, RGB(0x80, 0x80, 0x80));
+        }
+        DrawTextW( hdc, lpitem->text + i + 1, -1, &rect, uFormat );
+    }
+
+    if (hfontOld)
+        SelectObject (hdc, hfontOld);
     }
 }
@@ -1512 +1514 @@
     if(TWEAK_WineLook == WIN31_LOOK)
     {
-        rect.bottom -= POPUP_YSHADE * GetSystemMetrics(SM_CYBORDER);
-        rect.right -= POPUP_XSHADE * GetSystemMetrics(SM_CXBORDER);
+    rect.bottom -= POPUP_YSHADE * GetSystemMetrics(SM_CYBORDER);
+    rect.right -= POPUP_XSHADE * GetSystemMetrics(SM_CXBORDER);
     }
 
@@ -1519 +1521 @@
         && (SelectObject( hdc, hMenuFont)))
     {
-        HPEN hPrevPen;
-
-        Rectangle( hdc, rect.left, rect.top, rect.right, rect.bottom );
-
-        hPrevPen = SelectObject( hdc, GetStockObject( NULL_PEN ) );
-        if( hPrevPen )
-        {
-            INT ropPrev, i;
-            POPUPMENU *menu;
-
-            /* draw 3-d shade */
-            if(TWEAK_WineLook == WIN31_LOOK) {
-                SelectObject( hdc, hShadeBrush );
-                SetBkMode( hdc, TRANSPARENT );
-                ropPrev = SetROP2( hdc, R2_MASKPEN );
-
-                i = rect.right;         /* why SetBrushOrg() doesn't? */
-                PatBlt( hdc, i & 0xfffffffe,
-                          rect.top + POPUP_YSHADE*GetSystemMetrics(SM_CYBORDER),
-                          i%2 + POPUP_XSHADE*GetSystemMetrics(SM_CXBORDER),
-                          rect.bottom - rect.top, 0x00a000c9 );
-                i = rect.bottom;
-                PatBlt( hdc, rect.left + POPUP_XSHADE*GetSystemMetrics(SM_CXBORDER),
-                          i & 0xfffffffe,rect.right - rect.left,
-                          i%2 + POPUP_YSHADE*GetSystemMetrics(SM_CYBORDER), 0x00a000c9 );
-                SelectObject( hdc, hPrevPen );
-                SelectObject( hdc, hPrevBrush );
-                SetROP2( hdc, ropPrev );
-            }
-            else
-                DrawEdge (hdc, &rect, EDGE_RAISED, BF_RECT);
-
-            /* draw menu items */
-
-            menu = MENU_GetMenu( hmenu );
-            if (menu && menu->nItems)
-            {
-                MENUITEM *item;
-                UINT u;
-
-                for (u = menu->nItems, item = menu->items; u > 0; u--, item++)
-                    MENU_DrawMenuItem( hwnd, hmenu, menu->hwndOwner, hdc, item,
-                                       menu->Height, FALSE, ODA_DRAWENTIRE );
-
-            }
-        } else
-        {
-            SelectObject( hdc, hPrevBrush );
-        }
+    HPEN hPrevPen;
+
+    Rectangle( hdc, rect.left, rect.top, rect.right, rect.bottom );
+
+    hPrevPen = SelectObject( hdc, GetStockObject( NULL_PEN ) );
+    if( hPrevPen )
+    {
+        INT ropPrev, i;
+        POPUPMENU *menu;
+
+        /* draw 3-d shade */
+        if(TWEAK_WineLook == WIN31_LOOK) {
+        SelectObject( hdc, hShadeBrush );
+        SetBkMode( hdc, TRANSPARENT );
+        ropPrev = SetROP2( hdc, R2_MASKPEN );
+
+        i = rect.right;     /* why SetBrushOrg() doesn't? */
+        PatBlt( hdc, i & 0xfffffffe,
+              rect.top + POPUP_YSHADE*GetSystemMetrics(SM_CYBORDER),
+              i%2 + POPUP_XSHADE*GetSystemMetrics(SM_CXBORDER),
+              rect.bottom - rect.top, 0x00a000c9 );
+        i = rect.bottom;
+        PatBlt( hdc, rect.left + POPUP_XSHADE*GetSystemMetrics(SM_CXBORDER),
+              i & 0xfffffffe,rect.right - rect.left,
+              i%2 + POPUP_YSHADE*GetSystemMetrics(SM_CYBORDER), 0x00a000c9 );
+        SelectObject( hdc, hPrevPen );
+        SelectObject( hdc, hPrevBrush );
+        SetROP2( hdc, ropPrev );
+        }
+        else
+        DrawEdge (hdc, &rect, EDGE_RAISED, BF_RECT);
+
+        /* draw menu items */
+
+        menu = MENU_GetMenu( hmenu );
+        if (menu && menu->nItems)
+        {
+        MENUITEM *item;
+        UINT u;
+
+        for (u = menu->nItems, item = menu->items; u > 0; u--, item++)
+            MENU_DrawMenuItem( hwnd, hmenu, menu->hwndOwner, hdc, item,
+                       menu->Height, FALSE, ODA_DRAWENTIRE );
+
+        }
+    } else
+    {
+        SelectObject( hdc, hPrevBrush );
+    }
     }
 }
@@ -1611 +1613 @@
     if (TWEAK_WineLook == WIN31_LOOK)
     {
-        SelectObject( hDC, GetSysColorPen(COLOR_WINDOWFRAME) );
-        MoveToEx( hDC, lprect->left, lprect->bottom, NULL );
-        LineTo( hDC, lprect->right, lprect->bottom );
+    SelectObject( hDC, GetSysColorPen(COLOR_WINDOWFRAME) );
+    MoveToEx( hDC, lprect->left, lprect->bottom, NULL );
+    LineTo( hDC, lprect->right, lprect->bottom );
     }
     else
     {
-        SelectObject( hDC, GetSysColorPen(COLOR_3DFACE));
-        MoveToEx( hDC, lprect->left, lprect->bottom, NULL );
-        LineTo( hDC, lprect->right, lprect->bottom );
+    SelectObject( hDC, GetSysColorPen(COLOR_3DFACE));
+    MoveToEx( hDC, lprect->left, lprect->bottom, NULL );
+    LineTo( hDC, lprect->right, lprect->bottom );
     }
 
@@ -1658 +1660 @@
     if (menu->FocusedItem != NO_SELECTED_ITEM)
     {
-        menu->items[menu->FocusedItem].fState &= ~(MF_HILITE|MF_MOUSESELECT);
-        menu->FocusedItem = NO_SELECTED_ITEM;
+    menu->items[menu->FocusedItem].fState &= ~(MF_HILITE|MF_MOUSESELECT);
+    menu->FocusedItem = NO_SELECTED_ITEM;
     }
 
@@ -1737 +1739 @@
     if (lppop->FocusedItem != NO_SELECTED_ITEM)
     {
-        lppop->items[lppop->FocusedItem].fState &= ~(MF_HILITE|MF_MOUSESELECT);
-        MENU_DrawMenuItem(lppop->hWnd, hmenu, hwndOwner, hdc,&lppop->items[lppop->FocusedItem],
+    lppop->items[lppop->FocusedItem].fState &= ~(MF_HILITE|MF_MOUSESELECT);
+    MENU_DrawMenuItem(lppop->hWnd, hmenu, hwndOwner, hdc,&lppop->items[lppop->FocusedItem],
                           lppop->Height, !(lppop->wFlags & MF_POPUP),
-                          ODA_SELECT );
+              ODA_SELECT );
     }
 
@@ -1756 +1758 @@
         {
             MENUITEM *ip = &lppop->items[lppop->FocusedItem];
-            SendMessageA( hwndOwner, WM_MENUSELECT,
+        SendMessageA( hwndOwner, WM_MENUSELECT,
                      MAKELONG(ip->fType & MF_POPUP ? wIndex: ip->wID,
                      ip->fType | ip->fState | MF_MOUSESELECT |
@@ -1797 +1799 @@
     if ( menu->FocusedItem != NO_SELECTED_ITEM )
     {
-        if( menu->nItems == 1 ) return; else
-        for (i = menu->FocusedItem + offset ; i >= 0 && i < menu->nItems
-                                            ; i += offset)
-            if (!(menu->items[i].fType & MF_SEPARATOR))
-            {
-                MENU_SelectItem( hwndOwner, hmenu, i, TRUE, 0 );
-                return;
-            }
+    if( menu->nItems == 1 ) return; else
+    for (i = menu->FocusedItem + offset ; i >= 0 && i < menu->nItems
+                        ; i += offset)
+        if (!(menu->items[i].fType & MF_SEPARATOR))
+        {
+        MENU_SelectItem( hwndOwner, hmenu, i, TRUE, 0 );
+        return;
+        }
     }
 
     for ( i = (offset > 0) ? 0 : menu->nItems - 1;
-                  i >= 0 && i < menu->nItems ; i += offset)
-        if (!(menu->items[i].fType & MF_SEPARATOR))
-        {
-            MENU_SelectItem( hwndOwner, hmenu, i, TRUE, 0 );
-            return;
-        }
+          i >= 0 && i < menu->nItems ; i += offset)
+    if (!(menu->items[i].fType & MF_SEPARATOR))
+    {
+        MENU_SelectItem( hwndOwner, hmenu, i, TRUE, 0 );
+        return;
+    }
 }
 
@@ -1863 +1865 @@
 
     if ((item->fType & MF_POPUP) && (flags & MF_POPUP) && (item->hSubMenu != id) )
-        DestroyMenu( item->hSubMenu );   /* ModifyMenu() spec */
+    DestroyMenu( item->hSubMenu );   /* ModifyMenu() spec */
 
     if (flags & MF_POPUP)
     {
-        POPUPMENU *menu = MENU_GetMenu((UINT16)id);
+    POPUPMENU *menu = MENU_GetMenu((UINT16)id);
         if (menu) menu->wFlags |= MF_POPUP;
-        else
+    else
         {
             item->wID = 0;
@@ -1875 +1877 @@
             item->fType = 0;
             item->fState = 0;
-            return FALSE;
+        return FALSE;
         }
     }
@@ -1929 +1931 @@
 
     /* Create new items array */
-
     newItems = HeapAlloc( GetProcessHeap(), 0, sizeof(MENUITEM) * (menu->nItems+1) );
     if (!newItems)
@@ -1938 +1939 @@
     if (menu->nItems > 0)
     {
-          /* Copy the old array into the new one */
-        if (pos > 0) memcpy( newItems, menu->items, pos * sizeof(MENUITEM) );
-        if (pos < menu->nItems) memcpy( &newItems[pos+1], &menu->items[pos],
-                                        (menu->nItems-pos)*sizeof(MENUITEM) );
+      /* Copy the old array into the new one */
+    if (pos > 0) memcpy( newItems, menu->items, pos * sizeof(MENUITEM) );
+    if (pos < menu->nItems) memcpy( &newItems[pos+1], &menu->items[pos],
+                    (menu->nItems-pos)*sizeof(MENUITEM) );
         HeapFree( GetProcessHeap(), 0, menu->items );
     }
@@ -2005 +2006 @@
     WORD resinfo;
     do {
-        MENUITEMINFOW mii;
-
-        mii.cbSize = sizeof(mii);
-        mii.fMask = MIIM_STATE | MIIM_ID | MIIM_TYPE;
-        mii.fType = GET_DWORD(res);
+    MENUITEMINFOW mii;
+
+    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);
+    mii.fState = GET_DWORD(res);
         res += sizeof(DWORD);
-        mii.wID = GET_DWORD(res);
+    mii.wID = GET_DWORD(res);
         res += sizeof(DWORD);
-        resinfo = GET_WORD(res); /* FIXME: for 16-bit apps this is a byte.  */
+    resinfo = GET_WORD(res); /* FIXME: for 16-bit apps this is a byte.  */
         res += sizeof(WORD);
-        /* Align the text on a word boundary.  */
-        res += (~((int)res - 1)) & 1;
-        mii.dwTypeData = (LPWSTR) res;
-        res += (1 + strlenW(mii.dwTypeData)) * sizeof(WCHAR);
-        /* Align the following fields on a dword boundary.  */
-        res += (~((int)res - 1)) & 3;
+    /* Align the text on a word boundary.  */
+    res += (~((int)res - 1)) & 1;
+    mii.dwTypeData = (LPWSTR) res;
+    res += (1 + strlenW(mii.dwTypeData)) * sizeof(WCHAR);
+    /* Align the following fields on a dword boundary.  */
+    res += (~((int)res - 1)) & 3;
 
         TRACE("Menu item: [%08x,%08x,%04x,%04x,%s]\n",
               mii.fType, mii.fState, mii.wID, resinfo, debugstr_w(mii.dwTypeData));
 
-        if (resinfo & 1) {      /* Pop-up? */
-            /* DWORD helpid = GET_DWORD(res); FIXME: use this.  */
-            res += sizeof(DWORD);
-            mii.hSubMenu = CreatePopupMenu();
-            if (!mii.hSubMenu)
-                return NULL;
-            if (!(res = MENUEX_ParseResource(res, mii.hSubMenu))) {
-                DestroyMenu(mii.hSubMenu);
+    if (resinfo & 1) {  /* Pop-up? */
+        /* DWORD helpid = GET_DWORD(res); FIXME: use this.  */
+        res += sizeof(DWORD);
+        mii.hSubMenu = CreatePopupMenu();
+        if (!mii.hSubMenu)
+        return NULL;
+        if (!(res = MENUEX_ParseResource(res, mii.hSubMenu))) {
+        DestroyMenu(mii.hSubMenu);
                 return NULL;
             }
-            mii.fMask |= MIIM_SUBMENU;
-            mii.fType |= MF_POPUP;
-        }
-        else if(!*mii.dwTypeData && !(mii.fType & MF_SEPARATOR))
-        {
-            WARN("Converting NULL menu item %04x, type %04x to SEPARATOR\n",
-                mii.wID, mii.fType);
-            mii.fType |= MF_SEPARATOR;
-        }
-        InsertMenuItemW(hMenu, -1, MF_BYPOSITION, &mii);
+        mii.fMask |= MIIM_SUBMENU;
+        mii.fType |= MF_POPUP;
+        }
+    else if(!*mii.dwTypeData && !(mii.fType & MF_SEPARATOR))
+    {
+        WARN("Converting NULL menu item %04x, type %04x to SEPARATOR\n",
+        mii.wID, mii.fType);
+        mii.fType |= MF_SEPARATOR;
+    }
+    InsertMenuItemW(hMenu, -1, MF_BYPOSITION, &mii);
     } while (!(resinfo & MF_END));
     return res;
@@ -2087 +2088 @@
     if (menu && top_popup)
     {
-        HMENU hsubmenu;
-        POPUPMENU *submenu;
-        MENUITEM *item;
-
-        if (menu->FocusedItem != NO_SELECTED_ITEM)
-        {
-            item = &menu->items[menu->FocusedItem];
-            if (!(item->fType & MF_POPUP) ||
-                !(item->fState & MF_MOUSESELECT)) return;
-            item->fState &= ~MF_MOUSESELECT;
-            hsubmenu = item->hSubMenu;
-        } else return;
-
-        submenu = MENU_GetMenu( hsubmenu );
-        MENU_HideSubPopups( hwndOwner, hsubmenu, FALSE );
-        MENU_SelectItem( hwndOwner, hsubmenu, NO_SELECTED_ITEM, sendMenuSelect, 0 );
+    HMENU hsubmenu;
+    POPUPMENU *submenu;
+    MENUITEM *item;
+
+    if (menu->FocusedItem != NO_SELECTED_ITEM)
+    {
+        item = &menu->items[menu->FocusedItem];
+        if (!(item->fType & MF_POPUP) ||
+        !(item->fState & MF_MOUSESELECT)) return;
+        item->fState &= ~MF_MOUSESELECT;
+        hsubmenu = item->hSubMenu;
+    } else return;
+
+    submenu = MENU_GetMenu( hsubmenu );
+    MENU_HideSubPopups( hwndOwner, hsubmenu, FALSE );
+    MENU_SelectItem( hwndOwner, hsubmenu, NO_SELECTED_ITEM, sendMenuSelect, 0 );
         DestroyWindow( submenu->hWnd );
         submenu->hWnd = 0;
@@ -2139 +2140 @@
     if (!(wFlags & TPM_NONOTIFY))
        SendMessageA( hwndOwner, WM_INITMENUPOPUP, item->hSubMenu,
-                   MAKELONG( menu->FocusedItem, IS_SYSTEM_MENU(menu) ));
+           MAKELONG( menu->FocusedItem, IS_SYSTEM_MENU(menu) ));
 
     item = &menu->items[menu->FocusedItem];
@@ -2154 +2155 @@
         item->fState |= MF_HILITE;
         MENU_DrawMenuItem( menu->hWnd, hmenu, hwndOwner, hdc, item, menu->Height, !(menu->wFlags & MF_POPUP), ODA_DRAWENTIRE );
-        ReleaseDC( menu->hWnd, hdc );
+    ReleaseDC( menu->hWnd, hdc );
     }
     if (!item->rect.top && !item->rect.left && !item->rect.bottom && !item->rect.right)
@@ -2163 +2164 @@
     if (IS_SYSTEM_MENU(menu))
     {
-        MENU_InitSysMenuPopup(item->hSubMenu,
+    MENU_InitSysMenuPopup(item->hSubMenu,
                               GetWindowLongA( menu->hWnd, GWL_STYLE ),
                               GetClassLongA( menu->hWnd, GCL_STYLE));
 
-        NC_GetSysPopupPos( menu->hWnd, &rect );
-        rect.top = rect.bottom;
-        rect.right = GetSystemMetrics(SM_CXSIZE);
+    NC_GetSysPopupPos( menu->hWnd, &rect );
+    rect.top = rect.bottom;
+    rect.right = GetSystemMetrics(SM_CXSIZE);
         rect.bottom = GetSystemMetrics(SM_CYSIZE);
     }
@@ -2175 +2176 @@
     {
         GetWindowRect( menu->hWnd, &rect );
-        if (menu->wFlags & MF_POPUP)
-        {
-            rect.left += item->rect.right - GetSystemMetrics(SM_CXBORDER);
-            rect.top += item->rect.top;
-            rect.right = item->rect.left - item->rect.right + GetSystemMetrics(SM_CXBORDER);
-            rect.bottom = item->rect.top - item->rect.bottom;
-        }
-        else
-        {
-            rect.left += item->rect.left;
-            rect.top += item->rect.bottom;
-            rect.right = item->rect.right - item->rect.left;
-            rect.bottom = item->rect.bottom - item->rect.top;
-        }
+    if (menu->wFlags & MF_POPUP)
+    {
+        rect.left += item->rect.right - GetSystemMetrics(SM_CXBORDER);
+        rect.top += item->rect.top;
+        rect.right = item->rect.left - item->rect.right + GetSystemMetrics(SM_CXBORDER);
+        rect.bottom = item->rect.top - item->rect.bottom;
+    }
+    else
+    {
+        rect.left += item->rect.left;
+        rect.top += item->rect.bottom;
+        rect.right = item->rect.right - item->rect.left;
+        rect.bottom = item->rect.bottom - item->rect.top;
+    }
     }
 
     MENU_ShowPopup( hwndOwner, item->hSubMenu, menu->FocusedItem,
-                    rect.left, rect.top, rect.right, rect.bottom );
+            rect.left, rect.top, rect.right, rect.bottom );
     if (selectFirst)
         MENU_MoveSelection( hwndOwner, item->hSubMenu, ITEM_NEXT );
@@ -2258 +2259 @@
 
     if (!menu || !menu->nItems ||
-        (menu->FocusedItem == NO_SELECTED_ITEM)) return -1;
+    (menu->FocusedItem == NO_SELECTED_ITEM)) return -1;
 
     item = &menu->items[menu->FocusedItem];
@@ -2267 +2268 @@
     if (!(item->fType & MF_POPUP))
     {
-        if (!(item->fState & (MF_GRAYED | MF_DISABLED)) && !(item->fType & MF_SEPARATOR))
-        {
-            /* If TPM_RETURNCMD is set you return the id, but
-               do not send a message to the owner */
-            if(!(wFlags & TPM_RETURNCMD))
-            {
-                if( menu->wFlags & MF_SYSMENU )
-                    PostMessageA( pmt->hOwnerWnd, WM_SYSCOMMAND, item->wID,
-                                  MAKELPARAM((INT16)pmt->pt.x, (INT16)pmt->pt.y) );
-                else
-                    PostMessageA( pmt->hOwnerWnd, WM_COMMAND, item->wID, 0 );
-            }
-            return item->wID;
-        }
+    if (!(item->fState & (MF_GRAYED | MF_DISABLED)) && !(item->fType & MF_SEPARATOR))
+    {
+        /* If TPM_RETURNCMD is set you return the id, but
+           do not send a message to the owner */
+        if(!(wFlags & TPM_RETURNCMD))
+        {
+        if( menu->wFlags & MF_SYSMENU )
+            PostMessageA( pmt->hOwnerWnd, WM_SYSCOMMAND, item->wID,
+                  MAKELPARAM((INT16)pmt->pt.x, (INT16)pmt->pt.y) );
+        else
+            PostMessageA( pmt->hOwnerWnd, WM_COMMAND, item->wID, 0 );
+        }
+        return item->wID;
+    }
     }
     else
-        pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd, hMenu, TRUE, wFlags);
+    pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd, hMenu, TRUE, wFlags);
 
     return -1;
@@ -2301 +2302 @@
 
     if( pmt->hTopMenu != hPtMenu &&
-        !((ptmenu->wFlags | topmenu->wFlags) & MF_POPUP) )
-    {
-        /* both are top level menus (system and menu-bar) */
-        MENU_HideSubPopups( pmt->hOwnerWnd, pmt->hTopMenu, FALSE );
-        MENU_SelectItem( pmt->hOwnerWnd, pmt->hTopMenu, NO_SELECTED_ITEM, FALSE, 0 );
+    !((ptmenu->wFlags | topmenu->wFlags) & MF_POPUP) )
+    {
+    /* both are top level menus (system and menu-bar) */
+    MENU_HideSubPopups( pmt->hOwnerWnd, pmt->hTopMenu, FALSE );
+    MENU_SelectItem( pmt->hOwnerWnd, pmt->hTopMenu, NO_SELECTED_ITEM, FALSE, 0 );
         pmt->hTopMenu = hPtMenu;
     }
@@ -2324 +2325 @@
     if (hPtMenu)
     {
-        UINT id = 0;
-        POPUPMENU *ptmenu = MENU_GetMenu( hPtMenu );
-        MENUITEM *item;
-
-        if( IS_SYSTEM_MENU(ptmenu) )
-            item = ptmenu->items;
-        else
-            item = MENU_FindItemByCoords( ptmenu, pmt->pt, &id );
-
-        if( item )
-        {
-            if( ptmenu->FocusedItem != id )
-                MENU_SwitchTracking( pmt, hPtMenu, id );
-
-            /* If the popup menu is not already "popped" */
-            if(!(item->fState & MF_MOUSESELECT ))
-            {
-                pmt->hCurrentMenu = MENU_ShowSubPopup( pmt->hOwnerWnd, hPtMenu, FALSE, wFlags );
-
-                /* In win31, a newly popped menu always remains opened for the next buttonup */
+    UINT id = 0;
+    POPUPMENU *ptmenu = MENU_GetMenu( hPtMenu );
+    MENUITEM *item;
+
+    if( IS_SYSTEM_MENU(ptmenu) )
+        item = ptmenu->items;
+    else
+        item = MENU_FindItemByCoords( ptmenu, pmt->pt, &id );
+
+    if( item )
+    {
+        if( ptmenu->FocusedItem != id )
+        MENU_SwitchTracking( pmt, hPtMenu, id );
+
+        /* If the popup menu is not already "popped" */
+        if(!(item->fState & MF_MOUSESELECT ))
+        {
+        pmt->hCurrentMenu = MENU_ShowSubPopup( pmt->hOwnerWnd, hPtMenu, FALSE, wFlags );
+
+        /* In win31, a newly popped menu always remains opened for the next buttonup */
 #ifdef __WIN32OS2__
-                if(fOS2Look)
-                    ptmenu->bTimeToHide = FALSE;                    
+        if(fOS2Look)
+            ptmenu->bTimeToHide = FALSE;
 #endif
-                if(TWEAK_WineLook == WIN31_LOOK)
-                    ptmenu->bTimeToHide = FALSE;
-            }
-
-            return TRUE;
-        }
-        /* Else the click was on the menu bar, finish the tracking */
+        if(TWEAK_WineLook == WIN31_LOOK)
+            ptmenu->bTimeToHide = FALSE;
+        }
+
+        return TRUE;
+    }
+    /* Else the click was on the menu bar, finish the tracking */
     }
     return FALSE;
@@ -2373 +2374 @@
     if (hPtMenu)
     {
-        UINT id = 0;
-        POPUPMENU *ptmenu = MENU_GetMenu( hPtMenu );
-        MENUITEM *item;
+    UINT id = 0;
+    POPUPMENU *ptmenu = MENU_GetMenu( hPtMenu );
+    MENUITEM *item;
 
         if( IS_SYSTEM_MENU(ptmenu) )
@@ -2382 +2383 @@
             item = MENU_FindItemByCoords( ptmenu, pmt->pt, &id );
 
-        if( item && (ptmenu->FocusedItem == id ))
-        {
-            if( !(item->fType & MF_POPUP) )
-                return MENU_ExecFocusedItem( pmt, hPtMenu, wFlags);
-
-            /* 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((pmt->hTopMenu == hPtMenu) && (ptmenu->bTimeToHide == TRUE))
-                return 0;
-        }
-        ptmenu->bTimeToHide = TRUE;
+    if( item && (ptmenu->FocusedItem == id ))
+    {
+        if( !(item->fType & MF_POPUP) )
+        return MENU_ExecFocusedItem( pmt, hPtMenu, wFlags);
+
+        /* 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((pmt->hTopMenu == hPtMenu) && (ptmenu->bTimeToHide == TRUE))
+        return 0;
+    }
+    ptmenu->bTimeToHide = TRUE;
     }
     return -1;
@@ -2411 +2412 @@
     if( hPtMenu )
     {
-        ptmenu = MENU_GetMenu( hPtMenu );
+    ptmenu = MENU_GetMenu( hPtMenu );
         if( IS_SYSTEM_MENU(ptmenu) )
-            id = 0;
+        id = 0;
         else
             MENU_FindItemByCoords( ptmenu, pmt->pt, &id );
@@ -2420 +2421 @@
     if( id == NO_SELECTED_ITEM )
     {
-        MENU_SelectItem( pmt->hOwnerWnd, pmt->hCurrentMenu,
-                         NO_SELECTED_ITEM, TRUE, pmt->hTopMenu);
+    MENU_SelectItem( pmt->hOwnerWnd, pmt->hCurrentMenu,
+             NO_SELECTED_ITEM, TRUE, pmt->hTopMenu);
 
     }
     else if( ptmenu->FocusedItem != id )
     {
-            MENU_SwitchTracking( pmt, hPtMenu, id );
-            pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd, hPtMenu, FALSE, wFlags);
+        MENU_SwitchTracking( pmt, hPtMenu, id );
+        pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd, hPtMenu, FALSE, wFlags);
     }
     return TRUE;
@@ -2446 +2447 @@
     {
         MDINEXTMENU next_menu;
-        HMENU hNewMenu;
-        HWND  hNewWnd;
-        UINT  id = 0;
+    HMENU hNewMenu;
+    HWND  hNewWnd;
+    UINT  id = 0;
 
         next_menu.hmenuIn = (IS_SYSTEM_MENU(menu)) ? GetSubMenu(pmt->hTopMenu,0) : pmt->hTopMenu;
@@ -2455 +2456 @@
         SendMessageW( pmt->hOwnerWnd, WM_NEXTMENU, vk, (LPARAM)&next_menu );
 
-        TRACE("%04x [%04x] -> %04x [%04x]\n",
+    TRACE("%04x [%04x] -> %04x [%04x]\n",
               pmt->hCurrentMenu, pmt->hOwnerWnd, next_menu.hmenuNext, next_menu.hwndNext );
 
-        if (!next_menu.hmenuNext || !next_menu.hwndNext)
-        {
+    if (!next_menu.hmenuNext || !next_menu.hwndNext)
+    {
             DWORD style = GetWindowLongA( pmt->hOwnerWnd, GWL_STYLE );
-            hNewWnd = pmt->hOwnerWnd;
-            if( IS_SYSTEM_MENU(menu) )
-            {
-                /* switch to the menu bar */
+        hNewWnd = pmt->hOwnerWnd;
+        if( IS_SYSTEM_MENU(menu) )
+        {
+        /* switch to the menu bar */
 
                 if(style & WS_CHILD || !(hNewMenu = GetMenu(hNewWnd))) return FALSE;
 
-                if( vk == VK_LEFT )
-                {
-                    menu = MENU_GetMenu( hNewMenu );
-                    id = menu->nItems - 1;
-                }
-            }
-            else if (style & WS_SYSMENU )
-            {
-                /* switch to the system menu */
-                hNewMenu = get_win_sys_menu( hNewWnd );
-            }
+            if( vk == VK_LEFT )
+            {
+            menu = MENU_GetMenu( hNewMenu );
+            id = menu->nItems - 1;
+            }
+        }
+        else if (style & WS_SYSMENU )
+        {
+        /* switch to the system menu */
+            hNewMenu = get_win_sys_menu( hNewWnd );
+        }
             else return FALSE;
-        }
-        else    /* application returned a new menu to switch to */
-        {
+    }
+    else    /* application returned a new menu to switch to */
+    {
             hNewMenu = next_menu.hmenuNext;
             hNewWnd = WIN_GetFullHandle( next_menu.hwndNext );
 
-            if( IsMenu(hNewMenu) && IsWindow(hNewWnd) )
-            {
+        if( IsMenu(hNewMenu) && IsWindow(hNewWnd) )
+        {
                 DWORD style = GetWindowLongA( hNewWnd, GWL_STYLE );
 
-                if (style & WS_SYSMENU &&
-                    GetSubMenu(get_win_sys_menu(hNewWnd), 0) == hNewMenu )
-                {
-                    /* get the real system menu */
-                    hNewMenu =  get_win_sys_menu(hNewWnd);
-                }
-                else if (style & WS_CHILD || GetMenu(hNewWnd) != hNewMenu )
-                {
-                    /* FIXME: Not sure what to do here;
-                     * perhaps try to track hNewMenu as a popup? */
-
-                    TRACE(" -- got confused.\n");
-                    return FALSE;
-                }
-            }
-            else return FALSE;
-        }
-
-        if( hNewMenu != pmt->hTopMenu )
-        {
-            MENU_SelectItem( pmt->hOwnerWnd, pmt->hTopMenu, NO_SELECTED_ITEM,
+        if (style & WS_SYSMENU &&
+            GetSubMenu(get_win_sys_menu(hNewWnd), 0) == hNewMenu )
+        {
+                /* get the real system menu */
+            hNewMenu =  get_win_sys_menu(hNewWnd);
+        }
+            else if (style & WS_CHILD || GetMenu(hNewWnd) != hNewMenu )
+        {
+            /* FIXME: Not sure what to do here;
+             * perhaps try to track hNewMenu as a popup? */
+
+            TRACE(" -- got confused.\n");
+            return FALSE;
+        }
+        }
+        else return FALSE;
+    }
+
+    if( hNewMenu != pmt->hTopMenu )
+    {
+        MENU_SelectItem( pmt->hOwnerWnd, pmt->hTopMenu, NO_SELECTED_ITEM,
                     FALSE, 0 );
-            if( pmt->hCurrentMenu != pmt->hTopMenu )
-                MENU_HideSubPopups( pmt->hOwnerWnd, pmt->hTopMenu, FALSE );
-        }
-
-        if( hNewWnd != pmt->hOwnerWnd )
-        {
-            ReleaseCapture();
-            pmt->hOwnerWnd = hNewWnd;
+        if( pmt->hCurrentMenu != pmt->hTopMenu )
+        MENU_HideSubPopups( pmt->hOwnerWnd, pmt->hTopMenu, FALSE );
+    }
+
+    if( hNewWnd != pmt->hOwnerWnd )
+    {
+        ReleaseCapture();
+        pmt->hOwnerWnd = hNewWnd;
 #ifdef __WIN32OS2__
             SetCapture(pmt->hOwnerWnd); //SvL: Don't know if this is good enough
 #else
-            EVENT_Capture( pmt->hOwnerWnd, HTMENU );
+        EVENT_Capture( pmt->hOwnerWnd, HTMENU );
 #endif
-        }
-
-        pmt->hTopMenu = pmt->hCurrentMenu = hNewMenu; /* all subpopups are hidden */
-        MENU_SelectItem( pmt->hOwnerWnd, pmt->hTopMenu, id, TRUE, 0 );
-
-        return TRUE;
+    }
+
+    pmt->hTopMenu = pmt->hCurrentMenu = hNewMenu; /* all subpopups are hidden */
+    MENU_SelectItem( pmt->hOwnerWnd, pmt->hTopMenu, id, TRUE, 0 );
+
+    return TRUE;
     }
     return FALSE;
@@ -2552 +2553 @@
     switch( uMsg )
     {
-        case WM_KEYDOWN:
-             PeekMessageA( &msg, 0, 0, 0, PM_NOYIELD | PM_NOREMOVE);
-             if( msg.message == WM_KEYUP || msg.message == WM_PAINT )
-             {
-                 PeekMessageA( &msg, 0, 0, 0, PM_NOYIELD | PM_REMOVE);
-                 PeekMessageA( &msg, 0, 0, 0, PM_NOYIELD | PM_NOREMOVE);
-                 if( msg.message == WM_KEYDOWN &&
-                    (msg.wParam == VK_LEFT || msg.wParam == VK_RIGHT))
-                 {
-                     pmt->trackFlags |= TF_SUSPENDPOPUP;
-                     return TRUE;
-                 }
-             }
-             break;
+    case WM_KEYDOWN:
+         PeekMessageA( &msg, 0, 0, 0, PM_NOYIELD | PM_NOREMOVE);
+         if( msg.message == WM_KEYUP || msg.message == WM_PAINT )
+         {
+         PeekMessageA( &msg, 0, 0, 0, PM_NOYIELD | PM_REMOVE);
+             PeekMessageA( &msg, 0, 0, 0, PM_NOYIELD | PM_NOREMOVE);
+             if( msg.message == WM_KEYDOWN &&
+            (msg.wParam == VK_LEFT || msg.wParam == VK_RIGHT))
+             {
+             pmt->trackFlags |= TF_SUSPENDPOPUP;
+             return TRUE;
+             }
+         }
+         break;
     }
 
@@ -2624 +2625 @@
     /* Try to move 1 column left (if possible) */
     if( (prevcol = MENU_GetStartOfPrevColumn( pmt->hCurrentMenu )) !=
-        NO_SELECTED_ITEM ) {
-
-        MENU_SelectItem( pmt->hOwnerWnd, pmt->hCurrentMenu,
-                         prevcol, TRUE, 0 );
-        return;
+    NO_SELECTED_ITEM ) {
+
+    MENU_SelectItem( pmt->hOwnerWnd, pmt->hCurrentMenu,
+             prevcol, TRUE, 0 );
+    return;
     }
 
@@ -2634 +2635 @@
     while (hmenutmp != pmt->hCurrentMenu)
     {
-        hmenuprev = hmenutmp;
-        hmenutmp = MENU_GetSubPopup( hmenuprev );
+    hmenuprev = hmenutmp;
+    hmenutmp = MENU_GetSubPopup( hmenuprev );
     }
 
@@ -2643 +2644 @@
     if ( (hmenuprev == pmt->hTopMenu) && !(menu->wFlags & MF_POPUP) )
     {
-        /* move menu bar selection if no more popups are left */
-
-        if( !MENU_DoNextMenu( pmt, VK_LEFT) )
-             MENU_MoveSelection( pmt->hOwnerWnd, pmt->hTopMenu, ITEM_PREV );
-
-        if ( hmenuprev != hmenutmp || pmt->trackFlags & TF_SUSPENDPOPUP )
-        {
-           /* A sublevel menu was displayed - display the next one
-            * unless there is another displacement coming up */
-
-            if( !MENU_SuspendPopup( pmt, WM_KEYDOWN ) )
-                pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd,
-                                                pmt->hTopMenu, TRUE, wFlags);
-        }
+    /* move menu bar selection if no more popups are left */
+
+    if( !MENU_DoNextMenu( pmt, VK_LEFT) )
+         MENU_MoveSelection( pmt->hOwnerWnd, pmt->hTopMenu, ITEM_PREV );
+
+    if ( hmenuprev != hmenutmp || pmt->trackFlags & TF_SUSPENDPOPUP )
+    {
+       /* A sublevel menu was displayed - display the next one
+        * unless there is another displacement coming up */
+
+        if( !MENU_SuspendPopup( pmt, WM_KEYDOWN ) )
+        pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd,
+                        pmt->hTopMenu, TRUE, wFlags);
+    }
     }
 }
@@ -2673 +2674 @@
 
     TRACE("MENU_KeyRight called, cur %x (%s), top %x (%s).\n",
-                  pmt->hCurrentMenu,
-                  debugstr_w((MENU_GetMenu(pmt->hCurrentMenu))->
-                     items[0].text),
-                  pmt->hTopMenu, debugstr_w(menu->items[0].text) );
+          pmt->hCurrentMenu,
+          debugstr_w((MENU_GetMenu(pmt->hCurrentMenu))->
+             items[0].text),
+          pmt->hTopMenu, debugstr_w(menu->items[0].text) );
 
     if ( (menu->wFlags & MF_POPUP) || (pmt->hCurrentMenu != pmt->hTopMenu))
     {
-        /* If already displaying a popup, try to display sub-popup */
-
-        hmenutmp = pmt->hCurrentMenu;
-        pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd, hmenutmp, TRUE, wFlags);
-
-        /* if subpopup was displayed then we are done */
-        if (hmenutmp != pmt->hCurrentMenu) return;
+    /* If already displaying a popup, try to display sub-popup */
+
+    hmenutmp = pmt->hCurrentMenu;
+    pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd, hmenutmp, TRUE, wFlags);
+
+    /* if subpopup was displayed then we are done */
+    if (hmenutmp != pmt->hCurrentMenu) return;
     }
 
     /* Check to see if there's another column */
     if( (nextcol = MENU_GetStartOfNextColumn( pmt->hCurrentMenu )) !=
-        NO_SELECTED_ITEM ) {
-        TRACE("Going to %d.\n", nextcol );
-        MENU_SelectItem( pmt->hOwnerWnd, pmt->hCurrentMenu,
-                         nextcol, TRUE, 0 );
-        return;
-    }
-
-    if (!(menu->wFlags & MF_POPUP))     /* menu bar tracking */
-    {
-        if( pmt->hCurrentMenu != pmt->hTopMenu )
-        {
-            MENU_HideSubPopups( pmt->hOwnerWnd, pmt->hTopMenu, FALSE );
-            hmenutmp = pmt->hCurrentMenu = pmt->hTopMenu;
-        } else hmenutmp = 0;
-
-        /* try to move to the next item */
-        if( !MENU_DoNextMenu( pmt, VK_RIGHT) )
-             MENU_MoveSelection( pmt->hOwnerWnd, pmt->hTopMenu, ITEM_NEXT );
-
-        if( hmenutmp || pmt->trackFlags & TF_SUSPENDPOPUP )
-            if( !MENU_SuspendPopup(pmt, WM_KEYDOWN) )
-                pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd,
-                                                       pmt->hTopMenu, TRUE, wFlags);
+    NO_SELECTED_ITEM ) {
+    TRACE("Going to %d.\n", nextcol );
+    MENU_SelectItem( pmt->hOwnerWnd, pmt->hCurrentMenu,
+             nextcol, TRUE, 0 );
+    return;
+    }
+
+    if (!(menu->wFlags & MF_POPUP)) /* menu bar tracking */
+    {
+    if( pmt->hCurrentMenu != pmt->hTopMenu )
+    {
+        MENU_HideSubPopups( pmt->hOwnerWnd, pmt->hTopMenu, FALSE );
+        hmenutmp = pmt->hCurrentMenu = pmt->hTopMenu;
+    } else hmenutmp = 0;
+
+    /* try to move to the next item */
+    if( !MENU_DoNextMenu( pmt, VK_RIGHT) )
+         MENU_MoveSelection( pmt->hOwnerWnd, pmt->hTopMenu, ITEM_NEXT );
+
+    if( hmenutmp || pmt->trackFlags & TF_SUSPENDPOPUP )
+        if( !MENU_SuspendPopup(pmt, WM_KEYDOWN) )
+        pmt->hCurrentMenu = MENU_ShowSubPopup(pmt->hOwnerWnd,
+                               pmt->hTopMenu, TRUE, wFlags);
     }
 }
@@ -2743 +2744 @@
 
     TRACE("hmenu=0x%04x flags=0x%08x (%d,%d) hwnd=0x%04x (%d,%d)-(%d,%d)\n",
-            hmenu, wFlags, x, y, hwnd, (lprect) ? lprect->left : 0, (lprect) ? lprect->top : 0,
-            (lprect) ? lprect->right : 0,  (lprect) ? lprect->bottom : 0);
+        hmenu, wFlags, x, y, hwnd, (lprect) ? lprect->left : 0, (lprect) ? lprect->top : 0,
+        (lprect) ? lprect->right : 0,  (lprect) ? lprect->bottom : 0);
 
     fEndMenu = FALSE;
@@ -2751 +2752 @@
     if (wFlags & TPM_BUTTONDOWN)
     {
-        /* Get the result in order to start the tracking or not */
-        fRemove = MENU_ButtonDown( &mt, hmenu, wFlags );
-        fEndMenu = !fRemove;
+    /* Get the result in order to start the tracking or not */
+    fRemove = MENU_ButtonDown( &mt, hmenu, wFlags );
+    fEndMenu = !fRemove;
     }
 
@@ -2769 +2770 @@
     while (!fEndMenu)
     {
-        menu = MENU_GetMenu( mt.hCurrentMenu );
-        if (!menu) /* sometimes happens if I do a window manager close */
-            break;
-
-        /* we have to keep the message in the queue until it's
-         * clear that menu loop is not over yet. */
+    menu = MENU_GetMenu( mt.hCurrentMenu );
+    if (!menu) /* sometimes happens if I do a window manager close */
+        break;
+
+    /* we have to keep the message in the queue until it's
+     * clear that menu loop is not over yet. */
 
         for (;;)
@@ -2803 +2804 @@
         }
 
-        /* check if EndMenu() tried to cancel us, by posting this message */
+    /* 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;
+    {
+        /* we are now out of the loop */
+            fEndMenu = TRUE;
 
 #ifdef __WIN32OS2__
-            /* remove the message from the queue */
-            PeekMessageA( &msg, 0, msg.message, msg.message, PM_REMOVE );
+        /* remove the message from the queue */
+        PeekMessageA( &msg, 0, msg.message, msg.message, PM_REMOVE );
 #endif
-            /* break out of internal loop, ala ESCAPE */
-            break;
-        }
+        /* break out of internal loop, ala ESCAPE */
+        break;
+    }
 
         TranslateMessage( &msg );
         mt.pt = msg.pt;
 
-        if ( (msg.hwnd==menu->hWnd) || (msg.message!=WM_TIMER) )
-          enterIdleSent=FALSE;
+    if ( (msg.hwnd==menu->hWnd) || (msg.message!=WM_TIMER) )
+      enterIdleSent=FALSE;
 
         fRemove = FALSE;
-        if ((msg.message >= WM_MOUSEFIRST) && (msg.message <= WM_MOUSELAST))
-        {
+    if ((msg.message >= WM_MOUSEFIRST) && (msg.message <= WM_MOUSELAST))
+    {
             /*
              * use the mouse coordinates in lParam instead of those in the MSG
@@ -2839 +2840 @@
             ClientToScreen(msg.hwnd,&mt.pt);
 
-            /* Find a menu for this mouse event */
-            hmenu = MENU_PtMenu( mt.hTopMenu, mt.pt );
-
-            switch(msg.message)
-            {
-                /* no WM_NC... messages in captured state */
-
-                case WM_RBUTTONDBLCLK:
-                case WM_RBUTTONDOWN:
+        /* Find a menu for this mouse event */
+        hmenu = MENU_PtMenu( mt.hTopMenu, mt.pt );
+
+        switch(msg.message)
+        {
+        /* no WM_NC... messages in captured state */
+
+        case WM_RBUTTONDBLCLK:
+        case WM_RBUTTONDOWN:
 #ifdef __WIN32OS2__
-                    if (!(wFlags & TPM_RIGHTBUTTON)) {
+            if (!(wFlags & TPM_RIGHTBUTTON)) {
                       MENUITEM *item;
-                      UINT id = 0;
-                        
+              UINT id = 0;
+
                        if( IS_SYSTEM_MENU(menu) )
                           item = menu->items;
                        else
                           item = MENU_FindItemByCoords( menu, mt.pt, &id );
-                       //@@PF If our pointer is over the menu - do nothing
-                       if (item) break; 
-                       //@@PF Time to close menu - win2k&98 checked
+               //@@PF If our pointer is over the menu - do nothing
+                       if (item) break;
+               //@@PF Time to close menu - win2k&98 checked
                        fEndMenu = 1;
                        break;
@@ -2865 +2866 @@
                     goto buttondown;
 #else
-                    if (!(wFlags & TPM_RIGHTBUTTON)) break;
+            if (!(wFlags & TPM_RIGHTBUTTON)) break;
 #endif
-                    /* fall through */
-                case WM_LBUTTONDBLCLK:
+            /* fall through */
+        case WM_LBUTTONDBLCLK:
 #ifdef __WIN32OS2__
                     if (bSysMenu && (hmenu == mt.hTopMenu))
@@ -2889 +2890 @@
 
 #endif
-                case WM_LBUTTONDOWN:
-                    /* If the message belongs to the menu, removes it from the queue */
-                    /* Else, end menu tracking */
-                    fRemove = MENU_ButtonDown( &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)
-                    {
+        case WM_LBUTTONDOWN:
+            /* If the message belongs to the menu, removes it from the queue */
+            /* Else, end menu tracking */
+            fRemove = MENU_ButtonDown( &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 = MENU_ButtonUp( &mt, hmenu, wFlags);
 
-                        /* End the loop if executedMenuId is an item ID */
-                        /* or if the job was done (executedMenuId = 0). */
+            /* 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
@@ -2915 +2916 @@
                         fEndMenu = ((wFlags & TPM_POPUPMENU) ? FALSE : TRUE);
 
-                    break;
-
-                case WM_MOUSEMOVE:
+            break;
+
+        case WM_MOUSEMOVE:
                     /* In win95 winelook, the selected menu item must be changed every time the
                        mouse moves. In Win31 winelook, the mouse button has to be held down */
@@ -2924 +2925 @@
                     //PF Win32 eats first mousemove to prevent auto-select of item
                     //on TrackMenuPopup pressed button - verified in Win2k
-                    if (bFirstMouseMove) 
-                    { 
-                        bFirstMouseMove = FALSE; 
+                    if (bFirstMouseMove)
+                    {
+                        bFirstMouseMove = FALSE;
                         break;
                     }
@@ -2937 +2938 @@
                            ((wFlags & TPM_RIGHTBUTTON) && (msg.wParam & MK_RBUTTON))) )
 
-                        fEndMenu |= !MENU_MouseMove( &mt, hmenu, wFlags );
-
-            } /* switch(msg.message) - mouse */
-        }
-        else if ((msg.message >= WM_KEYFIRST) && (msg.message <= WM_KEYLAST))
-        {
+            fEndMenu |= !MENU_MouseMove( &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:
-                switch(msg.wParam)
-                {
-                case VK_HOME:
-                case VK_END:
-                    MENU_SelectItem( mt.hOwnerWnd, mt.hCurrentMenu,
-                                     NO_SELECTED_ITEM, FALSE, 0 );
-                /* fall through */
-                case VK_UP:
-                    MENU_MoveSelection( mt.hOwnerWnd, mt.hCurrentMenu,
-                                       (msg.wParam == VK_HOME)? ITEM_NEXT : ITEM_PREV );
-                    break;
-
-                case VK_DOWN: /* If on menu bar, pull-down the menu */
-
-                    menu = MENU_GetMenu( mt.hCurrentMenu );
-                    if (!(menu->wFlags & MF_POPUP))
-                        mt.hCurrentMenu = MENU_ShowSubPopup(mt.hOwnerWnd, mt.hTopMenu, TRUE, wFlags);
-                    else      /* otherwise try to move selection */
-                        MENU_MoveSelection( mt.hOwnerWnd, mt.hCurrentMenu, ITEM_NEXT );
-                    break;
-
-                case VK_LEFT:
-                    MENU_KeyLeft( &mt, wFlags );
-                    break;
-
-                case VK_RIGHT:
-                    MENU_KeyRight( &mt, wFlags );
-                    break;
-
-                case VK_ESCAPE:
+        switch(msg.message)
+        {
+        case WM_KEYDOWN:
+        switch(msg.wParam)
+        {
+        case VK_HOME:
+        case VK_END:
+            MENU_SelectItem( mt.hOwnerWnd, mt.hCurrentMenu,
+                     NO_SELECTED_ITEM, FALSE, 0 );
+        /* fall through */
+        case VK_UP:
+            MENU_MoveSelection( mt.hOwnerWnd, mt.hCurrentMenu,
+                       (msg.wParam == VK_HOME)? ITEM_NEXT : ITEM_PREV );
+            break;
+
+        case VK_DOWN: /* If on menu bar, pull-down the menu */
+
+            menu = MENU_GetMenu( mt.hCurrentMenu );
+            if (!(menu->wFlags & MF_POPUP))
+            mt.hCurrentMenu = MENU_ShowSubPopup(mt.hOwnerWnd, mt.hTopMenu, TRUE, wFlags);
+            else      /* otherwise try to move selection */
+            MENU_MoveSelection( mt.hOwnerWnd, mt.hCurrentMenu, ITEM_NEXT );
+            break;
+
+        case VK_LEFT:
+            MENU_KeyLeft( &mt, wFlags );
+            break;
+
+        case VK_RIGHT:
+            MENU_KeyRight( &mt, wFlags );
+            break;
+
+        case VK_ESCAPE:
                     fEndMenu = MENU_KeyEscape(&mt, wFlags);
-                    break;
-
-                case VK_F1:
-                    {
-                        HELPINFO hi;
-                        hi.cbSize = sizeof(HELPINFO);
-                        hi.iContextType = HELPINFO_MENUITEM;
-                        if (menu->FocusedItem == NO_SELECTED_ITEM)
-                            hi.iCtrlId = 0;
-                        else
-                            hi.iCtrlId = menu->items[menu->FocusedItem].wID;
-                        hi.hItemHandle = hmenu;
-                        hi.dwContextId = menu->dwContextHelpID;
-                        hi.MousePos = msg.pt;
-                        SendMessageA(hwnd, WM_HELP, 0, (LPARAM)&hi);
-                        break;
-                    }
-
-                default:
-                    break;
-                }
-                break;  /* WM_KEYDOWN */
-
-            case WM_SYSKEYDOWN:
-                switch(msg.wParam)
-                {
-                case VK_MENU:
-                    fEndMenu = TRUE;
-                    break;
-
-                }
-                break;  /* WM_SYSKEYDOWN */
-
-            case WM_CHAR:
-                {
-                    UINT        pos;
-
-                    if (msg.wParam == '\r' || msg.wParam == ' ')
-                    {
+            break;
+
+        case VK_F1:
+            {
+            HELPINFO hi;
+            hi.cbSize = sizeof(HELPINFO);
+            hi.iContextType = HELPINFO_MENUITEM;
+            if (menu->FocusedItem == NO_SELECTED_ITEM)
+                hi.iCtrlId = 0;
+                else
+                hi.iCtrlId = menu->items[menu->FocusedItem].wID;
+            hi.hItemHandle = hmenu;
+            hi.dwContextId = menu->dwContextHelpID;
+            hi.MousePos = msg.pt;
+            SendMessageA(hwnd, WM_HELP, 0, (LPARAM)&hi);
+            break;
+            }
+
+        default:
+            break;
+        }
+        break;  /* WM_KEYDOWN */
+
+        case WM_SYSKEYDOWN:
+        switch(msg.wParam)
+        {
+        case VK_MENU:
+            fEndMenu = TRUE;
+            break;
+
+        }
+        break;  /* WM_SYSKEYDOWN */
+
+        case WM_CHAR:
+        {
+            UINT    pos;
+
+            if (msg.wParam == '\r' || msg.wParam == ' ')
+            {
                         executedMenuId = MENU_ExecFocusedItem(&mt,mt.hCurrentMenu, wFlags);
                         fEndMenu = (executedMenuId != -1);
 
-                        break;
-                    }
-
-                      /* Hack to avoid control chars. */
-                      /* We will find a better way real soon... */
-                    if ((msg.wParam <= 32) || (msg.wParam >= 127)) break;
-
-                    pos = MENU_FindItemByKey( mt.hOwnerWnd, mt.hCurrentMenu,
+            break;
+            }
+
+              /* Hack to avoid control chars. */
+              /* We will find a better way real soon... */
+            if ((msg.wParam <= 32) || (msg.wParam >= 127)) break;
+
+            pos = MENU_FindItemByKey( mt.hOwnerWnd, mt.hCurrentMenu,
                                               LOWORD(msg.wParam), FALSE );
-                    if (pos == (UINT)-2) fEndMenu = TRUE;
-                    else if (pos == (UINT)-1) MessageBeep(0);
-                    else
-                    {
-                        MENU_SelectItem( mt.hOwnerWnd, mt.hCurrentMenu, pos,
+            if (pos == (UINT)-2) fEndMenu = TRUE;
+            else if (pos == (UINT)-1) MessageBeep(0);
+            else
+            {
+            MENU_SelectItem( mt.hOwnerWnd, mt.hCurrentMenu, pos,
                                 TRUE, 0 );
                         executedMenuId = MENU_ExecFocusedItem(&mt,mt.hCurrentMenu, wFlags);
                         fEndMenu = (executedMenuId != -1);
-                    }
-                }
-                break;
-            }  /* switch(msg.message) - kbd */
-        }
-        else
-        {
-            DispatchMessageA( &msg );
-        }
-
-        if (!fEndMenu) fRemove = TRUE;
-
-        /* finally remove message from the queue */
+            }
+        }
+        break;
+        }  /* switch(msg.message) - kbd */
+    }
+    else
+    {
+        DispatchMessageA( &msg );
+    }
+
+    if (!fEndMenu) fRemove = TRUE;
+
+    /* finally remove message from the queue */
 
         if (fRemove && !(mt.trackFlags & TF_SKIPREMOVE) )
-#ifdef __WIN32OS2__ 
+#ifdef __WIN32OS2__
             ;
 #else
-            PeekMessageA( &msg, 0, msg.message, msg.message, PM_REMOVE );
+        PeekMessageA( &msg, 0, msg.message, msg.message, PM_REMOVE );
 #endif
-        else mt.trackFlags &= ~TF_SKIPREMOVE;
+    else mt.trackFlags &= ~TF_SKIPREMOVE;
     }
 
@@ -3071 +3072 @@
     if( IsMenu( mt.hTopMenu ) )
     {
-        menu = MENU_GetMenu( mt.hTopMenu );
+    menu = MENU_GetMenu( mt.hTopMenu );
 
         if( IsWindow( mt.hOwnerWnd ) )
         {
-            MENU_HideSubPopups( mt.hOwnerWnd, mt.hTopMenu, FALSE );
-
-            if (menu && menu->wFlags & MF_POPUP)
-            {
+        MENU_HideSubPopups( mt.hOwnerWnd, mt.hTopMenu, FALSE );
+
+        if (menu && menu->wFlags & MF_POPUP)
+        {
                 DestroyWindow( menu->hWnd );
                 menu->hWnd = 0;
-            }
-            MENU_SelectItem( mt.hOwnerWnd, mt.hTopMenu, NO_SELECTED_ITEM, FALSE, 0 );
-            SendMessageA( mt.hOwnerWnd, WM_MENUSELECT, MAKELONG(0,0xffff), 0 );
+        }
+        MENU_SelectItem( mt.hOwnerWnd, mt.hTopMenu, NO_SELECTED_ITEM, FALSE, 0 );
+        SendMessageA( mt.hOwnerWnd, WM_MENUSELECT, MAKELONG(0,0xffff), 0 );
         }
 
@@ -3153 +3154 @@
         if (parent != GetDesktopWindow()) ScreenToClient( parent, &pt );
 
-        MENU_InitTracking( hWnd, hMenu, FALSE, wFlags );
-        MENU_TrackMenu( hMenu, wFlags, pt.x, pt.y, hWnd, NULL );
-        MENU_ExitTracking(hWnd);
+    MENU_InitTracking( hWnd, hMenu, FALSE, wFlags );
+    MENU_TrackMenu( hMenu, wFlags, pt.x, pt.y, hWnd, NULL );
+    MENU_ExitTracking(hWnd);
     }
 }
@@ -3241 +3242 @@
 
     if (MENU_ShowPopup( hWnd, hMenu, 0, x, y, 0, 0 ))
-        ret = MENU_TrackMenu( hMenu, wFlags | TPM_POPUPMENU, 0, 0, hWnd, lpRect );
+    ret = MENU_TrackMenu( hMenu, wFlags | TPM_POPUPMENU, 0, 0, hWnd, lpRect );
     MENU_ExitTracking(hWnd);
 
     if( (!(wFlags & TPM_RETURNCMD)) && (ret != FALSE) )
-        ret = 1;
+    ret = 1;
 
     return ret;
@@ -3274 +3275 @@
     {
     case WM_CREATE:
-        {
-            CREATESTRUCTW *cs = (CREATESTRUCTW*)lParam;
-            SetWindowLongW( hwnd, 0, (LONG)cs->lpCreateParams );
+    {
+        CREATESTRUCTW *cs = (CREATESTRUCTW*)lParam;
+        SetWindowLongW( hwnd, 0, (LONG)cs->lpCreateParams );
             return 0;
-        }
+    }
 
     case WM_MOUSEACTIVATE:  /* We don't want to be activated */
@@ -3284 +3285 @@
 
     case WM_PAINT:
-        {
-            PAINTSTRUCT ps;
-            BeginPaint( hwnd, &ps );
-            MENU_DrawPopupMenu( hwnd, ps.hdc,
+    {
+        PAINTSTRUCT ps;
+        BeginPaint( hwnd, &ps );
+        MENU_DrawPopupMenu( hwnd, ps.hdc,
                                 (HMENU)GetWindowLongA( hwnd, 0 ) );
-            EndPaint( hwnd, &ps );
+        EndPaint( hwnd, &ps );
             return 0;
-        }
+    }
     case WM_ERASEBKGND:
         return 1;
@@ -3302 +3303 @@
     case WM_SHOWWINDOW:
 
-        if( wParam )
-        {
+    if( wParam )
+    {
             if (!GetWindowLongW( hwnd, 0 )) ERR("no menu to display\n");
-        }
-        else
+    }
+    else
             SetWindowLongW( hwnd, 0, 0 );
-        break;
+    break;
 
     case MM_SETMENUHANDLE:
@@ -3419 +3420 @@
 
     TRACE("(%04x, %04X, %04X) !\n",
-                 hMenu, wItemID, wFlags);
+         hMenu, wItemID, wFlags);
 
     /* Get the Popupmenu to access the owner menu */
     if (!(menu = MENU_GetMenu(hMenu)))
-        return (UINT)-1;
+    return (UINT)-1;
 
     if (!(item = MENU_FindItem( &hMenu, &wItemID, wFlags )))
-        return (UINT)-1;
+    return (UINT)-1;
 
     oldflags = item->fState & (MF_GRAYED | MF_DISABLED);
@@ -3433 +3434 @@
     /* In win95 if the close item in the system menu change update the close button */
     if (TWEAK_WineLook == WIN95_LOOK)
-        if((item->wID == SC_CLOSE) && (oldflags != wFlags))
-        {
-            if (menu->hSysMenuOwner != 0)
-            {
-                POPUPMENU* parentMenu;
-
-                /* Get the parent menu to access*/
-                if (!(parentMenu = MENU_GetMenu(menu->hSysMenuOwner)))
-                    return (UINT)-1;
-
-                /* Refresh the frame to reflect the change*/
-                SetWindowPos(parentMenu->hWnd, 0, 0, 0, 0, 0,
-                             SWP_NOACTIVATE | SWP_NOMOVE | SWP_NOSIZE | SWP_NOZORDER);
-            }
-        }
+    if((item->wID == SC_CLOSE) && (oldflags != wFlags))
+    {
+        if (menu->hSysMenuOwner != 0)
+        {
+        POPUPMENU* parentMenu;
+
+        /* Get the parent menu to access*/
+        if (!(parentMenu = MENU_GetMenu(menu->hSysMenuOwner)))
+            return (UINT)-1;
+
+        /* Refresh the frame to reflect the change*/
+        SetWindowPos(parentMenu->hWnd, 0, 0, 0, 0, 0,
+                 SWP_NOACTIVATE | SWP_NOMOVE | SWP_NOSIZE | SWP_NOZORDER);
+        }
+    }
 
     return oldflags;
@@ -3457 +3458 @@
  */
 INT WINAPI GetMenuStringA(
-        HMENU hMenu,    /* [in] menuhandle */
-        UINT wItemID,   /* [in] menu item (dep. on wFlags) */
-        LPSTR str,      /* [out] outbuffer. If NULL, func returns entry length*/
-        INT nMaxSiz,    /* [in] length of buffer. if 0, func returns entry len*/
-        UINT wFlags     /* [in] MF_ flags */
+    HMENU hMenu,    /* [in] menuhandle */
+    UINT wItemID,   /* [in] menu item (dep. on wFlags) */
+    LPSTR str,  /* [out] outbuffer. If NULL, func returns entry length*/
+    INT nMaxSiz,    /* [in] length of buffer. if 0, func returns entry len*/
+    UINT wFlags /* [in] MF_ flags */
 ) {
     MENUITEM *item;
@@ -3472 +3473 @@
     str[0] = '\0';
     if (!WideCharToMultiByte( CP_ACP, 0, item->text, -1, str, nMaxSiz, NULL, NULL ))
+#ifdef __WIN32OS2__
+        lstrtrunc( str, nMaxSiz );
+#else
         str[nMaxSiz-1] = 0;
+#endif
     TRACE("returning '%s'\n", str );
     return strlen(str);
@@ -3522 +3527 @@
     MENUITEM *item;
     TRACE("(menu=%04x, id=%04x, flags=%04x);\n",
-                 hMenu, wItemID, wFlags);
+         hMenu, wItemID, wFlags);
     if (!(item = MENU_FindItem( &hMenu, &wItemID, wFlags ))) return -1;
     debug_print_menuitem ("  item: ", item, "");
     if (item->fType & MF_POPUP)
     {
-        POPUPMENU *menu = MENU_GetMenu( item->hSubMenu );
-        if (!menu) return -1;
-        else return (menu->nItems << 8) | ((item->fState|item->fType) & 0xff);
+    POPUPMENU *menu = MENU_GetMenu( item->hSubMenu );
+    if (!menu) return -1;
+    else return (menu->nItems << 8) | ((item->fState|item->fType) & 0xff);
     }
     else
     {
-        /* We used to (from way back then) mask the result to 0xff.  */
-        /* I don't know why and it seems wrong as the documented */
-        /* return flag MF_SEPARATOR is outside that mask.  */
-        return (item->fType | item->fState);
+    /* We used to (from way back then) mask the result to 0xff.  */
+    /* I don't know why and it seems wrong as the documented */
+    /* return flag MF_SEPARATOR is outside that mask.  */
+    return (item->fType | item->fState);
     }
 }
@@ -3546 +3551 @@
 INT WINAPI GetMenuItemCount( HMENU hMenu )
 {
-    LPPOPUPMENU menu = MENU_GetMenu(hMenu);
+    LPPOPUPMENU menu = MENU_GetMenu(hMenu);
     if (!menu) return -1;
     TRACE("(%04x) returning %d\n",
@@ -3585 +3590 @@
     if (IS_STRING_ITEM(flags) && str)
         TRACE("hMenu %04x, pos %d, flags %08x, "
-                      "id %04x, str %s\n",
+              "id %04x, str %s\n",
                       hMenu, pos, flags, id, debugstr_w(str) );
     else TRACE("hMenu %04x, pos %d, flags %08x, "
-                       "id %04x, str %08lx (not a string)\n",
+               "id %04x, str %08lx (not a string)\n",
                        hMenu, pos, flags, id, (DWORD)str );
 
@@ -3600 +3605 @@
 
     if (flags & MF_POPUP)  /* Set the MF_POPUP flag on the popup-menu */
-        (MENU_GetMenu((HMENU16)id))->wFlags |= MF_POPUP;
+    (MENU_GetMenu((HMENU16)id))->wFlags |= MF_POPUP;
 
     item->hCheckBit = item->hUnCheckBit = 0;
@@ -3656 +3661 @@
 BOOL WINAPI RemoveMenu( HMENU hMenu, UINT nPos, UINT wFlags )
 {
-    LPPOPUPMENU menu;
+    LPPOPUPMENU menu;
     MENUITEM *item;
 
@@ -3674 +3679 @@
     else
     {
-        while(nPos < menu->nItems)
-        {
-            *item = *(item+1);
-            item++;
-            nPos++;
-        }
+    while(nPos < menu->nItems)
+    {
+        *item = *(item+1);
+        item++;
+        nPos++;
+    }
         menu->items = HeapReAlloc( GetProcessHeap(), 0, menu->items,
                                    menu->nItems * sizeof(MENUITEM) );
@@ -3711 +3716 @@
     if (IS_STRING_ITEM(flags))
     {
-        TRACE("%04x %d %04x %04x %s\n",
+    TRACE("%04x %d %04x %04x %s\n",
                       hMenu, pos, flags, id, debugstr_w(str) );
         if (!str) return FALSE;
@@ -3717 +3722 @@
     else
     {
-        TRACE("%04x %d %04x %04x %08lx\n",
+    TRACE("%04x %d %04x %04x %08lx\n",
                       hMenu, pos, flags, id, (DWORD)str );
     }
@@ -3789 +3794 @@
     if (!hNewCheck && !hNewUnCheck)
     {
-        item->fState &= ~MF_USECHECKBITMAPS;
+    item->fState &= ~MF_USECHECKBITMAPS;
     }
     else  /* Install new bitmaps */
     {
-        item->hCheckBit = hNewCheck;
-        item->hUnCheckBit = hNewUnCheck;
-        item->fState |= MF_USECHECKBITMAPS;
+    item->hCheckBit = hNewCheck;
+    item->hUnCheckBit = hNewUnCheck;
+    item->fState |= MF_USECHECKBITMAPS;
     }
     return TRUE;
@@ -3880 +3885 @@
     HMENU retvalue = 0;
 
-    if(IsWindow(hWnd)) 
+    if(IsWindow(hWnd))
     {
         HMENU hSysMenu = getSysMenu(hWnd);
@@ -3934 +3939 @@
     if (wndPtr)
     {
-        if( wndPtr->hSysMenu )
-        {
-            if( bRevert )
-            {
-                DestroyMenu(wndPtr->hSysMenu);
-                wndPtr->hSysMenu = 0;
-            }
-            else
-            {
-                POPUPMENU *menu = MENU_GetMenu( wndPtr->hSysMenu );
-                if( menu )
+    if( wndPtr->hSysMenu )
+    {
+        if( bRevert )
+        {
+        DestroyMenu(wndPtr->hSysMenu);
+        wndPtr->hSysMenu = 0;
+        }
+        else
+        {
+        POPUPMENU *menu = MENU_GetMenu( wndPtr->hSysMenu );
+        if( menu )
                 {
-                   if( menu->nItems > 0 && menu->items[0].hSubMenu == MENU_DefSysPopup )
-                      menu->items[0].hSubMenu = MENU_CopySysPopup();
-                }
-                else
-                {
-                   WARN("Current sys-menu (%04x) of wnd %04x is broken\n",
-                        wndPtr->hSysMenu, hWnd);
-                   wndPtr->hSysMenu = 0;
-                }
-            }
-        }
-
-        if(!wndPtr->hSysMenu && (wndPtr->dwStyle & WS_SYSMENU) )
-            wndPtr->hSysMenu = MENU_GetSysMenu( hWnd, (HMENU)(-1) );
-
-        if( wndPtr->hSysMenu )
-        {
-            POPUPMENU *menu;
-            retvalue = GetSubMenu(wndPtr->hSysMenu, 0);
-
-            /* Store the dummy sysmenu handle to facilitate the refresh */
-            /* of the close button if the SC_CLOSE item change */
-            menu = MENU_GetMenu(retvalue);
-            if ( menu )
-               menu->hSysMenuOwner = wndPtr->hSysMenu;
+           if( menu->nItems > 0 && menu->items[0].hSubMenu == MENU_DefSysPopup )
+              menu->items[0].hSubMenu = MENU_CopySysPopup();
+        }
+        else
+        {
+           WARN("Current sys-menu (%04x) of wnd %04x is broken\n",
+            wndPtr->hSysMenu, hWnd);
+           wndPtr->hSysMenu = 0;
+        }
+        }
+    }
+
+    if(!wndPtr->hSysMenu && (wndPtr->dwStyle & WS_SYSMENU) )
+        wndPtr->hSysMenu = MENU_GetSysMenu( hWnd, (HMENU)(-1) );
+
+    if( wndPtr->hSysMenu )
+        {
+        POPUPMENU *menu;
+        retvalue = GetSubMenu(wndPtr->hSysMenu, 0);
+
+        /* Store the dummy sysmenu handle to facilitate the refresh */
+        /* of the close button if the SC_CLOSE item change */
+        menu = MENU_GetMenu(retvalue);
+        if ( menu )
+           menu->hSysMenuOwner = wndPtr->hSysMenu;
         }
         WIN_ReleaseWndPtr(wndPtr);
@@ -3999 +4004 @@
     if (wndPtr)
     {
-        if (wndPtr->hSysMenu) DestroyMenu( wndPtr->hSysMenu );
-        wndPtr->hSysMenu = MENU_GetSysMenu( hwnd, hMenu );
+    if (wndPtr->hSysMenu) DestroyMenu( wndPtr->hSysMenu );
+    wndPtr->hSysMenu = MENU_GetSysMenu( hwnd, hMenu );
         WIN_ReleaseWndPtr(wndPtr);
-        return TRUE;
+    return TRUE;
     }
     return FALSE;
@@ -4126 +4131 @@
     if (!fEndMenu && top_popup)
     {
-        /* terminate the menu handling code */
+    /* 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 */
+    /* 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 */
         PostMessageA( top_popup, WM_CANCELMODE, 0, 0);
     }
@@ -4152 +4157 @@
 #ifndef __WIN32OS2__
 /**********************************************************************
- *          LoadMenu    (USER.150)
+ *      LoadMenu    (USER.150)
  */
 HMENU16 WINAPI LoadMenu16( HINSTANCE16 instance, LPCSTR name )
@@ -4204 +4209 @@
 
 /**********************************************************************
- *          LoadMenuIndirect    (USER.220)
+ *      LoadMenuIndirect    (USER.220)
  */
 HMENU16 WINAPI LoadMenuIndirect16( LPCVOID template )
@@ -4233 +4238 @@
 
 /**********************************************************************
- *          LoadMenuIndirectA    (USER32.@)
+ *      LoadMenuIndirectA    (USER32.@)
  */
 HMENU WINAPI LoadMenuIndirectA( LPCVOID template )
@@ -4247 +4252 @@
       {
       case 0:
-        offset = GET_WORD(p);
-        p += sizeof(WORD) + offset;
-        if (!(hMenu = CreateMenu())) return 0;
-        if (!MENU_ParseResource( p, hMenu, TRUE ))
-          {
-            DestroyMenu( hMenu );
-            return 0;
-          }
-        return hMenu;
+    offset = GET_WORD(p);
+    p += sizeof(WORD) + offset;
+    if (!(hMenu = CreateMenu())) return 0;
+    if (!MENU_ParseResource( p, hMenu, TRUE ))
+      {
+        DestroyMenu( hMenu );
+        return 0;
+      }
+    return hMenu;
       case 1:
-        offset = GET_WORD(p);
-        p += sizeof(WORD) + offset;
-        if (!(hMenu = CreateMenu())) return 0;
-        if (!MENUEX_ParseResource( p, hMenu))
-          {
-            DestroyMenu( hMenu );
-            return 0;
-          }
-        return hMenu;
+    offset = GET_WORD(p);
+    p += sizeof(WORD) + offset;
+    if (!(hMenu = CreateMenu())) return 0;
+    if (!MENUEX_ParseResource( p, hMenu))
+      {
+        DestroyMenu( hMenu );
+        return 0;
+      }
+    return hMenu;
       default:
         ERR("version %d not supported.\n", version);
@@ -4274 +4279 @@
 
 /**********************************************************************
- *          LoadMenuIndirectW    (USER32.@)
+ *      LoadMenuIndirectW    (USER32.@)
  */
 HMENU WINAPI LoadMenuIndirectW( LPCVOID template )
@@ -4284 +4289 @@
 
 /**********************************************************************
- *              IsMenu    (USER32.@)
+ *      IsMenu    (USER32.@)
  */
 BOOL WINAPI IsMenu(HMENU hmenu)
@@ -4293 +4298 @@
 
 /**********************************************************************
- *              GetMenuItemInfo_common
+ *      GetMenuItemInfo_common
  */
 
 static BOOL GetMenuItemInfo_common ( HMENU hmenu, UINT item, BOOL bypos,
-                                        LPMENUITEMINFOW lpmii, BOOL unicode)
+                    LPMENUITEMINFOW lpmii, BOOL unicode)
 {
     MENUITEM *menu = MENU_FindItem (&hmenu, &item, bypos? MF_BYPOSITION : 0);
@@ -4304 +4309 @@
 
     if (!menu)
-        return FALSE;
+    return FALSE;
 
     if (lpmii->fMask & MIIM_TYPE) {
-        lpmii->fType = menu->fType;
-        switch (MENU_ITEM_TYPE(menu->fType)) {
-        case MF_STRING:
+    lpmii->fType = menu->fType;
+    switch (MENU_ITEM_TYPE(menu->fType)) {
+    case MF_STRING:
             break;  /* will be done below */
-        case MF_OWNERDRAW:
-        case MF_BITMAP:
-            lpmii->dwTypeData = menu->text;
-            /* fall through */
-        default:
-            lpmii->cch = 0;
-        }
+    case MF_OWNERDRAW:
+    case MF_BITMAP:
+        lpmii->dwTypeData = menu->text;
+        /* fall through */
+    default:
+        lpmii->cch = 0;
+    }
     }
 
@@ -4337 +4342 @@
                 if (!WideCharToMultiByte( CP_ACP, 0, menu->text, -1,
                                           (LPSTR)lpmii->dwTypeData, lpmii->cch, NULL, NULL ))
+#ifdef __WIN32OS2__
+                    lstrtrunc((LPSTR)lpmii->dwTypeData, lpmii->cch );
+#else
                     ((LPSTR)lpmii->dwTypeData)[lpmii->cch-1] = 0;
+#endif
         }
         /* if we've copied a substring we return its length */
@@ -4349 +4358 @@
 
     if (lpmii->fMask & MIIM_FTYPE)
-        lpmii->fType = menu->fType;
+    lpmii->fType = menu->fType;
 
     if (lpmii->fMask & MIIM_BITMAP)
-        lpmii->hbmpItem = menu->hbmpItem;
+    lpmii->hbmpItem = menu->hbmpItem;
 
     if (lpmii->fMask & MIIM_STATE)
-        lpmii->fState = menu->fState;
+    lpmii->fState = menu->fState;
 
     if (lpmii->fMask & MIIM_ID)
-        lpmii->wID = menu->wID;
+    lpmii->wID = menu->wID;
 
     if (lpmii->fMask & MIIM_SUBMENU)
-        lpmii->hSubMenu = menu->hSubMenu;
+    lpmii->hSubMenu = menu->hSubMenu;
 
     if (lpmii->fMask & MIIM_CHECKMARKS) {
-        lpmii->hbmpChecked = menu->hCheckBit;
-        lpmii->hbmpUnchecked = menu->hUnCheckBit;
+    lpmii->hbmpChecked = menu->hCheckBit;
+    lpmii->hbmpUnchecked = menu->hUnCheckBit;
     }
     if (lpmii->fMask & MIIM_DATA)
-        lpmii->dwItemData = menu->dwItemData;
+    lpmii->dwItemData = menu->dwItemData;
 
   return TRUE;
@@ -4374 +4383 @@
 
 /**********************************************************************
- *              GetMenuItemInfoA    (USER32.@)
+ *      GetMenuItemInfoA    (USER32.@)
  */
 BOOL WINAPI GetMenuItemInfoA( HMENU hmenu, UINT item, BOOL bypos,
@@ -4384 +4393 @@
 
 /**********************************************************************
- *              GetMenuItemInfoW    (USER32.@)
+ *      GetMenuItemInfoW    (USER32.@)
  */
 BOOL WINAPI GetMenuItemInfoW( HMENU hmenu, UINT item, BOOL bypos,
@@ -4418 +4427 @@
 
 /**********************************************************************
- *              SetMenuItemInfo_common
+ *      SetMenuItemInfo_common
  */
 
 static BOOL SetMenuItemInfo_common(MENUITEM * menu,
-                                       const MENUITEMINFOW *lpmii,
-                                       BOOL unicode)
+                       const MENUITEMINFOW *lpmii,
+                       BOOL unicode)
 {
     if (!menu) return FALSE;
@@ -4430 +4439 @@
 
     if (lpmii->fMask & MIIM_TYPE ) {
-        /* Get rid of old string.  */
-        if ( IS_STRING_ITEM(menu->fType) && menu->text) {
-            HeapFree(GetProcessHeap(), 0, menu->text);
-            menu->text = NULL;
-        }
-
-        /* make only MENU_ITEM_TYPE bits in menu->fType equal lpmii->fType */
-        menu->fType &= ~MENU_ITEM_TYPE(menu->fType);
-        menu->fType |= MENU_ITEM_TYPE(lpmii->fType);
-
-        menu->text = lpmii->dwTypeData;
+    /* Get rid of old string.  */
+    if ( IS_STRING_ITEM(menu->fType) && menu->text) {
+        HeapFree(GetProcessHeap(), 0, menu->text);
+        menu->text = NULL;
+    }
+
+    /* make only MENU_ITEM_TYPE bits in menu->fType equal lpmii->fType */
+    menu->fType &= ~MENU_ITEM_TYPE(menu->fType);
+    menu->fType |= MENU_ITEM_TYPE(lpmii->fType);
+
+    menu->text = lpmii->dwTypeData;
 
        if (IS_STRING_ITEM(menu->fType))
@@ -4447 +4456 @@
 
     if (lpmii->fMask & MIIM_FTYPE ) {
-        /* free the string when the type is changing */
-        if ( (!IS_STRING_ITEM(lpmii->fType)) && IS_STRING_ITEM(menu->fType) && menu->text) {
-            HeapFree(GetProcessHeap(), 0, menu->text);
-            menu->text = NULL;
-        }
-        menu->fType &= ~MENU_ITEM_TYPE(menu->fType);
-        menu->fType |= MENU_ITEM_TYPE(lpmii->fType);
+    /* free the string when the type is changing */
+    if ( (!IS_STRING_ITEM(lpmii->fType)) && IS_STRING_ITEM(menu->fType) && menu->text) {
+        HeapFree(GetProcessHeap(), 0, menu->text);
+        menu->text = NULL;
+    }
+    menu->fType &= ~MENU_ITEM_TYPE(menu->fType);
+    menu->fType |= MENU_ITEM_TYPE(lpmii->fType);
         if ( IS_STRING_ITEM(menu->fType) && !menu->text )
             menu->fType |= MF_SEPARATOR;
@@ -4459 +4468 @@
 
     if (lpmii->fMask & MIIM_STRING ) {
-        /* free the string when used */
-        if ( IS_STRING_ITEM(menu->fType) && menu->text) {
-            HeapFree(GetProcessHeap(), 0, menu->text);
+    /* free the string when used */
+    if ( IS_STRING_ITEM(menu->fType) && menu->text) {
+        HeapFree(GetProcessHeap(), 0, menu->text);
             set_menu_item_text( menu, lpmii->dwTypeData, unicode );
-        }
+    }
     }
 
     if (lpmii->fMask & MIIM_STATE)
     {
-        /* FIXME: MFS_DEFAULT do we have to reset the other menu items? */
-        menu->fState = lpmii->fState;
+    /* FIXME: MFS_DEFAULT do we have to reset the other menu items? */
+    menu->fState = lpmii->fState;
     }
 
     if (lpmii->fMask & MIIM_ID)
-        menu->wID = lpmii->wID;
+    menu->wID = lpmii->wID;
 
     if (lpmii->fMask & MIIM_SUBMENU) {
-        menu->hSubMenu = lpmii->hSubMenu;
-        if (menu->hSubMenu) {
-            POPUPMENU *subMenu = MENU_GetMenu((UINT16)menu->hSubMenu);
-            if (subMenu) {
-                subMenu->wFlags |= MF_POPUP;
-                menu->fType |= MF_POPUP;
-            }
-            else
-                /* FIXME: Return an error ? */
-                menu->fType &= ~MF_POPUP;
-        }
-        else
-            menu->fType &= ~MF_POPUP;
+    menu->hSubMenu = lpmii->hSubMenu;
+    if (menu->hSubMenu) {
+        POPUPMENU *subMenu = MENU_GetMenu((UINT16)menu->hSubMenu);
+        if (subMenu) {
+        subMenu->wFlags |= MF_POPUP;
+        menu->fType |= MF_POPUP;
+        }
+        else
+        /* FIXME: Return an error ? */
+        menu->fType &= ~MF_POPUP;
+    }
+    else
+        menu->fType &= ~MF_POPUP;
     }
 
@@ -4496 +4505 @@
             menu->fType |= MFT_RADIOCHECK;
 
-        menu->hCheckBit = lpmii->hbmpChecked;
-        menu->hUnCheckBit = lpmii->hbmpUnchecked;
+    menu->hCheckBit = lpmii->hbmpChecked;
+    menu->hUnCheckBit = lpmii->hbmpUnchecked;
     }
     if (lpmii->fMask & MIIM_DATA)
-        menu->dwItemData = lpmii->dwItemData;
+    menu->dwItemData = lpmii->dwItemData;
 
     debug_print_menuitem("SetMenuItemInfo_common to : ", menu, "");
@@ -4507 +4516 @@
 
 /**********************************************************************
- *              SetMenuItemInfoA    (USER32.@)
+ *      SetMenuItemInfoA    (USER32.@)
  */
 BOOL WINAPI SetMenuItemInfoA(HMENU hmenu, UINT item, BOOL bypos,
@@ -4513 +4522 @@
 {
     if ((lpmii->fType & (MF_HILITE|MF_POPUP)) || (lpmii->fState)) {
-        /* QuickTime does pass invalid data into SetMenuItemInfo.
-         * do some of the checks Windows does.
-         */
+    /* QuickTime does pass invalid data into SetMenuItemInfo.
+     * do some of the checks Windows does.
+     */
         WARN("Bad masks for type (0x%08x) or state (0x%08x)\n",
              lpmii->fType,lpmii->fState );
-        return FALSE;
+    return FALSE;
     }
 
     return SetMenuItemInfo_common(MENU_FindItem(&hmenu, &item, bypos? MF_BYPOSITION : 0),
-                                    (const MENUITEMINFOW *)lpmii, FALSE);
+                    (const MENUITEMINFOW *)lpmii, FALSE);
 }
 
 /**********************************************************************
- *              SetMenuItemInfoW    (USER32.@)
+ *      SetMenuItemInfoW    (USER32.@)
  */
 BOOL WINAPI SetMenuItemInfoW(HMENU hmenu, UINT item, BOOL bypos,
@@ -4532 +4541 @@
 {
     return SetMenuItemInfo_common(MENU_FindItem(&hmenu, &item, bypos? MF_BYPOSITION : 0),
-                                    lpmii, TRUE);
+                    lpmii, TRUE);
 }
 
 /**********************************************************************
- *              SetMenuDefaultItem    (USER32.@)
+ *      SetMenuDefaultItem    (USER32.@)
  *
  */
 BOOL WINAPI SetMenuDefaultItem(HMENU hmenu, UINT uItem, UINT bypos)
 {
-        UINT i;
-        POPUPMENU *menu;
-        MENUITEM *item;
-
-        TRACE("(0x%x,%d,%d)\n", hmenu, uItem, bypos);
-
-        if (!(menu = MENU_GetMenu(hmenu))) return FALSE;
-
-        /* reset all default-item flags */
-        item = menu->items;
-        for (i = 0; i < menu->nItems; i++, item++)
-        {
-            item->fState &= ~MFS_DEFAULT;
-        }
-
-        /* no default item */
-        if ( -1 == uItem)
-        {
-            return TRUE;
-        }
-
-        item = menu->items;
-        if ( bypos )
-        {
-            if ( uItem >= menu->nItems ) return FALSE;
-            item[uItem].fState |= MFS_DEFAULT;
-            return TRUE;
-        }
-        else
-        {
-            for (i = 0; i < menu->nItems; i++, item++)
-            {
-                if (item->wID == uItem)
-                {
-                     item->fState |= MFS_DEFAULT;
-                     return TRUE;
-                }
-            }
-
-        }
-        return FALSE;
+    UINT i;
+    POPUPMENU *menu;
+    MENUITEM *item;
+
+    TRACE("(0x%x,%d,%d)\n", hmenu, uItem, bypos);
+
+    if (!(menu = MENU_GetMenu(hmenu))) return FALSE;
+
+    /* reset all default-item flags */
+    item = menu->items;
+    for (i = 0; i < menu->nItems; i++, item++)
+    {
+        item->fState &= ~MFS_DEFAULT;
+    }
+
+    /* no default item */
+    if ( -1 == uItem)
+    {
+        return TRUE;
+    }
+
+    item = menu->items;
+    if ( bypos )
+    {
+        if ( uItem >= menu->nItems ) return FALSE;
+        item[uItem].fState |= MFS_DEFAULT;
+        return TRUE;
+    }
+    else
+    {
+        for (i = 0; i < menu->nItems; i++, item++)
+        {
+        if (item->wID == uItem)
+        {
+             item->fState |= MFS_DEFAULT;
+             return TRUE;
+        }
+        }
+
+    }
+    return FALSE;
 }
 
 /**********************************************************************
- *              GetMenuDefaultItem    (USER32.@)
+ *      GetMenuDefaultItem    (USER32.@)
  */
 UINT WINAPI GetMenuDefaultItem(HMENU hmenu, UINT bypos, UINT flags)
 {
-        POPUPMENU *menu;
-        MENUITEM * item;
-        UINT i = 0;
-
-        TRACE("(0x%x,%d,%d)\n", hmenu, bypos, flags);
-
-        if (!(menu = MENU_GetMenu(hmenu))) return -1;
-
-        /* find default item */
-        item = menu->items;
-
-        /* empty menu */
-        if (! item) return -1;
-
-        while ( !( item->fState & MFS_DEFAULT ) )
-        {
-            i++; item++;
-            if  (i >= menu->nItems ) return -1;
-        }
-
-        /* default: don't return disabled items */
-        if ( (!(GMDI_USEDISABLED & flags)) && (item->fState & MFS_DISABLED )) return -1;
-
-        /* search rekursiv when needed */
-        if ( (item->fType & MF_POPUP) &&  (flags & GMDI_GOINTOPOPUPS) )
-        {
-            UINT ret;
-            ret = GetMenuDefaultItem( item->hSubMenu, bypos, flags );
-            if ( -1 != ret ) return ret;
-
-            /* when item not found in submenu, return the popup item */
-        }
-        return ( bypos ) ? i : item->wID;
+    POPUPMENU *menu;
+    MENUITEM * item;
+    UINT i = 0;
+
+    TRACE("(0x%x,%d,%d)\n", hmenu, bypos, flags);
+
+    if (!(menu = MENU_GetMenu(hmenu))) return -1;
+
+    /* find default item */
+    item = menu->items;
+
+    /* empty menu */
+    if (! item) return -1;
+
+    while ( !( item->fState & MFS_DEFAULT ) )
+    {
+        i++; item++;
+        if  (i >= menu->nItems ) return -1;
+    }
+
+    /* default: don't return disabled items */
+    if ( (!(GMDI_USEDISABLED & flags)) && (item->fState & MFS_DISABLED )) return -1;
+
+    /* search rekursiv when needed */
+    if ( (item->fType & MF_POPUP) &&  (flags & GMDI_GOINTOPOPUPS) )
+    {
+        UINT ret;
+        ret = GetMenuDefaultItem( item->hSubMenu, bypos, flags );
+        if ( -1 != ret ) return ret;
+
+        /* when item not found in submenu, return the popup item */
+    }
+    return ( bypos ) ? i : item->wID;
 
 }
@@ -4627 +4636 @@
 
 /**********************************************************************
- *              InsertMenuItemA    (USER32.@)
+ *      InsertMenuItemA    (USER32.@)
  */
 BOOL WINAPI InsertMenuItemA(HMENU hMenu, UINT uItem, BOOL bypos,
@@ -4638 +4647 @@
 
 /**********************************************************************
- *              InsertMenuItemW    (USER32.@)
+ *      InsertMenuItemW    (USER32.@)
  */
 BOOL WINAPI InsertMenuItemW(HMENU hMenu, UINT uItem, BOOL bypos,
@@ -4648 +4657 @@
 
 /**********************************************************************
- *              CheckMenuRadioItem    (USER32.@)
+ *      CheckMenuRadioItem    (USER32.@)
  */
 
 BOOL WINAPI CheckMenuRadioItem(HMENU hMenu,
-                                   UINT first, UINT last, UINT check,
-                                   UINT bypos)
+                   UINT first, UINT last, UINT check,
+                   UINT bypos)
 {
      MENUITEM *mifirst, *milast, *micheck;
@@ -4659 +4668 @@
 
      TRACE("ox%x: %d-%d, check %d, bypos=%d\n",
-                  hMenu, first, last, check, bypos);
+          hMenu, first, last, check, bypos);
 
      mifirst = MENU_FindItem (&mfirst, &first, bypos);
@@ -4666 +4675 @@
 
      if (mifirst == NULL || milast == NULL || micheck == NULL ||
-         mifirst > milast || mfirst != mlast || mfirst != mcheck ||
-         micheck > milast || micheck < mifirst)
-          return FALSE;
+     mifirst > milast || mfirst != mlast || mfirst != mcheck ||
+     micheck > milast || micheck < mifirst)
+      return FALSE;
 
      while (mifirst <= milast)
      {
-          if (mifirst == micheck)
-          {
-               mifirst->fType |= MFT_RADIOCHECK;
-               mifirst->fState |= MFS_CHECKED;
-          } else {
-               mifirst->fType &= ~MFT_RADIOCHECK;
-               mifirst->fState &= ~MFS_CHECKED;
-          }
-          mifirst++;
+      if (mifirst == micheck)
+      {
+           mifirst->fType |= MFT_RADIOCHECK;
+           mifirst->fState |= MFS_CHECKED;
+      } else {
+           mifirst->fType &= ~MFT_RADIOCHECK;
+           mifirst->fState &= ~MFS_CHECKED;
+      }
+      mifirst++;
      }
 
@@ -4687 +4696 @@
 
 /**********************************************************************
- *              GetMenuItemRect    (USER32.@)
+ *      GetMenuItemRect    (USER32.@)
  *
  *      ATTENTION: Here, the returned values in rect are the screen
@@ -4695 +4704 @@
  */
 BOOL WINAPI GetMenuItemRect (HWND hwnd, HMENU hMenu, UINT uItem,
-                                 LPRECT rect)
+                 LPRECT rect)
 {
      POPUPMENU *itemMenu;
@@ -4708 +4717 @@
      if(!hwnd)
      {
-         itemMenu = MENU_GetMenu(hMenu);
-         if (itemMenu == NULL)
-             return FALSE;
-
-         if(itemMenu->hWnd == 0)
-             return FALSE;
-         referenceHwnd = itemMenu->hWnd;
+     itemMenu = MENU_GetMenu(hMenu);
+     if (itemMenu == NULL)
+         return FALSE;
+
+     if(itemMenu->hWnd == 0)
+         return FALSE;
+     referenceHwnd = itemMenu->hWnd;
      }
 
      if ((rect == NULL) || (item == NULL))
-         return FALSE;
+     return FALSE;
 
      *rect = item->rect;
@@ -4729 +4738 @@
 
 /**********************************************************************
- *              SetMenuInfo    (USER32.@)
+ *      SetMenuInfo    (USER32.@)
  *
  * FIXME
- *      MIM_APPLYTOSUBMENUS
- *      actually use the items to draw the menu
+ *  MIM_APPLYTOSUBMENUS
+ *  actually use the items to draw the menu
  */
 BOOL WINAPI SetMenuInfo (HMENU hMenu, LPCMENUINFO lpmi)
@@ -4744 +4753 @@
     {
 
-        if (lpmi->fMask & MIM_BACKGROUND)
-            menu->hbrBack = lpmi->hbrBack;
-
-        if (lpmi->fMask & MIM_HELPID)
-            menu->dwContextHelpID = lpmi->dwContextHelpID;
-
-        if (lpmi->fMask & MIM_MAXHEIGHT)
-            menu->cyMax = lpmi->cyMax;
-
-        if (lpmi->fMask & MIM_MENUDATA)
-            menu->dwMenuData = lpmi->dwMenuData;
-
-        if (lpmi->fMask & MIM_STYLE)
-            menu->dwStyle = lpmi->dwStyle;
-
-        return TRUE;
+    if (lpmi->fMask & MIM_BACKGROUND)
+        menu->hbrBack = lpmi->hbrBack;
+
+    if (lpmi->fMask & MIM_HELPID)
+        menu->dwContextHelpID = lpmi->dwContextHelpID;
+
+    if (lpmi->fMask & MIM_MAXHEIGHT)
+        menu->cyMax = lpmi->cyMax;
+
+    if (lpmi->fMask & MIM_MENUDATA)
+        menu->dwMenuData = lpmi->dwMenuData;
+
+    if (lpmi->fMask & MIM_STYLE)
+        menu->dwStyle = lpmi->dwStyle;
+
+    return TRUE;
     }
     return FALSE;
@@ -4765 +4774 @@
 
 /**********************************************************************
- *              GetMenuInfo    (USER32.@)
+ *      GetMenuInfo    (USER32.@)
  *
  *  NOTES
- *      win98/NT5.0
+ *  win98/NT5.0
  *
  */
@@ -4779 +4788 @@
     {
 
-        if (lpmi->fMask & MIM_BACKGROUND)
-            lpmi->hbrBack = menu->hbrBack;
-
-        if (lpmi->fMask & MIM_HELPID)
-            lpmi->dwContextHelpID = menu->dwContextHelpID;
-
-        if (lpmi->fMask & MIM_MAXHEIGHT)
-            lpmi->cyMax = menu->cyMax;
-
-        if (lpmi->fMask & MIM_MENUDATA)
-            lpmi->dwMenuData = menu->dwMenuData;
-
-        if (lpmi->fMask & MIM_STYLE)
-            lpmi->dwStyle = menu->dwStyle;
-
-        return TRUE;
+    if (lpmi->fMask & MIM_BACKGROUND)
+        lpmi->hbrBack = menu->hbrBack;
+
+    if (lpmi->fMask & MIM_HELPID)
+        lpmi->dwContextHelpID = menu->dwContextHelpID;
+
+    if (lpmi->fMask & MIM_MAXHEIGHT)
+        lpmi->cyMax = menu->cyMax;
+
+    if (lpmi->fMask & MIM_MENUDATA)
+        lpmi->dwMenuData = menu->dwMenuData;
+
+    if (lpmi->fMask & MIM_STYLE)
+        lpmi->dwStyle = menu->dwStyle;
+
+    return TRUE;
     }
     return FALSE;
@@ -4810 +4819 @@
     if ((menu = MENU_GetMenu(hMenu)))
     {
-        menu->dwContextHelpID = dwContextHelpID;
-        return TRUE;
+    menu->dwContextHelpID = dwContextHelpID;
+    return TRUE;
     }
     return FALSE;
@@ -4827 +4836 @@
     if ((menu = MENU_GetMenu(hMenu)))
     {
-        return menu->dwContextHelpID;
+    return menu->dwContextHelpID;
     }
     return 0;
@@ -4847 +4856 @@
 }
 
-#ifdef __WIN32OS2__ 
+#ifdef __WIN32OS2__
 //******************************************************************************
 //******************************************************************************
@@ -4853 +4862 @@
 {
     fDisableOdinSysMenuItems = TRUE;
+}
+/**********************************************************************
+ *           KBD_translate_accelerator
+ *
+ * FIXME: should send some WM_INITMENU or/and WM_INITMENUPOPUP  -messages
+ */
+static BOOL KBD_translate_accelerator(HWND hWnd,LPMSG msg,
+                                      BYTE fVirt,WORD key,WORD cmd)
+{
+    BOOL sendmsg = FALSE;
+
+    if(msg->wParam == key)
+    {
+        if (msg->message == WM_CHAR) {
+            if ( !(fVirt & FALT) && !(fVirt & FVIRTKEY) )
+            {
+                dprintf(("TranslateAccelerator: found accel for WM_CHAR: ('%c')\n", msg->wParam&0xff));
+                sendmsg=TRUE;
+            }
+        }
+        else
+        {
+            if(fVirt & FVIRTKEY) {
+                INT mask = 0;
+                if(GetKeyState(VK_SHIFT) & 0x8000) mask |= FSHIFT;
+                if(GetKeyState(VK_CONTROL) & 0x8000) mask |= FCONTROL;
+                if(GetKeyState(VK_MENU) & 0x8000) mask |= FALT;
+
+                if(mask == (fVirt & (FSHIFT | FCONTROL | FALT)))
+                        sendmsg=TRUE;
+                else    dprintf(("TranslateAccelerator: but incorrect SHIFT/CTRL/ALT-state %x != %x", mask, fVirt));
+            }
+            else
+            {
+                if (!(msg->lParam & 0x01000000))  /* no special_key */
+                {
+                    if ((fVirt & FALT) && (msg->lParam & 0x20000000))
+                    {                                                   /* ^^ ALT pressed */
+                        dprintf(("TranslateAccelerator: found accel for Alt-%c\n", msg->wParam&0xff));
+                        sendmsg=TRUE;
+                    }
+                }
+            }
+        }
+
+            dprintf(("TranslateAccelerator: not match for %x %x %x", fVirt, key, cmd));
+    }
+
+    else if( key >= 1 && key <= 26 && !( fVirt & FVIRTKEY )) // Ctrl-A to Ctrl-Z
+    {
+        int ctrlCh;
+
+        if( msg->message == WM_CHAR )
+            ctrlCh = toupper( msg->wParam ) - 'A' + 1;
+        else
+            ctrlCh = msg->wParam - VK_A + 1;
+
+        if( ctrlCh == key )
+        {
+            INT mask = 0;
+            if(GetKeyState(VK_SHIFT) & 0x8000) mask |= FSHIFT;
+            if(GetKeyState(VK_CONTROL) & 0x8000) mask |= FCONTROL;
+            if(GetKeyState(VK_MENU) & 0x8000) mask |= FALT;
+
+            if(mask == FCONTROL)
+            {
+                sendmsg=TRUE;
+            }
+        }
+    }
+
+        if (sendmsg)      /* found an accelerator, but send a message... ? */
+        {
+            INT     iSysStat = -1,iStat = -1,mesg=0;
+            HMENU   hSysMenu, hMenu, hSubMenu;
+            UINT    nPos;
+
+            if (msg->message == WM_KEYUP || msg->message == WM_SYSKEYUP) {
+                mesg=1;
+            }
+            else
+                if (GetCapture())
+                    mesg=2;
+                else
+                if (!IsWindowEnabled(hWnd))
+                    mesg=3;
+                else
+                {
+                    hMenu = (GetWindowLongA(hWnd, GWL_STYLE) & WS_CHILD) ? NULL : GetMenu(hWnd);
+                    hSysMenu = getSysMenu(hWnd);
+
+                    hSubMenu = hSysMenu;
+                    nPos = cmd;
+                    if(MENU_FindItem(&hSubMenu, &nPos, MF_BYCOMMAND))
+                    {
+                        SendMessageA(hWnd, WM_INITMENU, (WPARAM)hSysMenu, 0L);
+                        if(hSubMenu != hSysMenu)
+                        {
+                            nPos = MENU_FindSubMenu(&hSysMenu, hSubMenu);
+                            dprintf(("hSysMenu = %04x, hSubMenu = %04x, nPos = %d\n", hSysMenu, hSubMenu, nPos));
+                            SendMessageA(hWnd, WM_INITMENUPOPUP, (WPARAM)hSubMenu, MAKELPARAM(nPos, TRUE));
+                        }
+                        iSysStat = GetMenuState(GetSubMenu(hSysMenu, 0), cmd, MF_BYCOMMAND);
+                    }
+                    else
+                    {
+                        hSubMenu = hMenu;
+                        nPos = cmd;
+                        if(MENU_FindItem(&hSubMenu, &nPos, MF_BYCOMMAND))
+                        {
+                            SendMessageA(hWnd, WM_INITMENU, (WPARAM)hMenu, 0L);
+                            if(hSubMenu != hMenu)
+                            {
+                                nPos = MENU_FindSubMenu(&hMenu, hSubMenu);
+                                dprintf(("hMenu = %04x, hSubMenu = %04x, nPos = %d\n", hMenu, hSubMenu, nPos));
+                                SendMessageA(hWnd, WM_INITMENUPOPUP, (WPARAM)hSubMenu, MAKELPARAM(nPos, FALSE));
+                            }
+                            iStat = GetMenuState(hMenu, cmd, MF_BYCOMMAND);
+                        }
+                    }
+
+                    if (iSysStat!=-1)
+                    {
+                        if (iSysStat & (MF_DISABLED|MF_GRAYED))
+                            mesg=4;
+                        else
+                            mesg=WM_SYSCOMMAND;
+                    }
+                    else
+                    {
+                        if (iStat!=-1)
+                        {
+                            if (IsIconic(hWnd)) {
+                                mesg=5;
+                            }
+                            else
+                            {
+                                if (iStat & (MF_DISABLED|MF_GRAYED))
+                                    mesg=6;
+                                else
+                                    mesg=WM_COMMAND;
+                            }
+                        }
+                        else
+                            mesg=WM_COMMAND;
+                    }
+                }
+                if ( mesg==WM_COMMAND )
+                    SendMessageA(hWnd, mesg, MAKEWPARAM( cmd, 1 ), NULL);
+                else if( mesg==WM_SYSCOMMAND )
+                    SendMessageA( hWnd, mesg, cmd, MAKELPARAM( 0, 1 ));
+                else
+                {
+                    /*  some reasons for NOT sending the WM_{SYS}COMMAND message:
+                     *   #0: unknown (please report!)
+                     *   #1: for WM_KEYUP,WM_SYSKEYUP
+                     *   #2: mouse is captured
+                     *   #3: window is disabled
+                     *   #4: it's a disabled system menu option
+                     *   #5: it's a menu option, but window is iconic
+                     *   #6: it's a menu option, but disabled
+                     */
+                    if(mesg==0)
+                            dprintf(("ERROR: unknown reason - please report!"));
+                    else    dprintf(("but won't send WM_{SYS}COMMAND, reason is #%d\n",mesg));
+
+                }
+                return TRUE;
+            }
+
+    return FALSE;
+}
+/*****************************************************************************
+ * Name      : int WIN32API TranslateAcceleratorA
+ * Purpose   : Translate WM_*KEYDOWN messages to WM_COMMAND messages
+ *             according to Accelerator table
+ * Parameters: HWND hwnd, HACCEL haccel, LPMSG lpmsg
+ * Variables :
+ * Result    : int FALSE (no accelerator found) TRUE (accelerator found)
+ * Remark    : if a accelerator is found it is not neccesarely executed
+ *             depends on window stat
+ *
+ *****************************************************************************/
+INT WINAPI TranslateAcceleratorA(HWND hWnd, HACCEL hAccel, LPMSG msg)
+{
+    /* YES, Accel16! */
+    LPACCEL lpAccelTbl;
+    int i;
+
+    SetLastError(ERROR_SUCCESS);
+    if (msg == NULL)
+    {
+          dprintf(("TranslateAcceleratorAmsg null; should hang here to be win compatible"));
+          SetLastError(ERROR_INVALID_PARAMETER);
+          return 0;
+    }
+    if (!hAccel || !(lpAccelTbl = (LPACCEL)GlobalLock(hAccel)))
+    {
+          dprintf(("TranslateAcceleratorA: invalid accel handle=%x", hAccel));
+          SetLastError(ERROR_INVALID_PARAMETER);
+          return 0;
+    }
+    if(!IsWindow(hWnd)) {
+          dprintf(("TranslateAccelerator, window %x not found", hWnd));
+          SetLastError(ERROR_INVALID_WINDOW_HANDLE);
+          return 0;
+    }
+    if ((msg->message != WM_KEYDOWN &&
+         msg->message != WM_KEYUP &&
+         msg->message != WM_SYSKEYDOWN &&
+         msg->message != WM_SYSKEYUP &&
+         msg->message != WM_CHAR))
+    {
+          return 0;
+    }
+
+/*    TRACE_(accel)("TranslateAccelerators hAccel=%04x, hWnd=%04x,"
+      "msg->hwnd=%04x, msg->message=%04x, wParam=%08x, lParam=%lx\n",
+      hAccel,hWnd,msg->hwnd,msg->message,msg->wParam,msg->lParam); */
+
+    i = 0;
+    do
+    {
+        if (KBD_translate_accelerator(hWnd,msg,lpAccelTbl[i].fVirt,
+                                      lpAccelTbl[i].key,lpAccelTbl[i].cmd))
+        {
+            return 1;
+        }
+    }
+    while ((lpAccelTbl[i++].fVirt & 0x80) == 0);
+
+//    WARN_(accel)("couldn't translate accelerator key\n");
+    return 0;
 }
 //******************************************************************************
@@ -5046 +5288 @@
 }
 #endif
+
+