Merge pull request #18229 from zeertzjq/backport-18227-to-release-0.7

[Backport release-0.7] fix(lua): don't mutate opts parameter of vim.keymap.del (#18227)
This commit is contained in:
zeertzjq
2022-04-23 08:41:47 +08:00
committed by GitHub
2 changed files with 33 additions and 1 deletions

View File

@ -130,7 +130,6 @@ function keymap.del(modes, lhs, opts)
local buffer = false
if opts.buffer ~= nil then
buffer = opts.buffer == true and 0 or opts.buffer
opts.buffer = nil
end
if buffer == false then

View File

@ -2755,6 +2755,39 @@ describe('vim.keymap', function()
eq('\nNo mapping found', helpers.exec_capture('nmap asdf'))
end)
it('works with buffer-local mappings', function()
eq(0, exec_lua [[
GlobalCount = 0
vim.keymap.set('n', 'asdf', function() GlobalCount = GlobalCount + 1 end, {buffer=true})
return GlobalCount
]])
feed('asdf\n')
eq(1, exec_lua[[return GlobalCount]])
exec_lua [[
vim.keymap.del('n', 'asdf', {buffer=true})
]]
feed('asdf\n')
eq(1, exec_lua[[return GlobalCount]])
eq('\nNo mapping found', helpers.exec_capture('nmap asdf'))
end)
it('does not mutate the opts parameter', function()
eq(true, exec_lua [[
opts = {buffer=true}
vim.keymap.set('n', 'asdf', function() end, opts)
return opts.buffer
]])
eq(true, exec_lua [[
vim.keymap.del('n', 'asdf', opts)
return opts.buffer
]])
end)
it('can do <Plug> mappings', function()
eq(0, exec_lua [[
GlobalCount = 0