Mercurial > vim
changeset 5006:f451d60ab8ec v7.3.1247
updated for version 7.3.1247
Problem: New regexp engine: '[ ]\@!\p\%([ ]\@!\p\)*:' does not always match.
Solution: When there is a PIM add a duplicate state that starts at another
position.
| author | Bram Moolenaar <bram@vim.org> |
|---|---|
| date | Wed, 26 Jun 2013 18:16:58 +0200 |
| parents | b314c98a48d4 |
| children | e285b27368d1 |
| files | src/regexp_nfa.c src/testdir/test64.in src/testdir/test64.ok src/version.c |
| diffstat | 4 files changed, 15 insertions(+), 8 deletions(-) [+] |
line wrap: on
line diff
--- a/src/regexp_nfa.c +++ b/src/regexp_nfa.c @@ -3642,14 +3642,14 @@ sub_equal(sub1, sub2) if (i < sub1->in_use) s1 = sub1->list.multi[i].start.lnum; else - s1 = 0; + s1 = -1; if (i < sub2->in_use) s2 = sub2->list.multi[i].start.lnum; else - s2 = 0; + s2 = -1; if (s1 != s2) return FALSE; - if (s1 != 0 && sub1->list.multi[i].start.col + if (s1 != -1 && sub1->list.multi[i].start.col != sub2->list.multi[i].start.col) return FALSE; } @@ -3931,8 +3931,9 @@ addstate(l, state, subs, pim, off) if (state->lastlist[nfa_ll_index] == l->id) { /* This state is already in the list, don't add it again, - * unless it is an MOPEN that is used for a backreference. */ - if (!nfa_has_backref) + * unless it is an MOPEN that is used for a backreference or + * when there is a PIM. */ + if (!nfa_has_backref && pim == NULL) { skip_add: #ifdef ENABLE_LOG @@ -3949,9 +3950,9 @@ skip_add: goto skip_add; } - /* When there are backreferences the number of states may be (a - * lot) bigger than anticipated. */ - if (nfa_has_backref && l->n == l->len) + /* When there are backreferences or PIMs the number of states may + * be (a lot) bigger than anticipated. */ + if (l->n == l->len) { int newlen = l->len * 3 / 2 + 50;
--- a/src/testdir/test64.in +++ b/src/testdir/test64.in @@ -338,6 +338,7 @@ STARTTEST :call add(tl, [2, '^\%(.*bar\)\@!.*\zsfoo', ' bar foo ']) :call add(tl, [2, '^\%(.*bar\)\@!.*\zsfoo', ' foo bar ']) :call add(tl, [2, '^\%(.*bar\)\@!.*\zsfoo', ' foo xxx ', 'foo']) +:call add(tl, [2, '[ ]\@!\p\%([ ]\@!\p\)*:', 'implicit mappings:', 'mappings:']) :" :"""" Combining different tests and features :call add(tl, [2, '[[:alpha:]]\{-2,6}', '787abcdiuhsasiuhb4', 'ab'])
--- a/src/testdir/test64.ok +++ b/src/testdir/test64.ok @@ -770,6 +770,9 @@ OK 2 - ^\%(.*bar\)\@!.*\zsfoo OK 0 - ^\%(.*bar\)\@!.*\zsfoo OK 1 - ^\%(.*bar\)\@!.*\zsfoo OK 2 - ^\%(.*bar\)\@!.*\zsfoo +OK 0 - [ ]\@!\p\%([ ]\@!\p\)*: +OK 1 - [ ]\@!\p\%([ ]\@!\p\)*: +OK 2 - [ ]\@!\p\%([ ]\@!\p\)*: OK 0 - [[:alpha:]]\{-2,6} OK 1 - [[:alpha:]]\{-2,6} OK 2 - [[:alpha:]]\{-2,6}