diff options
author | Sadie Powell <sadie@witchery.services> | 2025-03-02 14:51:02 +0000 |
---|---|---|
committer | Sadie Powell <sadie@witchery.services> | 2025-03-02 15:27:47 +0000 |
commit | f9911dde529adf3dc03f4f14bbd70756ac2f665c (patch) | |
tree | 7c720e4f82fdb30b7d8a22fc0809f50bc862fae3 /modules/global/global.cpp | |
parent | a5e5eb5eb084e8343260ce7bc26ea86798f64fe1 (diff) |
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.
Diffstat (limited to 'modules/global/global.cpp')
-rw-r--r-- | modules/global/global.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
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()); |