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_operjoin.cpp b/src/modules/m_operjoin.cpp
index d1d9ef3b0..b9adb890c 100644
--- a/src/modules/m_operjoin.cpp
+++ b/src/modules/m_operjoin.cpp
@@ -9,25 +9,24 @@ using namespace std;
 
 /* $ModDesc: Forces opers to join a specified channel on oper-up */
 
-Server *Srv;
-
 class ModuleOperjoin : public Module
 {
 	private:
 		std::string operChan;
 		ConfigReader* conf;
+		Server* Srv;
 
 	public:
-		ModuleOperjoin()
+		ModuleOperjoin(Server* Me)
+			: Module::Module(Me)
 		{
-			Srv = new Server;
+			Srv = Me;
 			conf = new ConfigReader;
 			operChan = conf->ReadValue("operjoin", "channel", 0);
 		}
 
 		virtual ~ModuleOperjoin()
 		{
-			delete Srv;
 			delete conf;
 		}
 
@@ -58,9 +57,9 @@ class ModuleOperjoinFactory : public ModuleFactory
         	{
 	        }
 
-	        virtual Module * CreateModule()
+	        virtual Module * CreateModule(Server* Me)
         	{
-                	return new ModuleOperjoin;
+                	return new ModuleOperjoin(Me);
 	        }
 };