Christian Brabandt wrote:
> On Sa, 09 Jul 2011, Bram Moolenaar wrote:
>
> >
> > Bill Bee beeyawned wrote:
> >
> > > These have a problem with "f" and "F"
> > > Mac gui vim-app 7.3.237
> > > Mac terminal vim 7.3.237
> > > WinXP gui vim 7.3.237
> > >
> > > If there are consecutive characters as "4444", using "f" to find the
> > > next "4" will skip to every other "4" or in the case of "44" with no
> > > other "4's" in the line, will not progress to the next "4".
> > >
> > > These work "correctly"
> > > Mac gui vim-app 7.3.146
> > > Mac terminal 7.3.146
> > > Mac terminal vi 6.2
> > > Linux gui vim 7.3.189
> > > Linux terminal vim 7.3.189
> > >
> > > Is this a new feature or a bug?
> >
> > I don't see a problem with "f4", but using ";" skips one 4. That's
> > a bug, probably introduced by patch 7.3.235.
>
> Ah, okay, seems like one needs to check for t_cmd as well. This patch
> should fix it:
> diff --git a/src/search.c b/src/search.c
> --- a/src/search.c
> +++ b/src/search.c
> @@ -1585,7 +1585,7 @@
> /* Force a move of at least one char, so ";" and "," will move the
> * cursor, even if the cursor is right in front of char we are looking
> * at. */
> - if (vim_strchr(p_cpo, CPO_SCOLON) == NULL && count == 1)
> + if (vim_strchr(p_cpo, CPO_SCOLON) == NULL && count == 1 && t_cmd)
> stop = FALSE;
> }
>
>
> A temporary work around would be: :set cpo+=;
Thanks for the fix. Can you also update the test so that it fails
without this patch?
--
God made the integers; all else is the work of Man.
-- Kronecker
/// 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
