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.

1 files changed, 8 insertions, 8 deletions

diff --git a/modules/redis.cpp b/modules/redis.cpp
index 25ecf0678..f50b4c67f 100644
--- a/modules/redis.cpp
+++ b/modules/redis.cpp
@@ -535,19 +535,19 @@ public:
 		}
 	}
 
-	void OnReload(Configuration::Conf *conf) override
+	void OnReload(Configuration::Conf &conf) override
 	{
-		Configuration::Block *block = conf->GetModule(this);
+		Configuration::Block &block = conf.GetModule(this);
 		std::vector<Anope::string> new_services;
 
-		for (int i = 0; i < block->CountBlock("redis"); ++i)
+		for (int i = 0; i < block.CountBlock("redis"); ++i)
 		{
-			Configuration::Block *redis = block->GetBlock("redis", i);
+			Configuration::Block &redis = block.GetBlock("redis", i);
 
-			const Anope::string &n = redis->Get<const Anope::string>("name"),
-						&ip = redis->Get<const Anope::string>("ip");
-			int port = redis->Get<int>("port");
-			unsigned db = redis->Get<unsigned>("db");
+			const Anope::string &n = redis.Get<const Anope::string>("name"),
+						&ip = redis.Get<const Anope::string>("ip");
+			int port = redis.Get<int>("port");
+			unsigned db = redis.Get<unsigned>("db");
 
 			delete services[n];
 			services[n] = new MyRedisService(this, n, ip, port, db);