To: vim_dev@googlegroups.com

Subject: Patch 7.4.292

Fcc: outbox

From: Bram Moolenaar <Bram@moolenaar.net>

Mime-Version: 1.0

Content-Type: text/plain; charset=UTF-8

Content-Transfer-Encoding: 8bit

------------

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 -- Bram@Moolenaar.net -- 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    ///