[Tarantool-patches] [PATCH v2 luajit 10/30] test: adapt test for debug.setlocal in Lua suite
imun at tarantool.org
Wed Mar 31 01:14:39 MSK 2021
Thanks for the patch! LGTM, considering your changes on the branch. Also
consider the comments below.
On 26.03.21, Sergey Kaplun wrote:
> LuaJIT: Lua 5.1 interprets `...` in the vararg functions like
Looks like you just copied the comment below. There is no need for
> an additional first argument unlike LuaJIT does.
> This behaviour is extension is from Lua 5.2.
Typo: s/is extension is/is extension/.
> This patch adapted test considering LuaJIT's and Lua 5.2 behaviour.
Side note: Here it is -- you wrote LuaJIT's but Lua 5.2 (neither Lua's
5.2 nor Lua 5.2's). How come?
> The test is adapted like it done in Lua 5.2 test suite taken from
> Closes tarantool/tarantool#5694
As we discussed before: s/Closes/Resolves/.
> Part of tarantool/tarantool#5845
> Part of tarantool/tarantool#4473
More information about the Tarantool-patches