tarantool / luajit / 10939302872

70%

tarantool/master: 93%

Build Type

push

github

Committed by Buristan

Commit Message

Clear stack after print_jit_status() in CLI.

Suggested by Hydroque.

(cherry picked from commit 03cd5aa74)

This patch makes the Lua stack balanced after using the aforementioned
function, which is considered a good practice [1]. Be aware that the
only argument on the stack is dummy NULL from the `lua_cpcall()`, so it
can be easily cleared. This doesn't change the behaviour since all calls
of `print_jit_status()` are followed by `dotty()` with a call to
`loadline()`, which clears the stack anyway.

Sergey Kaplun:
* added the description for the patch

[1]: https://www.lua.org/manual/5.1/manual.html#lua_call

Part of tarantool/tarantool#10199

Reviewed-by: Sergey Bronnikov <sergeyb@tarantool.org>
Reviewed-by: Maxim Kokryashkin <m.kokryashkin@tarantool.org>
Signed-off-by: Sergey Kaplun <skaplun@tarantool.org>
(cherry picked from commit 84905c51f)

Run Details

5667 of 6007 branches covered (94.34%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

3 existing lines in 3 files now uncovered.

21600 of 23342 relevant lines covered (92.54%)

883077.6 hits per line