From 5ca01dc589b91dae95d4429ade21154bd6114455 Mon Sep 17 00:00:00 2001
From: Michael Fabian 'Xaymar' Dirks <info@xaymar.com>
Date: Sun, 14 May 2023 08:51:35 +0200
Subject: [PATCH] code: Migrate filter::sdf_effects to new dynamic loader

---
 source/filters/filter-sdf-effects.cpp | 37 ++++++++++++++-------------
 source/filters/filter-sdf-effects.hpp |  2 +-
 source/plugin.cpp                     |  9 -------
 3 files changed, 20 insertions(+), 28 deletions(-)

diff --git a/source/filters/filter-sdf-effects.cpp b/source/filters/filter-sdf-effects.cpp
index 1546abd..0acf779 100644
--- a/source/filters/filter-sdf-effects.cpp
+++ b/source/filters/filter-sdf-effects.cpp
@@ -685,26 +685,27 @@ bool sdf_effects_factory::on_manual_open(obs_properties_t* props, obs_property_t
 }
 #endif
 
-std::shared_ptr<sdf_effects_factory> _filter_sdf_effects_factory_instance = nullptr;
-
-void streamfx::filter::sdf_effects::sdf_effects_factory::initialize()
+std::shared_ptr<sdf_effects_factory> sdf_effects_factory::instance()
 {
-	try {
-		if (!_filter_sdf_effects_factory_instance)
-			_filter_sdf_effects_factory_instance = std::make_shared<sdf_effects_factory>();
-	} catch (const std::exception& ex) {
-		D_LOG_ERROR("Failed to initialize due to error: %s", ex.what());
-	} catch (...) {
-		D_LOG_ERROR("Failed to initialize due to unknown error.", "");
+	static std::weak_ptr<sdf_effects_factory> winst;
+	static std::mutex                          mtx;
+
+	std::unique_lock<decltype(mtx)> lock(mtx);
+	auto                            instance = winst.lock();
+	if (!instance) {
+		instance = std::shared_ptr<sdf_effects_factory>(new sdf_effects_factory());
+		winst    = instance;
 	}
+	return instance;
 }
 
-void streamfx::filter::sdf_effects::sdf_effects_factory::finalize()
-{
-	_filter_sdf_effects_factory_instance.reset();
-}
+static std::shared_ptr<sdf_effects_factory> loader_instance;
 
-std::shared_ptr<sdf_effects_factory> streamfx::filter::sdf_effects::sdf_effects_factory::get()
-{
-	return _filter_sdf_effects_factory_instance;
-}
+static auto loader = streamfx::loader(
+	[]() { // Initalizer
+		loader_instance = sdf_effects_factory::instance();
+	},
+	[]() { // Finalizer
+		loader_instance.reset();
+	},
+	streamfx::loader_priority::NORMAL);
diff --git a/source/filters/filter-sdf-effects.hpp b/source/filters/filter-sdf-effects.hpp
index 3d2ebd3..ff14077 100644
--- a/source/filters/filter-sdf-effects.hpp
+++ b/source/filters/filter-sdf-effects.hpp
@@ -100,7 +100,7 @@ namespace streamfx::filter::sdf_effects {
 
 		static void finalize();
 
-		static std::shared_ptr<sdf_effects_factory> get();
+		static std::shared_ptr<sdf_effects_factory> instance();
 	};
 
 } // namespace streamfx::filter::sdf_effects
diff --git a/source/plugin.cpp b/source/plugin.cpp
index 259bbb3..6780b1e 100644
--- a/source/plugin.cpp
+++ b/source/plugin.cpp
@@ -25,9 +25,6 @@
 #ifdef ENABLE_FILTER_DISPLACEMENT
 #include "filters/filter-displacement.hpp"
 #endif
-#ifdef ENABLE_FILTER_SDF_EFFECTS
-#include "filters/filter-sdf-effects.hpp"
-#endif
 #ifdef ENABLE_FILTER_UPSCALING
 #include "filters/filter-upscaling.hpp"
 #endif
@@ -147,9 +144,6 @@ MODULE_EXPORT bool obs_module_load(void)
 #ifdef ENABLE_FILTER_DISPLACEMENT
 			streamfx::filter::displacement::displacement_factory::initialize();
 #endif
-#ifdef ENABLE_FILTER_SDF_EFFECTS
-			streamfx::filter::sdf_effects::sdf_effects_factory::initialize();
-#endif
 #ifdef ENABLE_FILTER_UPSCALING
 			streamfx::filter::upscaling::upscaling_factory::initialize();
 #endif
@@ -199,9 +193,6 @@ MODULE_EXPORT void obs_module_unload(void)
 #ifdef ENABLE_FILTER_DISPLACEMENT
 			streamfx::filter::displacement::displacement_factory::finalize();
 #endif
-#ifdef ENABLE_FILTER_SDF_EFFECTS
-			streamfx::filter::sdf_effects::sdf_effects_factory::finalize();
-#endif
 #ifdef ENABLE_FILTER_UPSCALING
 			streamfx::filter::upscaling::upscaling_factory::finalize();
 #endif