Mercurial > vim
diff src/gui_athena.c @ 19934:3ff714d765ba v8.2.0523
patch 8.2.0523: loops are repeated
Commit: https://github.com/vim/vim/commit/00d253e2b2f435a5386582c3f857008e7ac355c2
Author: Bram Moolenaar <Bram@vim.org>
Date: Mon Apr 6 22:13:01 2020 +0200
patch 8.2.0523: loops are repeated
Problem: Loops are repeated.
Solution: Use FOR_ALL_ macros. (Yegappan Lakshmanan, closes https://github.com/vim/vim/issues/5882)
author | Bram Moolenaar <Bram@vim.org> |
---|---|
date | Mon, 06 Apr 2020 22:15:39 +0200 |
parents | 79e10adc821d |
children | fcccc29bd386 |
line wrap: on
line diff
--- a/src/gui_athena.c +++ b/src/gui_athena.c @@ -954,7 +954,7 @@ gui_mch_new_menu_font(void) vimmenu_T *mp; int max_height = 9999; - for (mp = root_menu; mp != NULL; mp = mp->next) + FOR_ALL_MENUS(mp) { if (menu_is_menubar(mp->dname)) { @@ -1280,7 +1280,7 @@ gui_mch_show_toolbar(int showit) vimmenu_T *toolbar; vimmenu_T *cur; - for (toolbar = root_menu; toolbar; toolbar = toolbar->next) + FOR_ALL_MENUS(toolbar) if (menu_is_toolbar(toolbar->dname)) break; // Assumption: toolbar is NULL if there is no toolbar, @@ -1632,7 +1632,7 @@ gui_athena_popup_callback( { vimmenu_T *i; - for (i = menu->parent->children; i != NULL; i = i->next) + FOR_ALL_CHILD_MENUS(menu->parent, i) { if (i->submenu_id != NULL && XtIsManaged(i->submenu_id)) XtPopdown(i->submenu_id);