fix: Accidentally removed early exit when UnregisterHook is called from within a pre-callback

Author: UE4SS

UE4SS/src/Mod/LuaMod.cpp

@@ -247,7 +247,8 @@ namespace RC
         // Fetch the data corresponding to this UFunction
         auto& lua_data = *static_cast<LuaUnrealScriptFunctionData*>(custom_data);
 
-        auto remove_if_scheduled = [&] {
+        // Returns true if a hooks were removed.
+        auto remove_if_scheduled = [&] -> bool {
             if (lua_data.scheduled_for_removal)
             {
                 const auto function_name_no_prefix = get_function_name_without_prefix(lua_data.unreal_function->GetFullName());
@@ -269,12 +270,19 @@ namespace RC
                     return elem.get() == &lua_data;
                 });
 
-                return;
+                return true;
+            }
+            else
+            {
+                return false;
             }
         };
 
         // Removes pre & post-hook callbacks if UnregisterHook was called in the pre-callback.
-        remove_if_scheduled();
+        if (remove_if_scheduled())
+        {
+            return;
+        }
 
         // This is a promise that we're in the game thread, used by other functions to ensure that we don't execute when unsafe
         set_is_in_game_thread(lua_data.lua, true);