changeset 30238:4a3b87cc6f04

(w32_menu_show): Call free_menubar_widget_value_tree after menu is finished with. (add_menu_item): Only consider wv->title as a menu title. (w32_menu_display_help): Add OBJECT and POS to show_help_echo.

--- a/src/w32menu.c	Sat Jul 15 11:50:42 2000 +0000
+++ b/src/w32menu.c	Sat Jul 15 11:52:21 2000 +0000
@@ -1747,9 +1747,6 @@
   pos.y = y;
   ClientToScreen (FRAME_W32_WINDOW (f), &pos);
 
-  /* Free the widget_value objects we used to specify the contents.  */
-  free_menubar_widget_value_tree (first_wv);
-
   /* No selection has been chosen yet.  */
   menu_item_selection = 0;
 
@@ -1762,6 +1759,9 @@
      during the call. */
   discard_mouse_events ();
 
+  /* Free the widget_value objects we used to specify the contents.  */
+  free_menubar_widget_value_tree (first_wv);
+
   DestroyMenu (menu);
 
   /* Find the selected item, and its pane, to return
@@ -2056,7 +2056,7 @@
       else
 	out_string = wv->name;
 
-      if (wv->title || wv->call_data == 0)
+      if (wv->title)
 	{
 #if 0  /* no GC while popup menu is active */
 	  out_string = LocalAlloc (0, strlen (wv->name) + 1);
@@ -2170,7 +2170,8 @@
       get_menu_item_info (menu, item, FALSE, &info);
 
       show_help_echo (info.dwItemData ?
-		      build_string ((char *) info.dwItemData) : Qnil, 1);
+		      build_string ((char *) info.dwItemData) : Qnil,
+                      Qnil, -1, 1);
     }
 }