Skip to content

Add additional tests for setting 'tagfunc' to a funcref#9250

Closed
yegappan wants to merge 2 commits intovim:masterfrom
yegappan:opfunc
Closed

Add additional tests for setting 'tagfunc' to a funcref#9250
yegappan wants to merge 2 commits intovim:masterfrom
yegappan:opfunc

Conversation

@yegappan
Copy link
Member

@yegappan yegappan commented Dec 1, 2021

Add Vim9 lambda expression support. Update the help text.

@codecov
Copy link

codecov bot commented Dec 1, 2021

Codecov Report

Merging #9250 (b14be40) into master (fad2742) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9250   +/-   ##
=======================================
  Coverage   90.19%   90.19%           
=======================================
  Files         151      151           
  Lines      171294   171294           
=======================================
  Hits       154491   154491           
  Misses      16803    16803           
Flag Coverage Δ
huge-clang-none 87.41% <100.00%> (-0.01%) ⬇️
huge-gcc-none 89.76% <100.00%> (+0.01%) ⬆️
huge-gcc-testgui 88.31% <100.00%> (-0.01%) ⬇️
huge-gcc-unittests 2.46% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/insexpand.c 92.69% <100.00%> (ø)
src/option.c 94.04% <100.00%> (ø)
src/message.c 88.59% <0.00%> (-0.31%) ⬇️
src/term.c 83.50% <0.00%> (-0.17%) ⬇️
src/window.c 93.00% <0.00%> (-0.04%) ⬇️
src/terminal.c 91.38% <0.00%> (+0.03%) ⬆️
src/tag.c 93.88% <0.00%> (+0.05%) ⬆️
src/os_unix.c 72.57% <0.00%> (+0.17%) ⬆️
src/if_xcmdsrv.c 89.78% <0.00%> (+0.52%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fad2742...b14be40. Read the comment docs.

@brammool brammool closed this in 05e59e3 Dec 1, 2021
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Aug 22, 2022
Problem:    Cannot use Vim9 lambda for 'tagfunc'.
Solution:   Make it work, add more tests. (Yegappan Lakshmanan, closes vim/vim#9250)
vim/vim@05e59e3

Omit Vim9 script in code and comment out in tests.
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Nov 5, 2022
Problem:    Cannot use Vim9 lambda for 'tagfunc'.
Solution:   Make it work, add more tests. (Yegappan Lakshmanan, closes vim/vim#9250)
vim/vim@05e59e3

Omit Vim9 script in code and comment out in tests.
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Nov 5, 2022
Problem:    Cannot use Vim9 lambda for 'tagfunc'.
Solution:   Make it work, add more tests. (Yegappan Lakshmanan, closes vim/vim#9250)
vim/vim@05e59e3

Omit Vim9 script in code and comment out in tests.

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Nov 7, 2022
Problem:    Cannot use Vim9 lambda for 'tagfunc'.
Solution:   Make it work, add more tests. (Yegappan Lakshmanan, closes vim/vim#9250)
vim/vim@05e59e3

Omit Vim9 script in code and comment out in tests.

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Nov 7, 2022
Problem:    Cannot use Vim9 lambda for 'tagfunc'.
Solution:   Make it work, add more tests. (Yegappan Lakshmanan, closes vim/vim#9250)
vim/vim@05e59e3

Omit Vim9 script in code and comment out in tests.

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
yesean pushed a commit to yesean/neovim that referenced this pull request Mar 25, 2023
Problem:    Cannot use Vim9 lambda for 'tagfunc'.
Solution:   Make it work, add more tests. (Yegappan Lakshmanan, closes vim/vim#9250)
vim/vim@05e59e3

Omit Vim9 script in code and comment out in tests.

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant