Return references instead of pointers from the config system.

We used to return NULL from these methods but now we return an empty
block so this can never actually be null now.

2 files changed, 7 insertions, 7 deletions

diff --git a/modules/global/gl_queue.cpp b/modules/global/gl_queue.cpp
index 583a46f6a..8a0f5c9f5 100644
--- a/modules/global/gl_queue.cpp
+++ b/modules/global/gl_queue.cpp
@@ -69,7 +69,7 @@ private:
 			return;
 		}
 
-		auto maxqueue = Config->GetModule(this->module)->Get<size_t>("maxqueue", "10");
+		auto maxqueue = Config->GetModule(this->module).Get<size_t>("maxqueue", "10");
 		if (global->CountQueue(source.nc) >= maxqueue)
 		{
 			source.Reply(_("You can not queue any more messages."));
diff --git a/modules/global/global.cpp b/modules/global/global.cpp
index 44b17326a..784088b97 100644
--- a/modules/global/global.cpp
+++ b/modules/global/global.cpp
@@ -61,9 +61,9 @@ public:
 		return q->size();
 	}
 
-	void OnReload(Configuration::Conf *conf) override
+	void OnReload(Configuration::Conf &conf) override
 	{
-		const auto glnick = conf->GetModule(this)->Get<const Anope::string>("client");
+		const auto glnick = conf.GetModule(this).Get<const Anope::string>("client");
 		if (glnick.empty())
 			throw ConfigException(Module::name + ": <client> must be defined");
 
@@ -76,21 +76,21 @@ public:
 
 	void OnRestart() override
 	{
-		const auto msg = Config->GetModule(this)->Get<const Anope::string>("globaloncycledown");
+		const auto msg = Config->GetModule(this).Get<const Anope::string>("globaloncycledown");
 		if (!msg.empty())
 			this->SendSingle(msg, nullptr, nullptr, nullptr);
 	}
 
 	void OnShutdown() override
 	{
-		const auto msg = Config->GetModule(this)->Get<const Anope::string>("globaloncycledown");
+		const auto msg = Config->GetModule(this).Get<const Anope::string>("globaloncycledown");
 		if (!msg.empty())
 			this->SendSingle(msg, nullptr, nullptr, nullptr);
 	}
 
 	void OnNewServer(Server *s) override
 	{
-		const auto msg = Config->GetModule(this)->Get<const Anope::string>("globaloncycleup");
+		const auto msg = Config->GetModule(this).Get<const Anope::string>("globaloncycleup");
 		if (!msg.empty() && !Me->IsSynced())
 			s->Notice(global, msg);
 	}
@@ -138,7 +138,7 @@ public:
 			return false;
 
 		Anope::string line;
-		if (source && !Config->GetModule(this)->Get<bool>("anonymousglobal"))
+		if (source && !Config->GetModule(this).Get<bool>("anonymousglobal"))
 		{
 			// A source is available and they're not anonymous.
 			line = Anope::printf("[%s] %s", source->GetNick().c_str(), message.c_str());