alex/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2026-02-22 10:22:17 +10:00
Code Issues Packages Projects Releases Wiki Activity

fix(lua): vim.wait(math.huge) fails #36885

Browse Source 
Problem:
`nlua_wait()` uses `luaL_checkinteger()` which doesn't support
`math.huge` since it's double type. On PUC Lua this fails with
'number has no integer representation' error and on LuaJIT this
overflows int.

Solution:
Use `luaL_checknumber()` and handle `math.huge`.

(cherry picked from commit bc0635a9fc)
This commit is contained in:
skewb1k
2025-12-09 21:59:06 +03:00
committed by github-actions[bot]
parent c1c0078138
commit 9acbf5102f
2 changed files with 16 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
test/functional/lua/vim_spec.lua
View File

@@ -3768,7 +3768,8 @@ stack traceback:
exec_lua([[
function _G.Wait()
vim.rpcnotify(vim.g.channel, 'ready')
local _, interrupted = vim.wait(4000)
-- handles math.huge #36854
local _, interrupted = vim.wait(math.huge)
vim.rpcnotify(vim.g.channel, 'wait', interrupted)
end
]])
@@ -3782,7 +3783,7 @@ stack traceback:
exec_lua([[
function _G.Wait()
vim.rpcnotify(vim.g.channel, 'ready')
local _, interrupted = vim.wait(4000, function() end)
local _, interrupted = vim.wait(math.huge, function() end)
vim.rpcnotify(vim.g.channel, 'wait', interrupted)
end
]])