From f20790370a7d3bee8264ee336ed308f8990a3421 Mon Sep 17 00:00:00 2001
From: Lioncash <mai.iam2048@gmail.com>
Date: Mon, 30 Jan 2023 12:57:40 -0500
Subject: [PATCH] Core/ActionReplay: Make use of std::span where applicable

Generifies the interface a little by not strictly requiring the use of
std::vector.
---
 Source/Core/Core/ActionReplay.cpp |  8 ++++----
 Source/Core/Core/ActionReplay.h   | 10 +++++-----
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/Source/Core/Core/ActionReplay.cpp b/Source/Core/Core/ActionReplay.cpp
index 31e0c6766e..c702748003 100644
--- a/Source/Core/Core/ActionReplay.cpp
+++ b/Source/Core/Core/ActionReplay.cpp
@@ -112,7 +112,7 @@ struct ARAddr
 
 // ----------------------
 // AR Remote Functions
-void ApplyCodes(const std::vector<ARCode>& codes)
+void ApplyCodes(std::span<const ARCode> codes)
 {
   if (!Config::Get(Config::MAIN_ENABLE_CHEATS))
     return;
@@ -132,7 +132,7 @@ void SetSyncedCodesAsActive()
   s_active_codes = s_synced_codes;
 }
 
-void UpdateSyncedCodes(const std::vector<ARCode>& codes)
+void UpdateSyncedCodes(std::span<const ARCode> codes)
 {
   s_synced_codes.clear();
   s_synced_codes.reserve(codes.size());
@@ -141,7 +141,7 @@ void UpdateSyncedCodes(const std::vector<ARCode>& codes)
   s_synced_codes.shrink_to_fit();
 }
 
-std::vector<ARCode> ApplyAndReturnCodes(const std::vector<ARCode>& codes)
+std::vector<ARCode> ApplyAndReturnCodes(std::span<const ARCode> codes)
 {
   if (Config::Get(Config::MAIN_ENABLE_CHEATS))
   {
@@ -250,7 +250,7 @@ std::vector<ARCode> LoadCodes(const IniFile& global_ini, const IniFile& local_in
   return codes;
 }
 
-void SaveCodes(IniFile* local_ini, const std::vector<ARCode>& codes)
+void SaveCodes(IniFile* local_ini, std::span<const ARCode> codes)
 {
   std::vector<std::string> lines;
   std::vector<std::string> enabled_lines;
diff --git a/Source/Core/Core/ActionReplay.h b/Source/Core/Core/ActionReplay.h
index 13b6294d30..1c7fb61fa9 100644
--- a/Source/Core/Core/ActionReplay.h
+++ b/Source/Core/Core/ActionReplay.h
@@ -3,8 +3,8 @@
 
 #pragma once
 
+#include <span>
 #include <string>
-#include <utility>
 #include <variant>
 #include <vector>
 
@@ -37,15 +37,15 @@ struct ARCode
 
 void RunAllActive();
 
-void ApplyCodes(const std::vector<ARCode>& codes);
+void ApplyCodes(std::span<const ARCode> codes);
 void SetSyncedCodesAsActive();
-void UpdateSyncedCodes(const std::vector<ARCode>& codes);
-std::vector<ARCode> ApplyAndReturnCodes(const std::vector<ARCode>& codes);
+void UpdateSyncedCodes(std::span<const ARCode> codes);
+std::vector<ARCode> ApplyAndReturnCodes(std::span<const ARCode> codes);
 void AddCode(ARCode new_code);
 void LoadAndApplyCodes(const IniFile& global_ini, const IniFile& local_ini);
 
 std::vector<ARCode> LoadCodes(const IniFile& global_ini, const IniFile& local_ini);
-void SaveCodes(IniFile* local_ini, const std::vector<ARCode>& codes);
+void SaveCodes(IniFile* local_ini, std::span<const ARCode> codes);
 
 using EncryptedLine = std::string;
 std::variant<std::monostate, AREntry, EncryptedLine> DeserializeLine(const std::string& line);