

<HTML><BODY><div>Hi, Igor!</div><div>Thanks for the patch!</div><div>Please consider my comments below.</div><div> </div><div>`lj-906-fix-err-mem.test.lua` has a `jit.off()`</div><div>call in the line 78, which should be wrapped in a pcall.</div><div> </div><div>This change should be moved to commit</div><div>«build: fix build with JIT disabled»</div><div>(a835fb0f778db6e9f0109a66ea1d2ac78fe682e4)</div><div>along with the change that I pointed out below.</div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"> <br>diff --git a/test/tarantool-tests/misclib-sysprof-lapi.test.lua b/test/tarantool-tests/misclib-sysprof-lapi.test.lua<br>index 8dc36592..dcde81b9 100644<br>--- a/test/tarantool-tests/misclib-sysprof-lapi.test.lua<br>+++ b/test/tarantool-tests/misclib-sysprof-lapi.test.lua<br>@@ -6,8 +6,10 @@ local test = tap.test("misc-sysprof-lapi"):skipcond({<br> <br> test:plan(19)<br> <br>-jit.off()<br>-jit.flush()<br>+-- XXX: Run JIT tuning functions in a safe frame to avoid errors<br>+-- thrown when LuaJIT is compiled with JIT engine disabled.<br>+pcall(jit.off)<br>+pcall(jit.flush)</blockquote><div>Same as for the change in `fix-err-mem`.</div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"> <br> local bufread = require("utils.bufread")<br> local symtab = require("utils.symtab")<br>@@ -125,5 +127,4 @@ check_mode("C", 100)<br> <br> os.remove(TMP_BINFILE)<br> <br>-jit.on()<br> os.exit(test:check() and 0 or 1)<br>--<br>2.30.2</blockquote><div><div>--<br>Best regards,</div><div>Maxim Kokryashkin</div></div><div> </div></BODY></HTML>