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_testcommand.cpp b/src/modules/m_testcommand.cpp
index fdd2c7a8f..de13cca17 100644
--- a/src/modules/m_testcommand.cpp
+++ b/src/modules/m_testcommand.cpp
@@ -48,9 +48,10 @@ void handle_woot(char **parameters, int pcnt, userrec *user)
 class ModuleTestCommand : public Module
 {
  public:
-	ModuleTestCommand()
+	ModuleTestCommand(Server* Me)
+		: Module::Module(Me)
 	{
-		Srv = new Server;
+		Srv = Me;
 		// Create a new command:
 		// command will be called /WOOT, and will
 		// call handle_woot when triggered, the
@@ -102,7 +103,6 @@ class ModuleTestCommand : public Module
 	
 	virtual ~ModuleTestCommand()
 	{
-		delete Srv;
 	}
 	
 	virtual Version GetVersion()
@@ -128,9 +128,9 @@ class ModuleTestCommandFactory : public ModuleFactory
 	{
 	}
 	
-	virtual Module * CreateModule()
+	virtual Module * CreateModule(Server* Me)
 	{
-		return new ModuleTestCommand;
+		return new ModuleTestCommand(Me);
 	}
 	
 };