[Tarantool-patches] [PATCH 2/3] lua: table.deepcopy ignores __pairs metamethod
olegrok at tarantool.org
olegrok at tarantool.org
Thu Feb 13 23:33:52 MSK 2020
From: Oleg Babin <babinoleg at mail.ru>
After 1d85144a9b4bbbb026402848efde1ab98bf72633
table.deepcopy changed the behaviour and started
iterate through tables considering __pairs metamethod.
In some cases it broke backward compatibility.
To avoid such problem let's ignore __pairs
and iterate through tables as it was before
Closes #4770
Follow-up #4560
---
Issue: https://github.com/tarantool/tarantool/issues/4770
Branch: https://github.com/tarantool/tarantool/tree/olegrok/table-fixes
src/lua/table.lua | 7 ++++++-
test/app-tap/table.test.lua | 26 +++++++++++++++++++++++++-
2 files changed, 31 insertions(+), 2 deletions(-)
diff --git a/src/lua/table.lua b/src/lua/table.lua
index d83217dcb..4fa9c421c 100644
--- a/src/lua/table.lua
+++ b/src/lua/table.lua
@@ -1,3 +1,8 @@
+-- This pairs implementation doesn't trigger __pairs metamethod
+local function internal_pairs(tbl)
+ return next, tbl, nil
+end
+
local function table_deepcopy_internal(orig, cyclic)
cyclic = cyclic or {}
local copy = orig
@@ -10,7 +15,7 @@ local function table_deepcopy_internal(orig, cyclic)
copy = cyclic[orig]
else
cyclic[orig] = copy
- for orig_key, orig_value in pairs(orig) do
+ for orig_key, orig_value in internal_pairs(orig) do
local key = table_deepcopy_internal(orig_key, cyclic)
copy[key] = table_deepcopy_internal(orig_value, cyclic)
end
diff --git a/test/app-tap/table.test.lua b/test/app-tap/table.test.lua
index 07894f69e..3faf2ed23 100755
--- a/test/app-tap/table.test.lua
+++ b/test/app-tap/table.test.lua
@@ -8,7 +8,7 @@ yaml.cfg{
encode_invalid_as_nil = true,
}
local test = require('tap').test('table')
-test:plan(33)
+test:plan(35)
do -- check basic table.copy (deepcopy)
local example_table = {
@@ -241,4 +241,28 @@ do -- gh-4340: deepcopy doesn't handle __metatable correctly.
)
end
+do -- gh-4770: deepcopy uses __pairs for iteration over table.
+ local original = { a = 1, b = 2 }
+
+ local function custom_pairs(self)
+ local function step(tbl, k)
+ local k, v = next(tbl, k)
+ if v ~= nil then
+ v = v + 1
+ end
+ return k, v
+ end
+ return step, self, nil
+ end
+
+ setmetatable(original, {__pairs = custom_pairs })
+
+ -- Don't use is deeply as it could use pairs for check
+ local copy = table.deepcopy(original)
+ test:is(original.a, copy.a,
+ "checking that the first values is correctly copied")
+ test:is(original.b, copy.b,
+ "checking that the second values is correctly copied")
+end
+
os.exit(test:check() == true and 0 or 1)
--
2.23.0
More information about the Tarantool-patches
mailing list