Kazunobu Kuriyama wrote:

> On Jan 22, 2015, at 0:15, Kazunobu Kuriyama <kazunobu.kuriy...@nifty.com> 
> wrote:
> 
> > Hi,
> > 
> > On Jan 21, 2015, at 3:01, Bram Moolenaar <b...@moolenaar.net> wrote:
> > 
> >> Patch 7.4.587
> >> Problem:    Conceal does not work properly with 'linebreak'. (cs86661)
> >> Solution:   Save and restore boguscols. (Christian Brabandt)
> >> Files:         src/screen.c, src/testdir/test_listlbr_utf8.in,
> >>        src/testdir/test_listlbr_utf8.ok
> > 
> > 
> > With normal build, Test 4 of test_listlbr_utf8 fails due to the fact that 
> > the executable resulting from normal build features linebreak but not 
> > conceal, and that the test does not support such a case.
> > 
> > Regards,
> > Kazunobu Kuriyama
> > 
> > -- 
> > -- 
> 
> How about the attached patch to fix the issue above?
> 
> Basically, the patch incorporates the same condition used in
> test_listlbr.in into test_listlbr_utf8.in so that, when the patch is
> included, the latter’s whole test will be skipped if both linebreak
> and conceal are not featured in vim in use.
> 
> As I’m not an expert of vim’s test system, I’m afraid my patch might
> skip some required tests inadvertently. So I’d be happy if someone
> could help to check it.
> 
> Or, does the issue happens only to me?  I’m feeling uneasy because no
> one comes up with the issue so far :-)

The conceal feature is only enabled in the big feature set.  I can
reproduce the problem.

Thanks for the patch!

-- 
hundred-and-one symptoms of being an internet addict:
106. When told to "go to your room" you inform your parents that you
     can't...because you were kicked out and banned.

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

-- 
-- 
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 vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Raspunde prin e-mail lui