feat!: add result to language module interface

Author: qubka

include/plugify/language_module.hpp

@@ -57,19 +57,20 @@ namespace plugify {
 
 		/**
 		 * @brief Shutdown the language module.
+		 * @return Result of the s.
 		 */
-		virtual void Shutdown() = 0;
+		virtual Result<void> Shutdown() = 0;
 
 		/**
 		 * @brief Handle actions to be performed on each frame.
 		 * @param deltaTime The time delta since the last update.
 		 */
-		virtual void OnUpdate(std::chrono::milliseconds deltaTime) = 0;
+		virtual Result<void> OnUpdate(std::chrono::milliseconds deltaTime) = 0;
 
 		/**
 		 * @brief
 		 */
-		// virtual void OnPluginInitialize(const Extension& plugin) = 0;
+		// virtual Result<void> OnPluginInitialize(const Extension& plugin) = 0;
 
 		/**
 		 * @brief Handle plugin load event.
@@ -82,26 +83,26 @@ namespace plugify {
 		 * @brief Handle plugin start event.
 		 * @param plugin Ref to the started plugin.
 		 */
-		virtual void OnPluginStart(const Extension& plugin) = 0;
+		virtual Result<void> OnPluginStart(const Extension& plugin) = 0;
 
 		/**
 		 * @brief Handle plugin update event.
 		 * @param plugin Ref to the started plugin.
 		 * @param deltaTime The time delta since the last update.
 		 */
-		virtual void OnPluginUpdate(const Extension& plugin, std::chrono::milliseconds deltaTime) = 0;
+		virtual Result<void> OnPluginUpdate(const Extension& plugin, std::chrono::milliseconds deltaTime) = 0;
 
 		/**
 		 * @brief Handle plugin end event.
 		 * @param plugin Ref to the ended plugin.
 		 */
-		virtual void OnPluginEnd(const Extension& plugin) = 0;
+		virtual Result<void> OnPluginEnd(const Extension& plugin) = 0;
 
 		/**
 		 * @brief Handle method export event.
 		 * @param plugin Ref to the plugin exporting a method.
 		 */
-		virtual void OnMethodExport(const Extension& plugin) = 0;
+		virtual Result<void> OnMethodExport(const Extension& plugin) = 0;
 
 		/**
 		 * @brief Determine if language module is build with debugging mode.

src/core/extension_loader.hpp

@@ -139,8 +139,7 @@ namespace plugify {
 				return {};
 			}
 			auto result = SafeCall<void>("OnUpdate", module.GetName(), [&] {
-				module.GetLanguageModule()->OnUpdate(deltaTime);
-				return Result<void>{};
+				return module.GetLanguageModule()->OnUpdate(deltaTime);
 			});
 			_extensionLifecycle->OnUpdate(module, deltaTime);
 			return result;
@@ -151,8 +150,7 @@ namespace plugify {
 
 			if (auto* languageModule = module.GetLanguageModule()) {
 				result = SafeCall<void>("Shutdown", module.GetName(), [&] {
-					languageModule->Shutdown();
-					return Result<void>{};
+					return languageModule->Shutdown();
 				});
 				module.SetLanguageModule(nullptr);
 			}
@@ -210,8 +208,7 @@ namespace plugify {
 				return {};
 			}
 			auto result = SafeCall<void>("OnPluginStart", plugin.GetName(), [&] {
-				plugin.GetLanguageModule()->OnPluginStart(plugin);
-				return Result<void>{};
+				return plugin.GetLanguageModule()->OnPluginStart(plugin);
 			});
 			_extensionLifecycle->OnStart(plugin);
 			return result;
@@ -223,8 +220,7 @@ namespace plugify {
 				return {};
 			}
 			auto result = SafeCall<void>("OnPluginEnd", plugin.GetName(), [&] {
-				plugin.GetLanguageModule()->OnPluginEnd(plugin);
-				return Result<void>{};
+				return plugin.GetLanguageModule()->OnPluginEnd(plugin);
 			});
 			_extensionLifecycle->OnEnd(plugin);
 			return result;
@@ -236,8 +232,7 @@ namespace plugify {
 				return {};
 			}
 			auto result = SafeCall<void>("OnPluginUpdate", plugin.GetName(), [&] {
-				plugin.GetLanguageModule()->OnPluginUpdate(plugin, deltaTime);
-				return Result<void>{};
+				return plugin.GetLanguageModule()->OnPluginUpdate(plugin, deltaTime);
 			});
 			_extensionLifecycle->OnUpdate(plugin, deltaTime);
 			return result;
@@ -260,8 +255,7 @@ namespace plugify {
 				return {};
 			}
 			auto result = SafeCall<void>("OnMethodExport", module.GetName(), [&] {
-				module.GetLanguageModule()->OnMethodExport(plugin);
-				return Result<void>{};
+				return module.GetLanguageModule()->OnMethodExport(plugin);
 			});
 			//_lifecycle->OnExport(module, plugin);
 			return result;

src/core/manager.cpp

@@ -234,7 +234,6 @@ struct Manager::Impl {
 						break;
 					}
 				}
-
 				if (!result) {
 					logger->Log(result.error(), Severity::Error);
 				}