From 942fd2bcfd384a12c900999fe663202c87319a68 Mon Sep 17 00:00:00 2001 From: Sadie Powell Date: Tue, 6 Apr 2021 20:06:18 +0100 Subject: Switch simple iterator loops to use range-based for loops. --- src/modules/m_cloaking.cpp | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'src/modules/m_cloaking.cpp') diff --git a/src/modules/m_cloaking.cpp b/src/modules/m_cloaking.cpp index 79fa98faa..6462a1ef3 100644 --- a/src/modules/m_cloaking.cpp +++ b/src/modules/m_cloaking.cpp @@ -373,9 +373,8 @@ class ModuleCloaking : public Module return MOD_RES_PASSTHRU; // Check if they have a cloaked host but are not using it. - for (CloakList::const_iterator iter = cloaklist->begin(); iter != cloaklist->end(); ++iter) + for (const auto& cloak : *cloaklist) { - const std::string& cloak = *iter; if (cloak != user->GetDisplayedHost()) { const std::string cloakMask = user->nick + "!" + user->ident + "@" + cloak; @@ -528,8 +527,8 @@ class ModuleCloaking : public Module return; CloakList cloaklist; - for (std::vector::const_iterator iter = cloaks.begin(); iter != cloaks.end(); ++iter) - cloaklist.push_back(GenCloak(*iter, dest->client_sa, dest->GetIPString(), dest->GetRealHost())); + for (const auto& cloak : cloaks) + cloaklist.push_back(GenCloak(cloak, dest->client_sa, dest->GetIPString(), dest->GetRealHost())); cu.ext.Set(dest, cloaklist); } }; @@ -543,9 +542,9 @@ CmdResult CommandCloak::Handle(User* user, const Params& parameters) const char* ipaddr = irc::sockets::aptosa(parameters[0], 0, sa) ? parameters[0].c_str() : ""; unsigned int id = 0; - for (std::vector::const_iterator iter = mod->cloaks.begin(); iter != mod->cloaks.end(); ++iter) + for (const auto& info : mod->cloaks) { - const std::string cloak = mod->GenCloak(*iter, sa, ipaddr, parameters[0]); + const std::string cloak = mod->GenCloak(info, sa, ipaddr, parameters[0]); user->WriteNotice(InspIRCd::Format("*** Cloak #%u for %s is %s", ++id, parameters[0].c_str(), cloak.c_str())); } return CmdResult::SUCCESS; -- cgit v1.3.1-10-gc9f91