api/lua: make nvim_execute_lua use native lua floats, not special tables

Rationale: the purpose of nvim_execute_lua is to simply call lua code with lua
values. If a lua function expects a floating point value, it should be enough
to specify a float as argument to nvim_execute_lua.

However, make sure to preserve the existing roundtripping behavior of
API values when using `vim.api` functions. This is covered by existing
lua/api_spec.lua tests.
This commit is contained in:
Björn Linse
2019-07-01 15:50:57 +02:00
parent 740fb337dd
commit 6ee05536ca
4 changed files with 35 additions and 21 deletions

View File

@@ -562,7 +562,7 @@ Object executor_exec_lua_api(const String str, const Array args, Error *err)
}
for (size_t i = 0; i < args.size; i++) {
nlua_push_Object(lstate, args.items[i]);
nlua_push_Object(lstate, args.items[i], false);
}
if (lua_pcall(lstate, (int)args.size, 1, 0)) {
@@ -583,7 +583,7 @@ Object executor_exec_lua_cb(LuaRef ref, const char *name, Array args,
nlua_pushref(lstate, ref);
lua_pushstring(lstate, name);
for (size_t i = 0; i < args.size; i++) {
nlua_push_Object(lstate, args.items[i]);
nlua_push_Object(lstate, args.items[i], false);
}
if (lua_pcall(lstate, (int)args.size+1, retval ? 1 : 0, 0)) {