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 b30d31102e35f98bf2e454aa21560c62c579c0ff..8ed878d73b75ec6e42abfa7cd45fa486980fc671 100644 (file)
--- a/src/fileio.c
+++ b/src/fileio.c
@@ -8898,10 +8898,11 @@ aucmd_restbuf(aco)
                    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 13851923f1ea0c47115df3dd032b81bea3202c45..a9f1c2d9701f21aabaaf47469fa01a8a8f64552e 100644 (file)
--- a/src/globals.h
+++ b/src/globals.h
@@ -535,6 +535,10 @@ EXTERN win_T       *lastwin;               /* last window */
 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 2c7ad8fb6b2060c945a26bb7a74adcfb4a9acd40..59498a4caa34e43ee1e26d25f835f6c8e013757e 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -714,6 +714,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    437,
 /**/
     436,
 /**/