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_sqloper.cpp | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

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

diff --git a/src/modules/m_sqloper.cpp b/src/modules/m_sqloper.cpp
index 7ff0b1759..76e50cedc 100644
--- a/src/modules/m_sqloper.cpp
+++ b/src/modules/m_sqloper.cpp
@@ -56,10 +56,8 @@ class OperQuery : public SQL::Query
 		ServerConfig::OperIndex& oper_blocks = ServerInstance->Config->oper_blocks;
 
 		// Remove our previous blocks from oper_blocks for a clean update
-		for (std::vector<std::string>::const_iterator i = my_blocks.begin(); i != my_blocks.end(); ++i)
-		{
-			oper_blocks.erase(*i);
-		}
+		for (const auto& block : my_blocks)
+			oper_blocks.erase(block);
 		my_blocks.clear();
 
 		SQL::Row row;
@@ -198,10 +196,8 @@ public:
 	~ModuleSQLOper() override
 	{
 		// Remove all oper blocks that were from the DB
-		for (std::vector<std::string>::const_iterator i = my_blocks.begin(); i != my_blocks.end(); ++i)
-		{
-			ServerInstance->Config->oper_blocks.erase(*i);
-		}
+		for (const auto& block : my_blocks)
+			ServerInstance->Config->oper_blocks.erase(block);
 	}
 
 	ModResult OnPreCommand(std::string& command, CommandBase::Params& parameters, LocalUser* user, bool validated) override
-- 
cgit v1.3.1-10-gc9f91