[Tarantool-patches] [PATCH luajit v3 1/4] utils: remove unnecessary insertion in AVL-tree

Igor Munkin imun at tarantool.org
Tue Aug 15 21:50:45 MSK 2023


Max,

Thanks for the patch! The change is quite clear, but so-called 'gen' was
introduced on purpose AFAIR. Hence, I would like to ask you to describe
this change or clarify my gen misunderstanding.

On 31.07.23, Maxim Kokryashkin wrote:
> This patch fixes a bug in the AVL-tree implementation,
> which produced unnecessary inserts of values into nodes,
> instead of replacement.
> 
> Needed for tarantool/tarantool#8700
> ---
>  .../tarantool-tests/gh-5813-resolving-of-c-symbols.test.lua | 6 ++----
>  tools/utils/avl.lua                                         | 4 ++--
>  tools/utils/symtab.lua                                      | 2 +-
>  3 files changed, 5 insertions(+), 7 deletions(-)
> 

<snipped>

> -- 
> 2.41.0
> 

-- 
Best regards,
IM


More information about the Tarantool-patches mailing list