Christian wrote:
> > > > > On Fr, 30 Aug 2019, Bram Moolenaar wrote:
> > > > >
> > > > > > Thanks. Yes, I think we should do this. But the allocation should
> > > > > > probably be done differently, it looks like with cmd_silent set it
> > > > > > still
> > > > > > computes the size of the command. This will require some more "if"
> > > > > > statements, but makes the size computation more accurate.
> > > > >
> > > > > Well, yeah I thought this wouldn't hurt.
> > > > >
> > > > > So how about the attached patch then? It will simply use all
> > > > > available
> > > > > space in the command line. Not sure this is correct however.
> > > >
> > > > It's tricky with all the conditions. But it looks OK. All tests pass.
> > > > Let's include this and check that nothing goes wrong. Can we cover this
> > > > with a test?
> > >
> > > Apologizes, the last patch was wrong and caused a strtrunc message
> > > ('...')
> > >
> > > Perhaps we don't even need the added condition `|| cmd_silent` at all?
> > >
> > > Here is a fix including a test.
> >
> > Thanks. The test doesn't fail without the fix though.
>
> Yeah, I did not update the test, I tried, but the message is not
> truncated when running the test. I think this happens because the output
> of `execute()` is actually scrolled. Not sure.
>
> This patch should do it, but only works when run interactively.
>
> diff --git a/src/testdir/test_search_stat.vim
> b/src/testdir/test_search_stat.vim
> index f23952915..33c3858bc 100644
> --- a/src/testdir/test_search_stat.vim
> +++ b/src/testdir/test_search_stat.vim
> @@ -176,7 +176,9 @@ func! Test_search_stat()
> let g:b = split(g:a, "\n")[-1]
> let stat = '\[1/2\]'
> call assert_notmatch(pat .. stat, g:b)
> - call assert_match(stat, g:b)
> + " Test that the message is not truncated
> + " it would insert '...' into the output.
> + call assert_match('^\s\+' .. stat, g:b)
> unmap n
>
> " Clean up
Hmm, since nobody runs the tests manually that doesn't help much.
How about using a screendump? That makes these things a lot easier to
write.
--
I'm so disorganized my keyboard isn't even in alphabetical order!
/// 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_use" 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_use" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to vim_use+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/vim_use/201909041121.x84BLsTM024711%40masaka.moolenaar.net.
