[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