mirrors/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-09-06 03:18:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(api): nvim_create_user_command addr option should allow ranges #35077

Browse Source 
Problem: Using `addr` without `range` in nvim_create_user_command gives
"No range allowed" error, inconsistent with `:command -addr` behavior.

Solution: Set EX_RANGE flag when `addr` option is specified to match
`:command` behavior.
This commit is contained in:
glepnir
2025-08-03 10:52:54 +08:00
committed by GitHub
parent 93f5bd0caf
commit 22df649210
2 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/nvim/api/command.c
View File

@@ -1170,6 +1170,7 @@ void create_user_command(uint64_t channel_id, String name, Union(String, LuaRef)
goto err; goto err;
}); });
argt |= EX_RANGE;
if (addr_type_arg != ADDR_LINES) { if (addr_type_arg != ADDR_LINES) {
argt |= EX_ZEROR; argt |= EX_ZEROR;
} }

7
test/functional/api/command_spec.lua
View File

@@ -782,6 +782,13 @@ describe('nvim_create_user_command', function()
eq(5, #api.nvim_list_tabpages()) eq(5, #api.nvim_list_tabpages())
eq(1, fn.tabpagenr()) eq(1, fn.tabpagenr())
end) end)
it('"addr" flag allows ranges without explicit "range" flag', function()
exec_lua([[
vim.api.nvim_create_user_command('Test2', 'echo "hello <line1><line2>"', { addr = 'other' })
]])
eq('hello 12', n.exec_capture('1,2Test2'))
end)
end) end)
describe('nvim_del_user_command', function() describe('nvim_del_user_command', function()