[Tarantool-patches] [PATCH 1/3] lua: fix incorrect table.deepcopy __metatable handling

Igor Munkin imun at tarantool.org
Thu Feb 20 14:12:37 MSK 2020


Oleg,

Thanks for the patch. I totally agree with Vlad regarding the debug.*
usage and porting table.deepcopy to C. Propose also to discard this
patch.

On 13.02.20, olegrok at tarantool.org wrote:
> From: Oleg Babin <babinoleg at mail.ru>
> 
> Before this patch we got metatabe of original table
> using "getmetatable" function. It leads to the error
> if original table contained __metatable that could be
> e.g. a string or a number.
> To fix this problem getmetatable was replaced to debug.getmetatable.
> 
> Closes #4340
> ---
> Issue: https://github.com/tarantool/tarantool/issues/4340
>  src/lua/table.lua           |  2 +-
>  test/app-tap/table.test.lua | 20 +++++++++++++++++++-
>  2 files changed, 20 insertions(+), 2 deletions(-)
> 

<snipped>

> -- 
> 2.23.0
> 

-- 
Best regards,
IM


More information about the Tarantool-patches mailing list