<!DOCTYPE html>
<html data-lt-installed="true">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body style="padding-bottom: 1px;">
<p>Sergey,<br>
</p>
<div class="moz-cite-prefix">On 22.08.2024 17:36, Sergey Kaplun
wrote:<br>
</div>
<blockquote type="cite" cite="mid:ZsdNAgLi4LKI8Xyd@root">
<pre class="moz-quote-pre" wrap="">Hi, Sergey!
Thanks for the review!
Please consider my answer below.
On 22.08.24, Sergey Bronnikov wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">Sergey,
On 21.08.2024 11:58, Sergey Kaplun wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">This patch moves the aforementioned test from the <misc> to the <lang/>
directory (with slightly renaming to be consistent with other names),
includes it in <index>, and names the subtest.
Part of tarantool/tarantool#9398
---
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">thanks for the patch! LGTM with comment below.
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap=""> test/LuaJIT-tests/lang/index | 1 +
test/LuaJIT-tests/lang/stackov_c.lua | 12 ++++++++++++
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">May be "stackoverflow_coro.lua"?
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
I suppose the main idea is the possible stack overflow in the C function
(more accurate built-in function) `unpack`. I prefer to leave the old
name to avoid confusion.
</pre>
<br>
</blockquote>
<p>LGTM as I said before.<br>
</p>
<p><br>
</p>
<snipped><br>
</body>
<lt-container></lt-container>
</html>