More stack tests and cleanups

Author: kunitoki

Diff for: CHANGES.md

@@ -26,7 +26,7 @@
 * Added support for `std::byte` as stack value type.
 * Added support for `std::string_view` as stack value type.
 * Added support for `std::tuple` as stack value type.
-* Added support for `std::optional` as stack value type by using `LuaBridge/Optional.h`.
+* Added support for `std::optional` as stack value type.
 * Added support for `std::set` as stack value type by using `LuaBridge/Set.h`.
 * Added single header amalgamated distribution file, to simplify including in projects.
 * Added more asserts for functions and property names.
@@ -39,6 +39,7 @@
 * Changed all generic functions in `LuaRef` and `TableItem` to accept arguments by const reference instead of by copy.
 * Fixed issue when `LuaRef::cast<>` fails with exceptions enabled, popping from the now empty stack could trigger the panic handler twice.
 * Fixed unaligned access in user allocated member pointers in 64bit machines reported by ASAN.
+* Included testing against Luau VM
 * Bumped lua 5.2.x in unit tests from lua 5.2.0 to 5.2.4.
 * Bumped lua 5.4.x in unit tests from lua 5.4.1 to 5.4.3.
 * Run against lua 5.3.6 and 5.4.3 in unit tests.

Diff for: Source/CMakeLists.txt

@@ -5,7 +5,6 @@ set (LUABRIDGE_HEADERS
   ${CMAKE_CURRENT_SOURCE_DIR}/LuaBridge/List.h
   ${CMAKE_CURRENT_SOURCE_DIR}/LuaBridge/LuaBridge.h
   ${CMAKE_CURRENT_SOURCE_DIR}/LuaBridge/Map.h
-  ${CMAKE_CURRENT_SOURCE_DIR}/LuaBridge/Optional.h
   ${CMAKE_CURRENT_SOURCE_DIR}/LuaBridge/RefCountedObject.h
   ${CMAKE_CURRENT_SOURCE_DIR}/LuaBridge/Set.h
   ${CMAKE_CURRENT_SOURCE_DIR}/LuaBridge/UnorderedMap.h

Diff for: Source/LuaBridge/Optional.h

@@ -324,6 +324,38 @@ void* lua_newuserdata_aligned(lua_State* L, Args&&... args)
     return pointer;
 }
 
+/**
+ * @brief Checks if the value on the stack is a number type and can fit into the corresponding c++ numerical type..
+ */
+template <class T>
+inline bool is_integral_instance(lua_State* L, int index)
+{
+    if (lua_type(L, index) == LUA_TNUMBER)
+    {
+        const auto value = luaL_checkinteger(L, index);
+        return value >= std::numeric_limits<T>::min()
+            && value <= std::numeric_limits<T>::max();
+    }
+
+    return false;
+}
+
+/**
+ * @brief Checks if the value on the stack is a number type and can fit into the corresponding c++ numerical type..
+ */
+template <class T>
+inline bool is_floating_point_instance(lua_State* L, int index)
+{
+    if (lua_type(L, index) == LUA_TNUMBER)
+    {
+        const auto value = luaL_checknumber(L, index);
+        return value >= std::numeric_limits<T>::min()
+            && value <= std::numeric_limits<T>::max();
+    }
+
+    return false;
+}
+
 /**
  * @brief Helper to write a lua string error.
  */