Use tmux bracketed paste

[carlocab]

Fixes #262, KristofferC/OhMyREPL.jl#224, #194 (comment)
Related to #254

I tried looking for terminal applications that don't support bracketed paste #254 (comment) but haven't managed to find any. Playing around a little bit with it Python hasn't revealed any problems either.

[jpalardy]

hi @carlocab

I've been running this patch (manually) for a few weeks now, I don't see bad side-effects.

Let's merge it for now, and see what comes up.

[carlocab]

Great! If it helps allay any worries you have, tmux is soon making bracketed-paste the default behaviour for <Prefix>-]:

https://github.com/tmux/tmux/blob/9a5b2c93a7129a6e6e74e4b6d4c2c78dd568b8d4/CHANGES#L13

[kdauria]

I don't quite understand why, but this change broke my usual workflow. Within a tmux session inside of an ubuntu20.04 docker container, I send commands from vim to ipython. With the latest commit (dein#add('jpalardy/vim-slime'))

When I manually deleted the two dashes causing the syntax error and then press Enter to issue the commands, the ipython terminal just hangs forever.

With an older commit (dein#add('jpalardy/vim-slime', { 'rev': 'e7b3cb0bb55a6ca65b218d7e825d9aac71d60638' }))

It seems like with this update that the cpaste -q is no longer needed?

[carlocab]

@kdauria Ugh, sorry about that. What happens if you remove %cpaste -q and -- entirely?

I don't use IPython much, but from my (admittedly limited, and possibly mistaken) understanding is that %paste and %cpaste were early implementations of bracketed-paste before terminal apps started supporting it. I suspect you should now be able to get by without %cpaste with the new update, but unfortunately, it will also break any old code you have.

[kdauria]

Thanks @carlocab . I'll try fiddling around with this line:

vim-slime/ftplugin/python/slime.vim

Line 8 in 92421f0

return ["%cpaste -q\n", g:slime_dispatch_ipython_pause, a:text, "--\n"]

[carlocab]

Thanks @carlocab . I'll try fiddling around with this line:

vim-slime/ftplugin/python/slime.vim

Line 8 in 92421f0

return ["%cpaste -q\n", g:slime_dispatch_ipython_pause, a:text, "--\n"]

@kdauria

Yes, I think that should do it! I think you'll want to keep the \n at the end, but with that, your snippet should execute fine. I'm not sure how it'll handle indentation though.

If you have a code snippet with indentation I could play with to see what works, please do share it so I can try to test it out too.

[carlocab]

Ok, now that I think about it playing around with that line will probably affect users who aren't using tmux.

It might be better to just deactivate bracketed paste if g:slime_python_ipython = 1. The master branch currently works for me in Python even if that variable is not set.

[kdauria]

@carlocab I gave it a try. https://github.com/kdauria/vim-slime/pull/1

I'm stuck on how to send a carriage return in ipython in order to "submit" the commands. I tried adding in my vim config, but that didn't work.

    xmap <Leader>e <Plug>SlimeRegionSend<CR>
    nmap <Leader>e <Plug>SlimeParagraphSend<CR>

FWIW, I really like the bracketed paste. It seems to be less buggy and also allows ipython to do syntax highlighting.

[carlocab]

Yea, I got stuck on the same thing. I tried looking up how to get IPython to read a character as a carriage return, but no luck.

[carlocab]

A potential, but possibly ugly, solution would be for TmuxSend not to invoke the -p flag, but to wrap the text it sends with the bracketed-paste escape codes directly. This would get around the problem of bracketed-paste eating the \n after %cpaste and --, and should in theory exhibit identical behaviour elsewhere.

However, I don't know what %cpaste does with those escape codes. I'll try to carve out time tomorrow to try it out.

[jpalardy]

reverted 10e8c2b