[vim/vim] add TextPutPre and TextPutPost autocommands (PR #20144)
Foxe Chen
You can view, comment on, or merge this pull request online at:
https://github.com/vim/vim/pull/20144
Commit Summary
- a446430 add TextPutPre and TextPutPost
File Changes
(9 files)
- M runtime/doc/autocmd.txt (39)
- M runtime/doc/tags (3)
- M src/autocmd.c (21)
- M src/getchar.c (4)
- M src/globals.h (8)
- M src/proto/autocmd.pro (2)
- M src/register.c (154)
- M src/testdir/test_autocmd.vim (93)
- M src/vim.h (4)
Patch Links:
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- ebf11f0 add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- 4fc08d1 add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- cc17f8e add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- 6df9c4d add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- 86c48b1 add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Foxe Chen
64-bitman left a comment (vim/vim#20144)Heres a script for highlighted put (adapted from hlyank plugin)
vim9script def HighlightedPut() var hlgroup = get(g:, "hlput_hlgroup", "IncSearch") var duration = min([get(g:, "hlput_duration", 300), 3000]) var [beg, end] = [getpos("'["), getpos("']")] var type = v:event.regtype ?? 'v' var pos = getregionpos(beg, end, {type: type, exclusive: false}) var m = matchaddpos(hlgroup, pos->mapnew((_, v) => { var col_beg = v[0][2] + v[0][3] var col_end = v[1][2] + v[1][3] + 1 return [v[0][1], col_beg, col_end - col_beg] })) var winid = win_getid() timer_start(duration, (_) => { if winbufnr(winid) != -1 m->matchdelete(winid) endif }) enddef augroup hlput autocmd! autocmd TextPutPost * HighlightedPut() augroup END
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- bd64305 add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- 5bc4209 add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- 0867027 add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
zeertzjq
@zeertzjq commented on this pull request.
In src/testdir/test_autocmd.vim:
> @@ -5967,4 +5967,97 @@ func Test_autocmd_add_secure()
call assert_fails('sandbox call autocmd_delete([{"event": "BufRead"}])', 'E48:')
endfunc
+func Test_TextPostX()
⬇️ Suggested change
-func Test_TextPostX() +func Test_TextPutX()
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
zeertzjq
@zeertzjq commented on this pull request.
In src/register.c:
> + // register type
+ switch (get_reg_type(regname, ®len))
+ {
+ case MLINE:
+ buf[0] = 'V';
+ buf[1] = NUL;
+ buflen = 1;
+ break;
+ case MCHAR:
+ buf[0] = 'v';
+ buf[1] = NUL;
+ buflen = 1;
+ break;
+ case MBLOCK:
+ buflen = vim_snprintf_safelen((char *)buf, sizeof(buf),
+ "%c%ld", Ctrl_V, reglen + 1);
+ break;
+ default:
+ buf[0] = NUL;
+ buflen = 0;
+ break;
+ }
This code is the same as yank_do_autocmd(). I think it should be deduplicated.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- 51d9a95 add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Foxe Chen
@64-bitman pushed 1 commit.
- 60ab75c add TextPutPre and TextPutPost
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Christian Brabandt
chrisbra left a comment (vim/vim#20144)Thanks. Does this handle arbitrary complex content from e.g. the expression register?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Foxe Chen
64-bitman left a comment (vim/vim#20144)Thanks. Does this handle arbitrary complex content from e.g. the expression register?
Well the autocommands trigger for the expression register, however you cannot set the register contents before it is put, since the expression register + the other special registers don't have a yankreg_T associated with them.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.