commit 2d6eabacc32ce25e6bf7c5f032a6e638291d3b88
author Huw D M Davies <h.davies1@physics.ox.ac.uk> Fri Mar 24 19:48:53 2000 +0000
committer Alexandre Julliard <julliard@winehq.org> Fri Mar 24 19:48:53 2000 +0000
tree 11a87d28ee24f01dc411d54623efacaca178dee5
parent 70e16ad0e16da12876a4d1b71664ddc83d98721c

Set the correct value for the cch element of MENUITEMINFO in
GetMenuItemInfo.

controls/menu.c

diff --git a/controls/menu.c b/controls/menu.c
index fa915d8..6eeb608 100644
--- a/controls/menu.c
+++ b/controls/menu.c
@@ -4293,34 +4293,40 @@
     if (lpmii->fMask & MIIM_TYPE) {
 	lpmii->fType = menu->fType;
 	switch (MENU_ITEM_TYPE(menu->fType)) {
-		case MF_STRING:
-		    if (menu->text && lpmii->dwTypeData && lpmii->cch) {
-			if (unicode) {
-			    lstrcpynAtoW((LPWSTR) lpmii->dwTypeData, menu->text, lpmii->cch);
-			    lpmii->cch = lstrlenW((LPWSTR)menu->text);
-			} else {
-			    lstrcpynA(lpmii->dwTypeData, menu->text, lpmii->cch);
-			    lpmii->cch = lstrlenA(menu->text);
-			}
-		    }
-		    break;
-		case MF_OWNERDRAW:
-		case MF_BITMAP:
-		    lpmii->dwTypeData = menu->text;
-		    /* fall through */
-		default:
-		    lpmii->cch = 0;
+	case MF_STRING:
+ 	    if (menu->text) {
+	        int len = lstrlenA(menu->text);
+	        if(lpmii->dwTypeData && lpmii->cch) {
+		    if (unicode)
+		        lstrcpynAtoW((LPWSTR) lpmii->dwTypeData, menu->text,
+				     lpmii->cch);
+		    else
+		        lstrcpynA(lpmii->dwTypeData, menu->text, lpmii->cch);
+		    /* if we've copied a substring we return its length */
+		    if(lpmii->cch <= len)
+		        lpmii->cch--;
+		} else /* return length of string */
+		    lpmii->cch = len;
+	    }
+	    break;
+	case MF_OWNERDRAW:
+	case MF_BITMAP:
+	    lpmii->dwTypeData = menu->text;
+	    /* fall through */
+	default:
+	    lpmii->cch = 0;
 	}
     }
 
     if (lpmii->fMask & MIIM_STRING) {
-	if (unicode) {
-	    lstrcpynAtoW((LPWSTR) lpmii->dwTypeData, menu->text, lpmii->cch);
-	    lpmii->cch = lstrlenW((LPWSTR)menu->text);
-	} else {
-	    lstrcpynA(lpmii->dwTypeData, menu->text, lpmii->cch);
-	    lpmii->cch = lstrlenA(menu->text);
+        if(lpmii->dwTypeData && lpmii->cch) {
+	    if (unicode)
+	        lstrcpynAtoW((LPWSTR) lpmii->dwTypeData, menu->text,
+			     lpmii->cch);
+	    else
+	        lstrcpynA(lpmii->dwTypeData, menu->text, lpmii->cch);
 	}
+	lpmii->cch = lstrlenA(menu->text);
     }
 
     if (lpmii->fMask & MIIM_FTYPE)