From 2c941cd8b3cb1774f4982b86735e276597a91750 Mon Sep 17 00:00:00 2001
From: glx22 <glx@openttd.org>
Date: Mon, 7 Jun 2021 23:24:37 +0200
Subject: Codechange: Use ChunkHandlers sub-classes

---
 src/saveload/cheat_sl.cpp | 71 +++++++++++++++++++++++------------------------
 1 file changed, 35 insertions(+), 36 deletions(-)

(limited to 'src/saveload/cheat_sl.cpp')

diff --git a/src/saveload/cheat_sl.cpp b/src/saveload/cheat_sl.cpp
index f68b5a5a5..939a734aa 100644
--- a/src/saveload/cheat_sl.cpp
+++ b/src/saveload/cheat_sl.cpp
@@ -35,47 +35,46 @@ static const SaveLoad _cheats_desc[] = {
 	SLE_VAR(Cheats, edit_max_hl.value, SLE_BOOL),
 };
 
-/**
- * Save the cheat values.
- */
-static void Save_CHTS()
-{
-	SlTableHeader(_cheats_desc);
 
-	SlSetArrayIndex(0);
-	SlObject(&_cheats, _cheats_desc);
-}
+struct CHTSChunkHandler : ChunkHandler {
+	CHTSChunkHandler() : ChunkHandler('CHTS', CH_TABLE) {}
 
-/**
- * Load the cheat values.
- */
-static void Load_CHTS()
-{
-	std::vector<SaveLoad> slt = SlCompatTableHeader(_cheats_desc, _cheats_sl_compat);
-
-	if (IsSavegameVersionBefore(SLV_TABLE_CHUNKS)) {
-		size_t count = SlGetFieldLength();
-		std::vector<SaveLoad> oslt;
-
-		/* Cheats were added over the years without a savegame bump. They are
-		 * stored as 2 SLE_BOOLs per entry. "count" indicates how many SLE_BOOLs
-		 * are stored for this savegame. So read only "count" SLE_BOOLs (and in
-		 * result "count / 2" cheats). */
-		for (auto &sld : slt) {
-			count--;
-			oslt.push_back(sld);
-
-			if (count == 0) break;
-		}
-		slt = oslt;
+	void Save() const override
+	{
+		SlTableHeader(_cheats_desc);
+
+		SlSetArrayIndex(0);
+		SlObject(&_cheats, _cheats_desc);
 	}
 
-	if (!IsSavegameVersionBefore(SLV_RIFF_TO_ARRAY) && SlIterateArray() == -1) return;
-	SlObject(&_cheats, slt);
-	if (!IsSavegameVersionBefore(SLV_RIFF_TO_ARRAY) && SlIterateArray() != -1) SlErrorCorrupt("Too many CHTS entries");
-}
+	void Load() const override
+	{
+		std::vector<SaveLoad> slt = SlCompatTableHeader(_cheats_desc, _cheats_sl_compat);
+
+		if (IsSavegameVersionBefore(SLV_TABLE_CHUNKS)) {
+			size_t count = SlGetFieldLength();
+			std::vector<SaveLoad> oslt;
+
+			/* Cheats were added over the years without a savegame bump. They are
+			 * stored as 2 SLE_BOOLs per entry. "count" indicates how many SLE_BOOLs
+			 * are stored for this savegame. So read only "count" SLE_BOOLs (and in
+			 * result "count / 2" cheats). */
+			for (auto &sld : slt) {
+				count--;
+				oslt.push_back(sld);
+
+				if (count == 0) break;
+			}
+			slt = oslt;
+		}
+
+		if (!IsSavegameVersionBefore(SLV_RIFF_TO_ARRAY) && SlIterateArray() == -1) return;
+		SlObject(&_cheats, slt);
+		if (!IsSavegameVersionBefore(SLV_RIFF_TO_ARRAY) && SlIterateArray() != -1) SlErrorCorrupt("Too many CHTS entries");
+	}
+};
 
-static const ChunkHandler CHTS{ 'CHTS', Save_CHTS, Load_CHTS, nullptr, nullptr, CH_TABLE };
+static const CHTSChunkHandler CHTS;
 static const ChunkHandlerRef cheat_chunk_handlers[] = {
 	CHTS,
 };
-- 
cgit v1.2.3-54-g00ecf