Mercurial > vim
diff src/window.c @ 19888:435726a03481 v8.2.0500
patch 8.2.0500: using the same loop in many places
Commit: https://github.com/vim/vim/commit/aeea72151c31d686bcbb7b06d895006d7363585c
Author: Bram Moolenaar <Bram@vim.org>
Date: Thu Apr 2 18:50:46 2020 +0200
patch 8.2.0500: using the same loop in many places
Problem: Using the same loop in many places.
Solution: Define more FOR_ALL macros. (Yegappan Lakshmanan, closes https://github.com/vim/vim/issues/5339)
author | Bram Moolenaar <Bram@vim.org> |
---|---|
date | Thu, 02 Apr 2020 19:00:05 +0200 |
parents | 1d6bc6b31c2e |
children | 8c401bc7f32b |
line wrap: on
line diff
--- a/src/window.c +++ b/src/window.c @@ -1428,10 +1428,10 @@ win_valid_popup(win_T *win UNUSED) #ifdef FEAT_PROP_POPUP win_T *wp; - for (wp = first_popupwin; wp != NULL; wp = wp->w_next) + FOR_ALL_POPUPWINS(wp) if (wp == win) return TRUE; - for (wp = curtab->tp_first_popupwin; wp != NULL; wp = wp->w_next) + FOR_ALL_POPUPWINS_IN_TAB(curtab, wp) if (wp == win) return TRUE; #endif @@ -1473,7 +1473,7 @@ win_valid_any_tab(win_T *win) return TRUE; } #ifdef FEAT_PROP_POPUP - for (wp = tp->tp_first_popupwin; wp != NULL; wp = wp->w_next) + FOR_ALL_POPUPWINS_IN_TAB(tp, wp) if (wp == win) return TRUE; #endif @@ -2276,7 +2276,7 @@ close_windows( { nexttp = tp->tp_next; if (tp != curtab) - for (wp = tp->tp_firstwin; wp != NULL; wp = wp->w_next) + FOR_ALL_WINDOWS_IN_TAB(tp, wp) if (wp->w_buffer == buf && !(wp->w_closing || wp->w_buffer->b_locked > 0)) { @@ -4785,7 +4785,7 @@ buf_jump_open_tab(buf_T *buf) FOR_ALL_TABPAGES(tp) if (tp != curtab) { - for (wp = tp->tp_firstwin; wp != NULL; wp = wp->w_next) + FOR_ALL_WINDOWS_IN_TAB(tp, wp) if (wp->w_buffer == buf) break; if (wp != NULL) @@ -4968,7 +4968,7 @@ win_free( // Remove the window from the b_wininfo lists, it may happen that the // freed memory is re-used for another window. FOR_ALL_BUFFERS(buf) - for (wip = buf->b_wininfo; wip != NULL; wip = wip->wi_next) + FOR_ALL_BUF_WININFO(buf, wip) if (wip->wi_win == wp) wip->wi_win = NULL;