Changed behaviour of module API to pass Server* to the constructor, rather than have to create one (makes more sense)

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@2252 e03df62e-2008-0410-955e-edbf42e46eb7

1 files changed, 5 insertions, 5 deletions

diff --git a/src/modules/m_helpop.cpp b/src/modules/m_helpop.cpp
index 6681d8f66..2a0a0e62b 100644
--- a/src/modules/m_helpop.cpp
+++ b/src/modules/m_helpop.cpp
@@ -158,9 +158,10 @@ class ModuleHelpop : public Module
 		std::string  h_file;
 
 	public:
-		ModuleHelpop()
+		ModuleHelpop(Server* Me)
+			: Module::Module(Me)
 		{
-			Srv  = new Server;
+			Srv  = Me;
 
 			ReadConfig();
 			if (!Srv->AddExtendedMode('h',MT_CLIENT,true,0,0))
@@ -223,7 +224,6 @@ class ModuleHelpop : public Module
 
 		virtual ~ModuleHelpop()
 		{
-			delete Srv;
 			delete conf;
 			delete helpop;
 		}
@@ -245,9 +245,9 @@ class ModuleHelpopFactory : public ModuleFactory
 	{
 	}
 	
-	virtual Module * CreateModule()
+	virtual Module * CreateModule(Server* Me)
 	{
-		return new ModuleHelpop;
+		return new ModuleHelpop(Me);
 	}
 	
 };