diff options
author | Paul Reilly <pmr@pajato.com> | 1994-10-02 21:23:53 +0000 |
---|---|---|
committer | Paul Reilly <pmr@pajato.com> | 1994-10-02 21:23:53 +0000 |
commit | 4e2ae018235f60dfc3c902d959cecb903dedf34d (patch) | |
tree | b124817af4f4f7ba2b39477d0fb8bc361ca649c4 /src/xmenu.c | |
parent | 9ee55f41fd320549bb5635d1f13037f3406a7664 (diff) | |
download | emacs-4e2ae018235f60dfc3c902d959cecb903dedf34d.tar.gz |
(popup_get_selection): Remove BLOCK_INPUT, UNBLOCK_INPUT since the
calls to popup_get_selection() provide the protection.
Diffstat (limited to 'src/xmenu.c')
-rw-r--r-- | src/xmenu.c | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/src/xmenu.c b/src/xmenu.c index 096ff34559c..516dee694f4 100644 --- a/src/xmenu.c +++ b/src/xmenu.c @@ -153,7 +153,7 @@ static int menu_items_n_panes; static int menu_items_submenu_depth; /* Flag which when set indicates a dialog or menu has been posted by - Xt on behalf of one of the widget sets. */ + Xt on behalf of one of the widget sets. */ static int popup_activated_flag; @@ -337,7 +337,7 @@ menu_item_equiv_key (item_string, item1, descrip_ptr) if (! EQ (def1, def) /* If the command is an alias for another (such as easymenu.el and lmenu.el set it up), - check if the original command matches the cached command. */ + check if the original command matches the cached command. */ && !(SYMBOLP (def) && SYMBOLP (XSYMBOL (def)->function) && EQ (def1, XSYMBOL (def)->function)) /* If something had no key binding before, don't recheck it-- @@ -1001,7 +1001,10 @@ on the left of the dialog box and all following items on the right.\n\ #ifdef USE_X_TOOLKIT /* Loop in Xt until the menu pulldown or dialog popup has been - popped down (deactivated). */ + popped down (deactivated). + + NOTE: All calls to popup_get_selection() should be protected + with BLOCK_INPUT, UNBLOCK_INPUT wrappers. */ void popup_get_selection (initial_event) XEvent *initial_event; @@ -1015,16 +1018,14 @@ popup_get_selection (initial_event) while (1) { - BLOCK_INPUT; XtDispatchEvent (&event); - UNBLOCK_INPUT; if (!popup_activated()) break; XtAppNextEvent (Xt_app_con, &event); } } -/* Detect if a dialog or menu has been posted. */ +/* Detect if a dialog or menu has been posted. */ int popup_activated () { @@ -1129,7 +1130,7 @@ menubar_selection_callback (widget, id, client_data) } /* This callback is invoked when a dialog or menu is finished being - used and has been unposted. */ + used and has been unposted. */ static void popup_deactivate_callback (widget, id, client_data) @@ -1357,7 +1358,7 @@ update_frame_menubar (f) XtVaSetValues (x->menubar_widget, XtNmappedWhenManaged, 1, 0); } - /* Re-manage the text-area widget, and then thrash the sizes. */ + /* Re-manage the text-area widget, and then thrash the sizes. */ XtManageChild (x->edit_widget); lw_refigure_widget (x->column_widget, True); @@ -1425,7 +1426,7 @@ set_frame_menubar (f, first_time) menu trees we supply, rather than just the menu bar item names. */ lw_modify_all_widgets (id, first_wv, 1); - /* Re-enable the edit widget to resize. */ + /* Re-enable the edit widget to resize. */ lw_allow_resizing (f->display.x->widget, True); } else @@ -1745,7 +1746,7 @@ xmenu_show (f, x, y, menubarp, keymaps, title, error) } } - /* Deal with the title, if it is non-nil. */ + /* Deal with the title, if it is non-nil. */ if (!NILP (title)) { widget_value *wv_title = malloc_widget_value (); |