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

(limited to 'src/serializable.cpp')

diff --git a/src/serializable.cpp b/src/serializable.cpp
index 5834d21fa..f2c62c4dc 100644
--- a/src/serializable.cpp
+++ b/src/serializable.cpp
@@ -69,14 +69,12 @@ bool Extensible::Deserialize(Serializable::Data& data)
 	if (culled)
 		return false;
 
-	const Serializable::Data::EntryMap& entries = data.GetEntries();
-	for (Serializable::Data::EntryMap::const_iterator iter = entries.begin(); iter != entries.end(); ++iter)
+	for (const auto& [name, entry] : data.GetEntries())
 	{
-		const std::string& name = iter->first;
 		ExtensionItem* item = ServerInstance->Extensions.GetItem(name);
 		if (item)
 		{
-			item->FromInternal(this, iter->second);
+			item->FromInternal(this, entry);
 			continue;
 		}
 
@@ -95,12 +93,11 @@ bool Extensible::Serialize(Serializable::Data& data)
 		return false;
 	}
 
-	for (Extensible::ExtensibleStore::const_iterator iter = extensions.begin(); iter != extensions.end(); ++iter)
+	for (const auto& [ext, item] : extensions)
 	{
-		ExtensionItem* item = iter->first;
-		const std::string value = item->ToInternal(this, iter->second);
+		const std::string value = ext->ToInternal(this, item);
 		if (!value.empty())
-			data.Store(item->name, value);
+			data.Store(ext->name, item);
 	}
 	return true;
 }
-- 
cgit v1.3.1-10-gc9f91