Re: [Tarantool-patches] [PATCH v2 luajit 34/45] test: enable <misc/stackov.lua> LuaJIT test

[Sergey Kaplun via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

Hi, Sergey!
Thanks for the review!
Please consider my answer below.

On 22.08.24, Sergey Bronnikov wrote:
> Sergey,
> 
> On 21.08.2024 11:58, Sergey Kaplun wrote:
> > This patch moves the aforementioned test from the <misc> to the <lang/>
> > directory, includes it in <index>, names the subtests, and adds testing
> > of the error message for all non-tail call cases.
> >
> > Part of tarantool/tarantool#9398
> > ---
> thanks for the patch! LGTM with a minor comment below.
> >   test/LuaJIT-tests/lang/index       |  1 +
> >   test/LuaJIT-tests/lang/stackov.lua | 48 ++++++++++++++++++++++++++++++
> Probably filename "stack_overflow.lua" will be better. Feel free to ignore.

I prefer to leave it as is, since it is consistent with the error name
in sources, if you don't insist.

> >   test/LuaJIT-tests/misc/stackov.lua | 40 -------------------------
> >   3 files changed, 49 insertions(+), 40 deletions(-)
> >   create mode 100644 test/LuaJIT-tests/lang/stackov.lua
> >   delete mode 100644 test/LuaJIT-tests/misc/stackov.lua
> >

<snipped>

-- 
Best regards,
Sergey Kaplun