<!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>Hi, Sergey,</p>
<p>LGTM<br>
</p>
<div class="moz-cite-prefix">On 13.03.2025 13:28, Sergey Kaplun
wrote:<br>
</div>
<blockquote type="cite" cite="mid:Z9KzTGum2CSsm6Ke@root">
<pre wrap="" class="moz-quote-pre">Hi, Sergey!
Thanks for the review!
Fixed your comment and force-pushed the branch.
On 12.03.25, Sergey Bronnikov wrote:
</pre>
<blockquote type="cite">
<pre wrap="" class="moz-quote-pre">Hi, Sergey,
thanks for the patch! LGTM wit ha minor comment.
Sergey
On 10.03.2025 17:51, Sergey Kaplun wrote:
</pre>
<blockquote type="cite">
<pre wrap="" class="moz-quote-pre">From: Mike Pall <mike>
Reported by Sergey Kaplun.
(cherry picked from commit eee16efa77b542e99c8e546a3d52fc023925c7bc)
This commit is a follow-up to the previous one. It fixes the case when
the `topslot` is adjusting for simple concatenation results. This patch
adds the update of the corresponding Lua stack slots to be restored.
This fixes back the <lj-839-concat-recording.test.lua> test.
Sergey Kaplun:
* added the description and the test for the problem
</pre>
</blockquote>
<pre wrap="" class="moz-quote-pre">
It is partially true actually :)
s/and the test//
</pre>
</blockquote>
<pre wrap="" class="moz-quote-pre">
Yes, the force of the habit (and copy-paste ;))
</pre>
<blockquote type="cite">
<pre wrap="" class="moz-quote-pre">
</pre>
<blockquote type="cite">
<pre wrap="" class="moz-quote-pre">
Part of tarantool/tarantool#11055
---
</pre>
</blockquote>
</blockquote>
<pre wrap="" class="moz-quote-pre">
<snipped>
</pre>
</blockquote>
</body>
<lt-container></lt-container>
</html>