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

diff --git a/src/modules/m_cloaking.cpp b/src/modules/m_cloaking.cpp
index 44a03a39b..d7f4082ee 100644
--- a/src/modules/m_cloaking.cpp
+++ b/src/modules/m_cloaking.cpp
@@ -268,10 +268,11 @@ class ModuleCloaking : public Module
 	}
 	 
  public:
-	ModuleCloaking()
+	ModuleCloaking(Server* Me)
+		: Module::Module(Me)
 	{
-		// We must create an instance of the Server class to work with
-		Srv = new Server;
+		// We must take a copy of the Server class to work with
+		Srv = Me;
 		
 		// we must create a new mode. Set the parameters so the
 		// mode doesn't require oper, and is a client usermode
@@ -291,8 +292,6 @@ class ModuleCloaking : public Module
 	
 	virtual ~ModuleCloaking()
 	{
-		// not really neccessary, but free it anyway
-		delete Srv;
 	}
 	
 	virtual Version GetVersion()
@@ -401,9 +400,9 @@ class ModuleCloakingFactory : public ModuleFactory
 	{
 	}
 	
-	virtual Module * CreateModule()
+	virtual Module * CreateModule(Server* Me)
 	{
-		return new ModuleCloaking;
+		return new ModuleCloaking(Me);
 	}
 	
 };