Tarantool development patches archive
 help / color / mirror / Atom feed
* [Tarantool-patches] [PATCH luajit 0/2] Fix tonumber("-0") inconsistencies
@ 2022-01-26 12:19 Sergey Kaplun via Tarantool-patches
  2022-01-26 12:19 ` [Tarantool-patches] [PATCH luajit 1/2] Fix tonumber("-0") Sergey Kaplun via Tarantool-patches
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Sergey Kaplun via Tarantool-patches @ 2022-01-26 12:19 UTC (permalink / raw)
  To: Sergey Ostanevich, Igor Munkin; +Cc: tarantool-patches

This patchset fixes incorrect behaviour for `tonumber("-0")` different
from Lua 5.1.

The first patch fixes it for non-dualnum mode (default on x86 and x64).
The second fixes it for dual-number mode too.
So, there is the corresponding skipcond in the test for the first patch
that is removed in the next patch.

Related issues/PR:
* https://github.com/LuaJIT/LuaJIT/issues/528
* https://github.com/LuaJIT/LuaJIT/pull/787
* https://github.com/tarantool/tarantool/issues/6548
Branch: https://github.com/tarantool/tarantool/tree/skaplun/lj-528-tonumber-0-full-ci
Tarantool branch: https://github.com/tarantool/luajit/tree/skaplun/lj-528-tonumber-0-full-ci

Mike Pall (2):
  Fix tonumber("-0").
  Fix tonumber("-0") in dual-number mode.

 src/lj_strscan.c                                | 13 ++++++++-----
 test/tarantool-tests/lj-528-tonumber-0.test.lua | 13 +++++++++++++
 2 files changed, 21 insertions(+), 5 deletions(-)
 create mode 100644 test/tarantool-tests/lj-528-tonumber-0.test.lua

-- 
2.34.1


^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2022-06-30 12:16 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-26 12:19 [Tarantool-patches] [PATCH luajit 0/2] Fix tonumber("-0") inconsistencies Sergey Kaplun via Tarantool-patches
2022-01-26 12:19 ` [Tarantool-patches] [PATCH luajit 1/2] Fix tonumber("-0") Sergey Kaplun via Tarantool-patches
2022-06-27 13:47   ` Igor Munkin via Tarantool-patches
2022-06-27 20:29   ` sergos via Tarantool-patches
2022-01-26 12:19 ` [Tarantool-patches] [PATCH luajit 2/2] Fix tonumber("-0") in dual-number mode Sergey Kaplun via Tarantool-patches
2022-01-27  9:50   ` Sergey Kaplun via Tarantool-patches
2022-06-27 13:48   ` Igor Munkin via Tarantool-patches
2022-06-27 15:51   ` sergos via Tarantool-patches
2022-06-30 12:09 ` [Tarantool-patches] [PATCH luajit 0/2] Fix tonumber("-0") inconsistencies Igor Munkin via Tarantool-patches

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox