[Tarantool-patches] [PATCH luajit] x86/x64: Fix math.ceil(-0.9) result sign.

Sergey Kaplun skaplun at tarantool.org
Thu Nov 9 17:32:53 MSK 2023


Hi, Sergey!
Thanks for the review!
Fixed your comment inline, see the iterative patch below.
Branch is rebased and force-pushed.

===================================================================
diff --git a/test/tarantool-tests/lj-859-math-ceil-sign.test.lua b/test/tarantool-tests/lj-859-math-ceil-sign.test.lua
index 831b6480..a67de888 100644
--- a/test/tarantool-tests/lj-859-math-ceil-sign.test.lua
+++ b/test/tarantool-tests/lj-859-math-ceil-sign.test.lua
@@ -11,7 +11,7 @@ test:plan(1)
 local IS_DUALNUM = tostring(tonumber('-0')) ~= tostring(-0)
 local IS_X86_64 = jit.arch == 'x86' or jit.arch == 'x64'
 
--- Use `tostirng()` to compare the sign of the returned value.
+-- Use `tostring()` to compare the sign of the returned value.
 -- Take any value from the mentioned range. The chosen one is
 -- mentioned in the commit message.
 test:ok((IS_DUALNUM and IS_X86_64) or tostring(math.ceil(-0.9)) == '-0',
===================================================================

-- 
Best regards,
Sergey Kaplun


More information about the Tarantool-patches mailing list