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, 7 deletions

diff --git a/src/modules/m_hostchange.cpp b/src/modules/m_hostchange.cpp
index 7bf19313f..7fb14a1ba 100644
--- a/src/modules/m_hostchange.cpp
+++ b/src/modules/m_hostchange.cpp
@@ -32,18 +32,16 @@ class ModuleHostChange : public Module
 	std::string MySuffix;
 	 
  public:
-	ModuleHostChange()
+	ModuleHostChange(Server* Me)
+		: Module::Module(Me)
 	{
-		// We must create an instance of the Server class to work with
-		Srv = new Server;
+		Srv = Me;
                 Conf = new ConfigReader;
 		MySuffix = Conf->ReadValue("host","suffix",0);
 	}
 	
 	virtual ~ModuleHostChange()
 	{
-		// not really neccessary, but free it anyway
-		delete Srv;
 		delete Conf;
 	}
 
@@ -122,9 +120,9 @@ class ModuleHostChangeFactory : public ModuleFactory
 	{
 	}
 	
-	virtual Module * CreateModule()
+	virtual Module * CreateModule(Server* Me)
 	{
-		return new ModuleHostChange;
+		return new ModuleHostChange(Me);
 	}
 	
 };