Merge pull request #38 from Akeit0/main

Fix: Implicit conversion from string to number, which is out of speci…

src/Lua/LuaTable.cs

@@ -247,9 +247,8 @@ public sealed class LuaTable
         // Move some of the elements of the hash part to a newly allocated array
         foreach (var kv in dictionary)
         {
-            if (kv.Key.TryRead<double>(out var d) && MathEx.IsInteger(d))
+            if (TryGetInteger(kv.Key, out var index))
             {
-                var index = (int)d;
                 if (index > prevLength && index <= newLength)
                 {
                     indexList.Add((index, kv.Value));
@@ -267,7 +266,7 @@ public sealed class LuaTable
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     static bool TryGetInteger(LuaValue value, out int integer)
     {
-        if (value.TryRead<double>(out var num) && MathEx.IsInteger(num))
+        if (value.TryReadNumber(out var num) && MathEx.IsInteger(num))
         {
             integer = (int)num;
             return true;

src/Lua/LuaValue.cs

@@ -203,7 +203,19 @@ public readonly struct LuaValue : IEquatable<LuaValue>
         result = default!;
         return false;
     }
-
+    
+    [MethodImpl(MethodImplOptions.AggressiveInlining)]
+    internal bool TryReadNumber(out double result)
+    {
+        if (type == LuaValueType.Number)
+        {
+            result = value;
+            return true;
+        }
+        result = default!;
+        return false;
+    }
+   
     public T Read<T>()
     {
         if (!TryRead<T>(out var result)) throw new InvalidOperationException($"Cannot convert LuaValueType.{Type} to {typeof(T).FullName}.");

src/Lua/Runtime/LuaVirtualMachine.cs

@@ -581,7 +581,7 @@ public static partial class LuaVirtualMachine
                             {
                                 compareResult = StringComparer.Ordinal.Compare(strB, strC) < 0;
                             }
-                            else if (vb.TryRead<double>(out var valueB) && vc.TryRead<double>(out var valueC))
+                            else if (vb.TryReadNumber(out var valueB) && vc.TryReadNumber(out var valueC))
                             {
                                 compareResult = valueB < valueC;
                             }
@@ -629,7 +629,7 @@ public static partial class LuaVirtualMachine
                             {
                                 compareResult = StringComparer.Ordinal.Compare(strB, strC) <= 0;
                             }
-                            else if (vb.TryRead<double>(out var valueB) && vc.TryRead<double>(out var valueC))
+                            else if (vb.TryReadNumber(out var valueB) && vc.TryReadNumber(out var valueC))
                             {
                                 compareResult = valueB <= valueC;
                             }