Pass LuaDebug by reference

Author: Quahu

src/Laylua.Tests/Tests/Moon/LuaStateTests.cs

@@ -25,7 +25,7 @@ public void CancellationTokenLuaHook_CancelsExecutionWithError()
         // Act & Assert
         Assert.That(() => Lua.Execute("while true do end"), Throws.TypeOf<LuaException>()
             .With.InnerException.TypeOf<OperationCanceledException>()
-                .And.InnerException.Property(nameof(OperationCanceledException.CancellationToken)).EqualTo(cts.Token));
+            .And.InnerException.Property(nameof(OperationCanceledException.CancellationToken)).EqualTo(cts.Token));
     }
 
     [Test]
@@ -37,7 +37,7 @@ public void MaxInstructionCountLuaHook_CancelsExecutionWithError()
         // Act & Assert
         Assert.That(() => Lua.Execute("while true do end"), Throws.TypeOf<LuaException>()
             .With.InnerException.TypeOf<MaxInstructionCountReachedException>()
-                .And.InnerException.Property(nameof(MaxInstructionCountReachedException.InstructionCount)).EqualTo(100));
+            .And.InnerException.Property(nameof(MaxInstructionCountReachedException.InstructionCount)).EqualTo(100));
     }
 
     private sealed class InstructionCountLuaHook(int instructionCount) : LuaHook
@@ -48,7 +48,7 @@ private sealed class InstructionCountLuaHook(int instructionCount) : LuaHook
 
         public int TimesCalled { get; private set; }
 
-        protected override void Execute(LuaThread lua, LuaDebug debug)
+        protected override void Execute(LuaThread lua, ref LuaDebug debug)
         {
             TimesCalled++;
         }
@@ -75,6 +75,42 @@ public void CombinedCountLuaHooks_CallsBothAsExpected()
         Assert.That(fiveInstructionsHook.TimesCalled, Is.EqualTo(2));
     }
 
+    private sealed class FunctionNameGrabberLuaHook : LuaHook
+    {
+        protected override LuaEventMask EventMask => LuaEventMask.Call;
+
+        protected override int InstructionCount => 0;
+
+        public List<string> FunctionNames { get; } = [];
+
+        protected override void Execute(LuaThread lua, ref LuaDebug debug)
+        {
+            var functionName = debug.FunctionName.ToString();
+            if (!string.IsNullOrWhiteSpace(functionName))
+            {
+                FunctionNames.Add(functionName);
+            }
+        }
+    }
+
+    [Test]
+    public void FunctionNameGrabberLuaHook_GrabsFunctionNames()
+    {
+        // Arrange
+        var hook = new FunctionNameGrabberLuaHook();
+        Lua.State.Hook = hook;
+
+        Lua.SetGlobal("func1", static () => { });
+        Lua.SetGlobal("func2", static () => { });
+        Lua.Execute("function func3() end");
+
+        // Act
+        Lua.Execute("func1() func2() func3()");
+
+        // Assert
+        Assert.That(hook.FunctionNames, Is.EqualTo(new[] { "func1", "func2", "func3" }));
+    }
+
     [Test]
     public void GC_IsRunning_ReturnsTrue()
     {

src/Laylua/Moon/Hook/Default/CancellationTokenLuaHook.cs

@@ -27,7 +27,7 @@ public CancellationTokenLuaHook(CancellationToken cancellationToken)
     }
 
     /// <inheritdoc/>
-    protected internal override void Execute(LuaThread lua, LuaDebug debug)
+    protected internal override void Execute(LuaThread lua, ref LuaDebug debug)
     {
         if (!_cancellationToken.IsCancellationRequested)
         {

src/Laylua/Moon/Hook/Default/CombinedLuaHook.cs

@@ -46,7 +46,7 @@ public CombinedLuaHook(params Span<LuaHook> hooks)
     }
 
     /// <inheritdoc/>
-    protected internal override void Execute(LuaThread lua, LuaDebug debug)
+    protected internal override void Execute(LuaThread lua, ref LuaDebug debug)
     {
         for (var hookIndex = 0; hookIndex < _hooks.Length; hookIndex++)
         {
@@ -65,7 +65,7 @@ protected internal override void Execute(LuaThread lua, LuaDebug debug)
                 _instructionCounts[hookIndex] = 0;
             }
 
-            hook.Execute(lua, debug);
+            hook.Execute(lua, ref debug);
         }
     }
 

src/Laylua/Moon/Hook/Default/MaxInstructionCountLuaHook.cs

@@ -27,7 +27,7 @@ public MaxInstructionCountLuaHook(int maxInstructionCount)
     }
 
     /// <inheritdoc/>
-    protected internal override void Execute(LuaThread lua, LuaDebug debug)
+    protected internal override void Execute(LuaThread lua, ref LuaDebug debug)
     {
         lua_getinfo(lua.State.L, "Sn", debug.ActivationRecord);
 

src/Laylua/Moon/Hook/LuaHook.cs

@@ -28,7 +28,7 @@ public abstract class LuaHook
     /// </remarks>
     /// <param name="lua"> The Lua state. </param>
     /// <param name="debug"> The debug information. </param>
-    protected internal abstract void Execute(LuaThread lua, LuaDebug debug);
+    protected internal abstract void Execute(LuaThread lua, ref LuaDebug debug);
 
     /// <summary>
     ///     Combines multiple hooks into one.

src/Laylua/Moon/LuaState.cs

@@ -191,7 +191,7 @@ internal LuaHookFunction WrapHookFunction(LuaHook hook)
             using (var thread = LuaThread.FromExtraSpace(state))
             {
                 var debug = new LuaDebug(thread, ar);
-                hook.Execute(thread, debug);
+                hook.Execute(thread, ref debug);
             }
         };