updated for version 7.3.437
Problem: Continue looping inside FOR_ALL_TAB_WINDOWS even when already done.
Solution: Use goto instead of break. (Hirohito Higashi)
diff --git a/src/fileio.c b/src/fileio.c
index b30d311..8ed878d 100644
--- a/src/fileio.c
+++ b/src/fileio.c
@@ -8898,10 +8898,11 @@
if (tp != curtab)
goto_tabpage_tp(tp);
win_goto(aucmd_win);
- break;
+ goto win_found;
}
}
}
+win_found:
/* Remove the window and frame from the tree of frames. */
(void)winframe_remove(curwin, &dummy, NULL);
diff --git a/src/globals.h b/src/globals.h
index 1385192..a9f1c2d 100644
--- a/src/globals.h
+++ b/src/globals.h
@@ -535,6 +535,10 @@
EXTERN win_T *prevwin INIT(= NULL); /* previous window */
# define W_NEXT(wp) ((wp)->w_next)
# define FOR_ALL_WINDOWS(wp) for (wp = firstwin; wp != NULL; wp = wp->w_next)
+/*
+ * When using this macro "break" only breaks out of the inner loop. Use "goto"
+ * to break out of the tabpage loop.
+ */
# define FOR_ALL_TAB_WINDOWS(tp, wp) \
for ((tp) = first_tabpage; (tp) != NULL; (tp) = (tp)->tp_next) \
for ((wp) = ((tp) == curtab) \
diff --git a/src/version.c b/src/version.c
index 2c7ad8f..59498a4 100644
--- a/src/version.c
+++ b/src/version.c
@@ -715,6 +715,8 @@
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 437,
+/**/
436,
/**/
435,