[Tarantool-patches] [PATCH luajit] Add missing LJ_MAX_JSLOTS check.

Sergey Kaplun skaplun at tarantool.org
Wed Apr 6 10:21:13 MSK 2022


Hi!

Thanks for the review!

On 30.03.22, sergos wrote:
> Hi! Thanks for updates!
> 
> LGTM with a very small nit fix you’ve missed in the test comment.

Fixed. Branch is force pushed.

===================================================================
diff --git a/test/tarantool-tests/fix-slot-check-for-mm-record.test.lua b/test/tarantool-tests/fix-slot-check-for-mm-record.test.lua
index e361830d..94a27d18 100644
--- a/test/tarantool-tests/fix-slot-check-for-mm-record.test.lua
+++ b/test/tarantool-tests/fix-slot-check-for-mm-record.test.lua
@@ -20,7 +20,7 @@ _G.a0 = a0

 -- Fixarg function with call to metamethod.
 local function a1()
-  -- This constant is not setted as an upvalue to simplify stack
+  -- This constant is not set as an upvalue to simplify stack
   -- slots counting. Just remember that it is 42.
   return a0 + 42
 end
===================================================================

> 
> Sergos
> 

-- 
Best regards,
Sergey Kaplun


More information about the Tarantool-patches mailing list