Thanks! LGTM
Hi, Sergey! Thanks for the review! Fixed your comment and force-pushed the branch. On 11.03.25, Sergey Bronnikov wrote:Hi, Sergey, thanks for the patch! LGTM with a minor comment below. Sergey On 10.03.2025 17:51, Sergey Kaplun wrote:<snipped>+local bigstr = string.rep('x', ALLOC_LIMIT) +local __concat = function() + return 'concated's/concated/concatenated/Fixed, thanks: =================================================================== diff --git a/test/tarantool-tests/lj-1298-oom-on-concat-recording.test.lua b/test/tarantool-tests/lj-1298-oom-on-concat-recording.test.lua index 961df798..20d93c43 100644 --- a/test/tarantool-tests/lj-1298-oom-on-concat-recording.test.lua +++ b/test/tarantool-tests/lj-1298-oom-on-concat-recording.test.lua @@ -20,7 +20,7 @@ local ALLOC_LIMIT = 2048 local bigstr = string.rep('x', ALLOC_LIMIT) local __concat = function() - return 'concated' + return 'concatenated' end -- Need to use metamethod call in the concat recording. =================================================================== I've also checked that this is not ruined the testcase.<snipped>