[Tarantool-patches] [PATCH] lua: do not use internal api in public headers

Igor Munkin imun at tarantool.org
Mon Jul 20 14:34:14 MSK 2020


Timur,

Thanks for your changes! It does solve your problem, but I propose to
drop this patch in favour of mine[1].

On 20.07.20, Timur Safin wrote:
> src/lua/utils.[ch] is part of public api which will
> be integrated into module.h at the build step. It
> should not refer to the internal LuaJIT API like
> lj_state_growstack1(), otherwise it has not chance
> to be linked in while building external binary module.
> 
> Relates to vshard-cluster-api#5

Side note: I doubt this line refers the issue the right way.

> ---
> 
> Branch: https://github.com/tarantool/tarantool/tree/tsafin/gh-vshard-cluster-api-5-no-private-api
> Issue: https://github.com/tarantool/vshard-cluster-api/issues/5
> 
>  src/lua/utils.c | 13 +++++++++++++
>  src/lua/utils.h | 14 ++------------
>  2 files changed, 15 insertions(+), 12 deletions(-)
> 

<snipped>

[1]: https://lists.tarantool.org/pipermail/tarantool-patches/2020-July/018745.html

-- 
Best regards,
IM


More information about the Tarantool-patches mailing list