rneovim.git/test/functional/clipboard, branch v0.3.7

clipboard: Support custom VimL functions #9304

2018-12-02T23:07:08+00:00

Up to now g:clipboard["copy"] only supported string values invoked as
system commands.

This commit enables the use of VimL functions instead. The function
signatures are the same as in provider/clipboard.vim. A clipboard
provider is expected to store and return a list of lines (i.e. the text)
and a register type (as seen in setreg()).

cache_enabled is ignored if "copy" is provided by a VimL function.

msg: do not scroll entire screen (#8088)

2018-03-31T09:12:27+00:00

doc: replace ":CheckHealth" with ":checkhealth"

2017-10-16T23:51:21+00:00

ops: save and restore clipboard batch status when entering cmdline window

2017-10-15T14:36:00+00:00

clipboard: disallow recursion; show hint only once (#7203)

2017-08-22T18:31:54+00:00

- Show hint only once per session.
- provider#clipboard#Call(): prevent recursion
- provider#clear_stderr(): use has_key(), because :silent! is still
  captured by :redir.

closes #7184

clipboard: test g:clipboard validation, fix a bug

2017-08-20T20:17:03+00:00

Also fix `:help foo` highlighting in health.vim

clipboard: remove start_batch_changes() in redir_write()

2017-08-20T18:01:22+00:00

start_batch_changes() doesn't avoid invoking the clipboard
once-per-line, because the loop is actually in ex_echo(), which calls
redir_write() for each message. But we've already entered
start_batch_changes() by then, so that was never the problem.

    redir_write at /home/vagrant/old.neovim/build/../src/nvim/message.c:2523
    msg_puts_attr_len at /home/vagrant/old.neovim/build/../src/nvim/message.c:1600
    msg_outtrans_len_attr at /home/vagrant/old.neovim/build/../src/nvim/message.c:1221
    ex_echo at /home/vagrant/old.neovim/build/../src/nvim/eval.c:19433
    do_one_cmd at /home/vagrant/old.neovim/build/../src/nvim/ex_docmd.c:2242

Trying to defer _explicit_ clipboard updates is difficult.
    :redir @+ | silent echo system('cat foo') | redir END
is essentially equivalent to:
    for l in readfile('foo')
        let @+ .= l
    endfor
We cannot make judgements about when to ignore a script's bad decisions.
start_batch_changes() only works around the case of clipboard=unnamed,
i.e. _implicit_ clipboard updates (`:g/foo/d`).  Not explicit
assignment.

clipboard: avoid error flood during :redir

2017-08-20T17:49:42+00:00

redir_write():
- This is a "batch" operation which was not yet covered by
  start_batch_changes()

adjust_clipboard_name():
- msg() and friends during :redir will, of course, cause redir_write()
  to try to capture that message, which causes recursion.
- EMSG() here is trouble: if it interrupts :redir it is a mess.
  Rather than deal with the mess, show a non-error message.

closes #7182
closes #7184
closes #7183
ref #6048
ref #7032

functests: Replace execute with either command or feed_command

2017-04-09T00:24:08+00:00

Hope this will make people using feed_command less likely: this hides bugs.
Already found at least two:

1. msgpackparse() will show internal error: hash_add() in case of duplicate
   keys, though it will still work correctly. Currently silenced.
2. ttimeoutlen was spelled incorrectly, resulting in option not being set when
   expected. Test was still functioning somehow though. Currently fixed.

clipboard: set v:register after startup (#5708)

2017-03-18T18:33:25+00:00

Fixes #5697