updated for version 7.3.625

Problem:    "gn" does not handle zero-width matches correctly.
Solution:   Handle zero-width patterns specially. (Christian Brabandt)

diff --git a/src/search.c b/src/search.c
index aa858f9eef6d95082df0cf7869a6ccea0517cea7..5ace05d6ae00a7b1558a24594235958633478642 100644 (file)
--- a/src/search.c
+++ b/src/search.c
@@ -4546,6 +4546,9 @@ current_search(count, forward)
     int                visual_active = FALSE;
     int                flags = 0;
     pos_T      save_VIsual;
+    regmmatch_T        regmatch;
+    int                nmatched = 0;
+    int                zerowidth = FALSE;
 
 
     /* wrapping should not occur */
@@ -4580,6 +4583,24 @@ current_search(count, forward)
     else
        orig_pos = pos = start_pos = curwin->w_cursor;
 
+    /*
+     * Check for zero-width pattern.
+     */
+    if (search_regcomp(spats[last_idx].pat, RE_SEARCH, RE_SEARCH,
+                            ((SEARCH_HIS + SEARCH_KEEP)), &regmatch) == FAIL)
+       return FAIL;
+
+    /* Zero-width pattern should match somewhere, then we can check if start
+     * and end are in the same position. */
+    nmatched = vim_regexec_multi(&regmatch, curwin, curbuf,
+                                    curwin->w_cursor.lnum, (colnr_T)0, NULL);
+    if (called_emsg)
+       return FAIL;
+    if (nmatched && regmatch.startpos[0].lnum == regmatch.endpos[0].lnum
+                && regmatch.endpos[0].col == regmatch.startpos[0].col)
+       zerowidth = TRUE;
+    vim_free(regmatch.regprog);
+
     /*
      * The trick is to first search backwards and then search forward again,
      * so that a match at the current cursor position will be correctly
@@ -4587,17 +4608,18 @@ current_search(count, forward)
      */
     for (i = 0; i < 2; i++)
     {
-       if (i && count == 1)
-           flags = SEARCH_START;
-
        if (forward)
            dir = i;
        else
            dir = !i;
+
+       flags = 0;
+       if (!dir && !zerowidth)
+           flags = SEARCH_END;
+
        result = searchit(curwin, curbuf, &pos, (dir ? FORWARD : BACKWARD),
                spats[last_idx].pat, (long) (i ? count : 1),
-               SEARCH_KEEP | flags | (dir ? 0 : SEARCH_END),
-               RE_SEARCH, 0, NULL);
+               SEARCH_KEEP | flags, RE_SEARCH, 0, NULL);
 
        /* First search may fail, but then start searching from the
         * beginning of the file (cursor might be on the search match)
@@ -4629,10 +4651,12 @@ current_search(count, forward)
     }
 
     start_pos = pos;
-    flags = (forward ? SEARCH_END : 0);
+    flags = forward ? SEARCH_END : 0;
 
-    /* move to match */
-    result = searchit(curwin, curbuf, &pos, (forward ? FORWARD : BACKWARD),
+    /* move to match, except for zero-width matches, in which case, we are
+     * already on the next match */
+    if (!zerowidth)
+       result = searchit(curwin, curbuf, &pos, (forward ? FORWARD : BACKWARD),
            spats[last_idx].pat, 0L, flags | SEARCH_KEEP, RE_SEARCH, 0, NULL);
 
     if (!VIsual_active)

diff --git a/src/version.c b/src/version.c
index c8889e49e1ee0cf319f76ebf54ffa46bf9f989a6..c381e03b899a498017c91478369529c8aee182e8 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 */
+/**/
+    625,
 /**/
     624,
 /**/