commit 2932359000b2f918d5fade79ea4d124d5943cd07
author Bram Moolenaar <Bram@vim.org> Sun Jul 24 22:04:11 2016 +0200
committer Bram Moolenaar <Bram@vim.org> Sun Jul 24 22:04:11 2016 +0200
tree 1ca4fa98569a4d20ce066c7d6254404bcf431b47
parent 6835dc61aebca2b602d85a9d63c449ace58683b4

patch 7.4.2101
Problem:    Looping over windows, buffers and tab pages is inconsistant.
Solution:   Use FOR_ALL_ macros everywhere. (Yegappan Lakshmanan)

src/quickfix.c

diff --git a/src/quickfix.c b/src/quickfix.c
index 51f96dc..b607170 100644
--- a/src/quickfix.c
+++ b/src/quickfix.c
@@ -2040,7 +2040,7 @@
 	if (cmdmod.tab != 0)
 	    wp = NULL;
 	else
-	    for (wp = firstwin; wp != NULL; wp = wp->w_next)
+	    FOR_ALL_WINDOWS(wp)
 		if (wp->w_buffer != NULL && wp->w_buffer->b_help)
 		    break;
 	if (wp != NULL && wp->w_buffer->b_nwindows > 0)