Lech Lorens wrote:
> > > The attached patch fixes the problems with patch 7.2.356 (by actually
> > > reverting the patch) which caused the folds to be closed in too many
> > > situations.
> >
> > I think some of this doesn't need to be reverted. E.g.:
> >
> > > fold_T *fp;
> > >
> > > /* Mark all folds from top to bot as maybe-small. */
> > > - (void)foldFind(&curwin->w_folds, top, &fp);
> > > + (void)foldFind(&curwin->w_folds, curwin->w_cursor.lnum, &fp);
> > > while (fp < (fold_T *)curwin->w_folds.ga_data +
> > > curwin->w_folds.ga_len
> > > && fp->fd_top < bot)
> > > {
> >
> > I think "top" is the right value. Didn't run some tests with/without
> > this change though.
>
> OK.
>
> I updated test45.in and test45.ok so that they discover the problem with
> syntax-based folds getting closed when ^L is hit. The updated test 45
> - succeeds for Vim 7.2.355,
> - fails for 7.2.356 and any more recent Vim version,
> - succeeds again if the patch from my last email is applied.
Thanks, I'll include the un-patch without that one line and include the
test. I verified the test fails before the un-patch and succeeds after.
--
"Time flies like an arrow". So I put an arrow on my desk, now
awaiting one of these time flies showing up.
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ download, build and distribute -- http://www.A-A-P.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
