Patch 7.4.292
Problem: Searching for "a" does not match accented "a" with new regexp
engine, does match with old engine. (David Bürgin)
"ca" does not match "ca" with accented "a" with either engine.
Solution: Change the old engine, check for following composing character
also for single-byte patterns.
Files: src/regexp.c, src/testdir/test95.in, src/testdir/test95.ok
*** ../vim-7.4.291/src/regexp.c 2014-05-13 16:46:25.693696760 +0200
--- src/regexp.c 2014-05-13 17:45:50.977727970 +0200
***************
*** 4692,4722 ****
/* match empty string always works; happens when "~" is
* empty. */
}
! else if (opnd[1] == NUL
#ifdef FEAT_MBYTE
&& !(enc_utf8 && ireg_ic)
#endif
)
! ++reginput; /* matched a single char */
! else
! {
! len = (int)STRLEN(opnd);
! /* Need to match first byte again for multi-byte. */
! if (cstrncmp(opnd, reginput, &len) != 0)
! status = RA_NOMATCH;
#ifdef FEAT_MBYTE
/* Check for following composing character. */
! else if (enc_utf8
! && UTF_COMPOSINGLIKE(reginput, reginput + len))
{
/* raaron: This code makes a composing character get
* ignored, which is the correct behavior (sometimes)
* for voweled Hebrew texts. */
! if (!ireg_icombine)
! status = RA_NOMATCH;
}
#endif
! else
reginput += len;
}
}
--- 4692,4728 ----
/* match empty string always works; happens when "~" is
* empty. */
}
! else
! {
! if (opnd[1] == NUL
#ifdef FEAT_MBYTE
&& !(enc_utf8 && ireg_ic)
#endif
)
! {
! len = 1; /* matched a single byte above */
! }
! else
! {
! /* Need to match first byte again for multi-byte. */
! len = (int)STRLEN(opnd);
! if (cstrncmp(opnd, reginput, &len) != 0)
! status = RA_NOMATCH;
! }
#ifdef FEAT_MBYTE
/* Check for following composing character. */
! if (status != RA_NOMATCH
! && enc_utf8
! && UTF_COMPOSINGLIKE(reginput, reginput + len)
! && !ireg_icombine)
{
/* raaron: This code makes a composing character get
* ignored, which is the correct behavior (sometimes)
* for voweled Hebrew texts. */
! status = RA_NOMATCH;
}
#endif
! if (status != RA_NOMATCH)
reginput += len;
}
}
*** ../vim-7.4.291/src/testdir/test95.in 2013-07-21 16:53:52.000000000
+0200
--- src/testdir/test95.in 2014-05-13 17:49:00.201729626 +0200
***************
*** 50,55 ****
--- 50,57 ----
:call add(tl, [2, ".\u05b9", " y\u05bb\u05b9 x\u05b9 ", "y\u05bb\u05b9"])
:call add(tl, [1, "\u05b9\u05bb", " y\u05b9 x\u05b9\u05bb ", "x\u05b9\u05bb"])
:call add(tl, [2, ".\u05b9\u05bb", " y\u05bb x\u05b9\u05bb ",
"x\u05b9\u05bb"])
+ :call add(tl, [2, "a", "ca\u0300t"])
+ :call add(tl, [2, "a\u0300", "ca\u0300t", "a\u0300"])
:"""" Test \Z
*** ../vim-7.4.291/src/testdir/test95.ok 2013-07-21 17:01:22.000000000
+0200
--- src/testdir/test95.ok 2014-05-13 17:49:46.709730033 +0200
***************
*** 67,72 ****
--- 67,78 ----
OK 0 - .ֹֻ
OK 1 - .ֹֻ
OK 2 - .ֹֻ
+ OK 0 - a
+ OK 1 - a
+ OK 2 - a
+ OK 0 - à
+ OK 1 - à
+ OK 2 - à
OK 0 - ú\Z
OK 1 - ú\Z
OK 2 - ú\Z
*** ../vim-7.4.291/src/version.c 2014-05-13 16:46:25.693696760 +0200
--- src/version.c 2014-05-13 18:00:22.149735596 +0200
***************
*** 736,737 ****
--- 736,739 ----
{ /* Add new patch number below this line */
+ /**/
+ 292,
/**/
--
hundred-and-one symptoms of being an internet addict:
154. You fondle your mouse.
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.