Changeset 2577 for trunk/src/user32/menu.cpp

Timestamp:

Jan 30, 2000, 7:48:29 PM (26 years ago)

Author:

sandervl

Message:

Menu bugfixes

File:

• trunk/src/user32/menu.cpp (modified) (7 diffs)

trunk/src/user32/menu.cpp

@@ -1 @@
-/* $Id: menu.cpp,v 1.12 2000-01-20 16:48:54 cbratschi Exp $*/
+/* $Id: menu.cpp,v 1.13 2000-01-30 18:48:27 sandervl Exp $*/
 /*
  * Menu functions
@@ -3216 +3216 @@
  *         GetMenuString32A    (USER32.268)
  */
-INT WINAPI GetMenuStringA( HMENU hMenu, UINT wItemID,
-                               LPSTR str, INT nMaxSiz, UINT wFlags )
+INT WINAPI GetMenuStringA(HMENU hMenu, UINT wItemID,
+                          LPSTR str, INT nMaxSiz, UINT wFlags )
 {
     MENUITEM *item;
 
-    dprintf(("USER32: GetMenuStringA"));
+    dprintf(("USER32: GetMenuStringA %x %d %d %x", hMenu, wItemID, nMaxSiz, wFlags));
 
     //TRACE("menu=%04x item=%04x ptr=%p len=%d flags=%04x\n",
     //             hMenu, wItemID, str, nMaxSiz, wFlags );
-    if (!str || !nMaxSiz) return 0;
-    str[0] = '\0';
     if (!(item = MENU_FindItem( &hMenu, &wItemID, wFlags ))) return 0;
     if (!IS_STRING_ITEM(item->fType)) return 0;
+    if (!str || !nMaxSiz) return strlen(item->text);
+    str[0] = '\0';
     lstrcpynA( str, item->text, nMaxSiz );
     //TRACE("returning '%s'\n", str );
@@ -3247 +3247 @@
     //TRACE("menu=%04x item=%04x ptr=%p len=%d flags=%04x\n",
     //             hMenu, wItemID, str, nMaxSiz, wFlags );
-    if (!str || !nMaxSiz) return 0;
-    str[0] = '\0';
     if (!(item = MENU_FindItem( &hMenu, &wItemID, wFlags ))) return 0;
     if (!IS_STRING_ITEM(item->fType)) return 0;
+    if (!str || !nMaxSiz) return strlen(item->text);
+    str[0] = '\0';
     lstrcpynAtoW( str, item->text, nMaxSiz );
     return lstrlenW(str);
@@ -3773 +3773 @@
             lpmenu->Height = 0;  /* Make sure we recalculate the size */
         }
-        if (IsWindowVisible(hWnd))
+//SvL: This fixes the menu in standard mine sweeper (window isn't visible
+//     when SetMenu is called)
+//        if (IsWindowVisible(hWnd))
             SetWindowPos( hWnd, 0, 0, 0, 0, 0, SWP_NOSIZE | SWP_NOMOVE |
                         SWP_NOACTIVATE | SWP_NOZORDER | SWP_FRAMECHANGED );
@@ -4316 +4318 @@
      HWND referenceHwnd;
 
-     dprintf(("USER32: GetMenuItemRect"));
      //TRACE("(0x%x,0x%x,%d,%p)\n", hwnd, hMenu, uItem, rect);
 
@@ -4325 +4326 @@
      {
          itemMenu = (POPUPMENU*)hMenu;
-         if (itemMenu == NULL)
-             return FALSE;
+         if (itemMenu == NULL) {
+                SetLastError(ERROR_INVALID_PARAMETER);
+                return FALSE;
+         }
 
          if(itemMenu->hWnd == 0)
@@ -4333 +4336 @@
      }
 
-     if ((rect == NULL) || (item == NULL))
-         return FALSE;
-
+     if ((rect == NULL) || (item == NULL)) {
+        SetLastError(ERROR_INVALID_PARAMETER);
+        return FALSE;
+     }
      *rect = item->rect;
 
      MapWindowPoints(referenceHwnd, 0, (LPPOINT)rect, 2);
+     dprintf(("USER32: GetMenuItemRect %x %x %d (%d,%d)(%d,%d)", hwnd, hMenu, uItem, rect->left, rect->top, rect->right, rect->bottom));
 
      return TRUE;