<!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>thanks for review!</p>
<p>Updated and force-pushed.</p>
<p>Sergey<br>
</p>
<div class="moz-cite-prefix">On 24.02.2025 12:45, Sergey Kaplun via
Tarantool-patches wrote:<br>
</div>
<blockquote type="cite" cite="mid:Z7w_xHsEnfoo_Q_A@root">
<pre wrap="" class="moz-quote-pre">Hi, Sergey!
Thanks for the fixes!
LGTM, except a few nits regarding the commit message.
On 20.02.25, Sergey Bronnikov wrote:
</pre>
<blockquote type="cite">
<pre wrap="" class="moz-quote-pre">The patch introduce an environment variable LUAJIT_DISABLE_MEMPROF
</pre>
</blockquote>
<pre wrap="" class="moz-quote-pre">
Typo: s/introduce/introduces/</pre>
</blockquote>
Fixed.<br>
<blockquote type="cite" cite="mid:Z7w_xHsEnfoo_Q_A@root">
<pre wrap="" class="moz-quote-pre">
</pre>
<blockquote type="cite">
<pre wrap="" class="moz-quote-pre">needed by tests in suite `tarantool-tests` for skip testcases when
</pre>
</blockquote>
<pre wrap="" class="moz-quote-pre">
Typo: s/suite/the suite/
Typo: s/for skip/to skip/
</pre>
</blockquote>
Fixed.<br>
<blockquote type="cite" cite="mid:Z7w_xHsEnfoo_Q_A@root">
<pre wrap="" class="moz-quote-pre">
</pre>
<blockquote type="cite">
<pre wrap="" class="moz-quote-pre">memprof is disabled in build.
</pre>
</blockquote>
<pre wrap="" class="moz-quote-pre">
Typo: s/build/the build/</pre>
</blockquote>
Fixed.<br>
<blockquote type="cite" cite="mid:Z7w_xHsEnfoo_Q_A@root">
<pre wrap="" class="moz-quote-pre">
</pre>
<blockquote type="cite">
<pre wrap="" class="moz-quote-pre">
Needed for the following commit.
---
</pre>
</blockquote>
<pre wrap="" class="moz-quote-pre">
<snipped>
</pre>
</blockquote>
</body>
<lt-container></lt-container>
</html>