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_pbkdf2.cpp | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

(limited to 'src/modules/m_pbkdf2.cpp')

diff --git a/src/modules/m_pbkdf2.cpp b/src/modules/m_pbkdf2.cpp
index ddaa7a8e3..7bad7c3fa 100644
--- a/src/modules/m_pbkdf2.cpp
+++ b/src/modules/m_pbkdf2.cpp
@@ -167,9 +167,8 @@ class ModulePBKDF2 : public Module
 
 	void ConfigureProviders()
 	{
-		for (std::vector<PBKDF2Provider*>::iterator i = providers.begin(); i != providers.end(); ++i)
+		for (const auto& pi : providers)
 		{
-			PBKDF2Provider* pi = *i;
 			ProviderConfig config = GetConfigForProvider(pi->name);
 			pi->iterations = config.iterations;
 			pi->dkey_length = config.dkey_length;
@@ -215,9 +214,8 @@ class ModulePBKDF2 : public Module
 	void init() override
 	{
 		// Let ourself know about any existing services.
-		const ModuleManager::DataProviderMap& dataproviders = ServerInstance->Modules.DataProviders;
-		for (ModuleManager::DataProviderMap::const_iterator it = dataproviders.begin(); it != dataproviders.end(); ++it)
-			OnServiceAdd(*it->second);
+		for (const auto& [_, service] : ServerInstance->Modules.DataProviders)
+			OnServiceAdd(*service);
 	}
 
 	void OnServiceAdd(ServiceProvider& provider) override
-- 
cgit v1.3.1-10-gc9f91