[Tarantool-patches] [PATCH v2 luajit 31/45] test: refactor <lang/gc_stack.lua> LuaJIT test

Sergey Bronnikov sergeyb at tarantool.org
Thu Aug 22 16:47:44 MSK 2024


Sergey,


On 21.08.2024 11:58, Sergey Kaplun wrote:
> This patch refactors the aforementioned test to make its code style
> closer to ours.
>
> Relates to tarantool/tarantool#9398
> ---

thanks for the patch! LGTM with a minor:

probably line 10 needs more whitespaces:

 >      do local a,b,c,d,e,f,g,h,i,j,k,l,m,n end -- Ensure bigger frame 
size.

>   test/LuaJIT-tests/lang/gc_stack.lua | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/LuaJIT-tests/lang/gc_stack.lua b/test/LuaJIT-tests/lang/gc_stack.lua
> index cd3d8f90..8aee57e3 100644
> --- a/test/LuaJIT-tests/lang/gc_stack.lua
> +++ b/test/LuaJIT-tests/lang/gc_stack.lua
> @@ -1,5 +1,5 @@
>   do --- Marking sparse stack.
> -  local t = setmetatable({}, { __index=function(t, k)
> +  local t = setmetatable({}, { __index = function(t, k)
>       k = k - 1
>       if k == 0 then
>         collectgarbage() -- Mark stack, including holes.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20240822/6dd8f1c4/attachment.htm>


More information about the Tarantool-patches mailing list