diff options
| author | Sadie Powell <sadie@witchery.services> | 2023-10-10 21:14:50 +0100 |
|---|---|---|
| committer | Sadie Powell <sadie@witchery.services> | 2023-10-11 15:51:52 +0100 |
| commit | a3241065c55fd2a69e8793b89a5d0b1a957b3fd0 (patch) | |
| tree | 82f80ce2f3bbbdc1c1ef05fe611093cf0b34eab6 /modules/m_dnsbl.cpp | |
| parent | dc371aad6d059dbf7f30f6878c680532bedd4146 (diff) | |
Start migrating to range-based for loops.
Diffstat (limited to 'modules/m_dnsbl.cpp')
| -rw-r--r-- | modules/m_dnsbl.cpp | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/modules/m_dnsbl.cpp b/modules/m_dnsbl.cpp index ca17dc287..59cd1f3fc 100644 --- a/modules/m_dnsbl.cpp +++ b/modules/m_dnsbl.cpp @@ -30,11 +30,11 @@ struct Blacklist Anope::string reason; std::vector<Reply> replies; - Reply *Find(int code) + const Reply *Find(int code) { - for (unsigned int i = 0; i < replies.size(); ++i) - if (replies[i].code == code) - return &replies[i]; + for (const auto &reply : replies) + if (reply.code == code) + return &reply; return NULL; } }; @@ -62,7 +62,7 @@ class DNSBLResolver : public Request sresult.pton(AF_INET, ans_record.rdata); int result = sresult.sa4.sin_addr.s_addr >> 24; - Blacklist::Reply *reply = blacklist.Find(result); + const Blacklist::Reply *reply = blacklist.Find(result); if (!blacklist.replies.empty() && !reply) return; @@ -173,10 +173,8 @@ class ModuleDNSBL : public Module Anope::string reverse = user->ip.reverse(); - for (unsigned i = 0; i < this->blacklists.size(); ++i) + for (const auto &b : this->blacklists) { - const Blacklist &b = this->blacklists[i]; - Anope::string dnsbl_host = reverse + "." + b.name; DNSBLResolver *res = NULL; try |