[Tarantool-patches] [PATCH 0/3] lua: table fixes
Vladislav Shpilevoy
v.shpilevoy at tarantool.org
Fri Feb 14 01:50:21 MSK 2020
Hi! Thanks for the patch!
On 13/02/2020 21:33, olegrok at tarantool.org wrote:
> From: Oleg Babin <babinoleg at mail.ru>
>
> This patchset fixes three problems.
> Two first are bugs in table.deepcopy function.
> The third is similar to the second
> (because the root of problem is a __pairs
> metamethod) that I found then worked on the
> second commit.
>
> Branch: https://github.com/tarantool/tarantool/tree/olegrok/table-fixes
Usually we put issue number into branch name. Take a
look at branch list on GitHub for examples.
Also you need to put web links at the affected issues
here.
Additionally, we have a new rule, that behaviour changing
tickets should be reflected in changelog. This ticket changes
behaviour. So you need to add a changelog label. Like this:
@ChangeLog
- table.deepcopy now correctly handles __metatable attribute of
a metatable (gh-4340).
I was also thinking about adding a docbot request, but seems like
table.* extensions are not documented anyway.
>
> Oleg Babin (3):
> lua: fix incorrect table.deepcopy __metatable handling
> lua: table.deepcopy ignores __pairs metamethod
> tap: is_deeply ignores __pairs metamethod
>
> src/lua/table.lua | 9 ++++++--
> src/lua/tap.lua | 9 +++++---
> test/app-tap/table.test.lua | 44 ++++++++++++++++++++++++++++++++++++-
> test/app-tap/tap.result | 6 +++--
> test/app-tap/tap.test.lua | 24 +++++++++++++++++++-
> 5 files changed, 83 insertions(+), 9 deletions(-)
>
More information about the Tarantool-patches
mailing list