Eliminate constant return value of the all events version of ModuleMa…

…nager::SetPriority()
inspircd · Feb 15, 2015 · 7bea242 · 7bea242
1 parent bfc1abb
commit 7bea242
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/include/modules.h b/include/modules.h
@@ -1124,7 +1124,7 @@ class CoreExport ModuleManager : public fakederef<ModuleManager>
 	 * SetPriority method for this, where you may specify other modules to
 	 * be prioritized against.
 	 */
-	bool SetPriority(Module* mod, Priority s);
+	void SetPriority(Module* mod, Priority s);
 
 	/** Attach an event to a module.
 	 * You may later detatch the event with ModuleManager::Detach().

diff --git a/src/modules.cpp b/src/modules.cpp
@@ -199,12 +199,10 @@ void ModuleManager::DetachAll(Module* mod)
 		Detach((Implementation)n, mod);
 }
 
-bool ModuleManager::SetPriority(Module* mod, Priority s)
+void ModuleManager::SetPriority(Module* mod, Priority s)
 {
 	for (size_t n = 0; n != I_END; ++n)
 		SetPriority(mod, (Implementation)n, s);
-
-	return true;
 }
 
 bool ModuleManager::SetPriority(Module* mod, Implementation i, Priority s, Module* which)