On 27/07/12 21:12, Bram Moolenaar wrote:

Patch 7.3.617 (after 7.3.615)
Problem:    Hang on completion.
Solution:   Skip over the space. (Yasuhiro Matsumoto)
Files:      src/ex_docmd.c


*** ../vim-7.3.616/src/ex_docmd.c       2012-07-25 17:22:17.000000000 +0200
--- src/ex_docmd.c      2012-07-27 21:07:42.000000000 +0200
***************
*** 3837,3852 ****
                    while (*p)
                    {
                        if (*p == ' ')
-                       {
                            /* argument starts after a space */
                            arg = p + 1;
!                       }
!                       else
!                       {
!                           if (*p == '\\' && *(p + 1) != NUL)
!                               ++p; /* skip over escaped character */
!                           mb_ptr_adv(p);
!                       }
                    }
                    xp->xp_pattern = arg;
                }
--- 3837,3847 ----
                    while (*p)
                    {
                        if (*p == ' ')
                            /* argument starts after a space */
                            arg = p + 1;
!                       else if (*p == '\\' && *(p + 1) != NUL)
!                           ++p; /* skip over escaped character */
!                       mb_ptr_adv(p);
                    }
                    xp->xp_pattern = arg;
                }
*** ../vim-7.3.616/src/version.c        2012-07-27 21:05:51.000000000 +0200
--- src/version.c       2012-07-27 21:08:31.000000000 +0200
***************
*** 716,717 ****
--- 716,719 ----
   {   /* Add new patch number below this line */
+ /**/
+     617,
   /**/


AFAICT this patch removes two pairs of unneeded braces but changes nothing to the _logic_ of the code. Bram, didn't you forget something in the "true" part of the if? Maybe replace arg=p+1 by arg=++p or something?


Best regards,
Tony.
--
Q:  How many IBM cpu's does it take to do a logical right shift?
A:  33.  1 to hold the bits and 32 to push the register.

--
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

Raspunde prin e-mail lui