Mercurial > vim
diff src/screen.c @ 9649:fd9727ae3c49 v7.4.2101
commit https://github.com/vim/vim/commit/2932359000b2f918d5fade79ea4d124d5943cd07
Author: Bram Moolenaar <Bram@vim.org>
Date: Sun Jul 24 22:04:11 2016 +0200
patch 7.4.2101
Problem: Looping over windows, buffers and tab pages is inconsistant.
Solution: Use FOR_ALL_ macros everywhere. (Yegappan Lakshmanan)
| author | Christian Brabandt <cb@256bit.org> |
|---|---|
| date | Sun, 24 Jul 2016 22:15:07 +0200 |
| parents | 123d3c102035 |
| children | b193bdb6ea25 |
line wrap: on
line diff
--- a/src/screen.c +++ b/src/screen.c @@ -730,7 +730,7 @@ update_screen(int type) #ifdef FEAT_WINDOWS /* Reset b_mod_set flags. Going through all windows is probably faster * than going through all buffers (there could be many buffers). */ - for (wp = firstwin; wp != NULL; wp = wp->w_next) + FOR_ALL_WINDOWS(wp) wp->w_buffer->b_mod_set = FALSE; #else curbuf->b_mod_set = FALSE; @@ -962,7 +962,7 @@ update_debug_sign(buf_T *buf, linenr_T l update_prepare(); # ifdef FEAT_WINDOWS - for (wp = firstwin; wp; wp = wp->w_next) + FOR_ALL_WINDOWS(wp) { if (wp->w_redr_type != 0) win_update(wp); @@ -6271,7 +6271,7 @@ status_redraw_all(void) { win_T *wp; - for (wp = firstwin; wp; wp = wp->w_next) + FOR_ALL_WINDOWS(wp) if (wp->w_status_height) { wp->w_redr_status = TRUE; @@ -6287,7 +6287,7 @@ status_redraw_curbuf(void) { win_T *wp; - for (wp = firstwin; wp; wp = wp->w_next) + FOR_ALL_WINDOWS(wp) if (wp->w_status_height != 0 && wp->w_buffer == curbuf) { wp->w_redr_status = TRUE; @@ -6303,7 +6303,7 @@ redraw_statuslines(void) { win_T *wp; - for (wp = firstwin; wp; wp = wp->w_next) + FOR_ALL_WINDOWS(wp) if (wp->w_redr_status) win_redr_status(wp); if (redraw_tabline) @@ -10360,7 +10360,7 @@ draw_tabline(void) else #endif { - for (tp = first_tabpage; tp != NULL; tp = tp->tp_next) + FOR_ALL_TABPAGES(tp) ++tabcount; tabwidth = (Columns - 1 + tabcount / 2) / tabcount;