[Tarantool-patches] [PATCH v2 luajit 14/26] test: enable <ffi_jit_conv.lua> in LuaJIT-tests

Sergey Bronnikov sergeyb at tarantool.org
Mon Feb 5 15:51:57 MSK 2024


Hi, Sergey

thanks for the patch. LGTM with a minor comment

On 1/29/24 13:45, Sergey Kaplun wrote:
> This patch renames the C library in the `require()` call. It adds
> groups with the names of subtests and enables the test in <index>. It
> renames enum definitions to avoid conflicts with cdef in other tests.
>
> Part of tarantool/tarantool#9398
> ---
>   test/LuaJIT-tests/lib/ffi/ffi_jit_conv.lua | 70 +++++++++++-----------
>   test/LuaJIT-tests/lib/ffi/index            |  1 +
>   2 files changed, 36 insertions(+), 35 deletions(-)
>
> diff --git a/test/LuaJIT-tests/lib/ffi/ffi_jit_conv.lua b/test/LuaJIT-tests/lib/ffi/ffi_jit_conv.lua
> index d4707db7..0e64fe75 100644
> --- a/test/LuaJIT-tests/lib/ffi/ffi_jit_conv.lua
> +++ b/test/LuaJIT-tests/lib/ffi/ffi_jit_conv.lua
> @@ -1,8 +1,8 @@
<snipped>
>   
> -do
> +do --- int16_t array arithmetic
>     local a = ffi.new("int16_t[100]", 1)
>     for i=1,99 do a[i] = a[i] + a[i-1] end
>     assert(a[99] == 100)
>   end
>   
> -do
> +do --- ligud to pointer
typo: ligud -> lightud
<snipped>


More information about the Tarantool-patches mailing list