diff options
| author | Karl Heuer <kwzh@gnu.org> | 1995-06-09 01:11:36 +0000 |
|---|---|---|
| committer | Karl Heuer <kwzh@gnu.org> | 1995-06-09 01:11:36 +0000 |
| commit | d8fa949e56a9d1674b7f6563a7883dd7c32de959 (patch) | |
| tree | 821684b1861055e1886e53516780f6c34be4bd06 /src | |
| parent | 4387a005c0057e71fbeed60b312b44c0f5bc2714 (diff) | |
| download | emacs-d8fa949e56a9d1674b7f6563a7883dd7c32de959.tar.gz | |
(xmenu_show): Do call lw_destroy_all_widgets.
(xdialog_show): Likewise.
(popup_get_selection): Don't call it here.
Diffstat (limited to 'src')
| -rw-r--r-- | src/xmenu.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/xmenu.c b/src/xmenu.c index d6e42039fbe..7e4db362faa 100644 --- a/src/xmenu.c +++ b/src/xmenu.c @@ -1083,7 +1083,6 @@ popup_get_selection (initial_event, dpyinfo, id) else if (event.type == KeyPress && dpyinfo->display == event.xbutton.display) { - lw_destroy_all_widgets (id); popup_activated_flag = 0; break; } @@ -1862,13 +1861,11 @@ xmenu_show (f, x, y, for_click, keymaps, title, error) /* Process events that apply to the menu. */ popup_get_selection ((XEvent *) 0, FRAME_X_DISPLAY_INFO (f), menu_id); -#if 0 /* fp turned off the following statement and wrote a comment that it is unnecessary--that the menu has already disappeared. - I observer that is not so. -- rms. */ - /* Make sure the menu disappears. */ + Nowadays the menu disappears ok, all right, but + we need to delete the widgets or multiple ones will pile up. */ lw_destroy_all_widgets (menu_id); -#endif /* Find the selected item, and its pane, to return the proper value. */ @@ -2080,6 +2077,8 @@ xdialog_show (f, keymaps, title, error) /* Process events that apply to the menu. */ popup_get_selection ((XEvent *) 0, FRAME_X_DISPLAY_INFO (f), dialog_id); + lw_destroy_all_widgets (dialog_id); + /* Find the selected item, and its pane, to return the proper value. */ if (menu_item_selection != 0) |