From af576839d894c3a34cbf6b7e007e674415f98d72 Mon Sep 17 00:00:00 2001
From: Lioncash <mathew1800@gmail.com>
Date: Tue, 16 Jul 2019 04:11:53 -0400
Subject: [PATCH 1/2] Common/SettingsHandler: Use std::string_view where
 applicable

Allows passed in strings to be non-allocating.
---
 Source/Core/Common/SettingsHandler.cpp | 16 ++++++++--------
 Source/Core/Common/SettingsHandler.h   |  5 +++--
 2 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/Source/Core/Common/SettingsHandler.cpp b/Source/Core/Common/SettingsHandler.cpp
index e7292438ef..49e142daad 100644
--- a/Source/Core/Common/SettingsHandler.cpp
+++ b/Source/Core/Common/SettingsHandler.cpp
@@ -38,27 +38,27 @@ void SettingsHandler::SetBytes(Buffer&& buffer)
   Decrypt();
 }
 
-std::string SettingsHandler::GetValue(const std::string& key) const
+std::string SettingsHandler::GetValue(std::string_view key) const
 {
-  std::string delim = std::string("\r\n");
-  std::string toFind = delim + key + "=";
+  constexpr char delim[] = "\r\n";
+  std::string toFind = std::string(delim).append(key).append("=");
   size_t found = decoded.find(toFind);
 
-  if (found != decoded.npos)
+  if (found != std::string_view::npos)
   {
     size_t delimFound = decoded.find(delim, found + toFind.length());
-    if (delimFound == decoded.npos)
+    if (delimFound == std::string_view::npos)
       delimFound = decoded.length() - 1;
     return decoded.substr(found + toFind.length(), delimFound - (found + toFind.length()));
   }
   else
   {
-    toFind = key + "=";
+    toFind = std::string(key).append("=");
     found = decoded.find(toFind);
     if (found == 0)
     {
       size_t delimFound = decoded.find(delim, found + toFind.length());
-      if (delimFound == decoded.npos)
+      if (delimFound == std::string_view::npos)
         delimFound = decoded.length() - 1;
       return decoded.substr(found + toFind.length(), delimFound - (found + toFind.length()));
     }
@@ -89,7 +89,7 @@ void SettingsHandler::Reset()
   m_buffer = {};
 }
 
-void SettingsHandler::AddSetting(const std::string& key, const std::string& value)
+void SettingsHandler::AddSetting(std::string_view key, std::string_view value)
 {
   for (const char& c : key)
   {
diff --git a/Source/Core/Common/SettingsHandler.h b/Source/Core/Common/SettingsHandler.h
index 4fd1b0ea42..bd32ebd420 100644
--- a/Source/Core/Common/SettingsHandler.h
+++ b/Source/Core/Common/SettingsHandler.h
@@ -8,6 +8,7 @@
 
 #include <array>
 #include <string>
+#include <string_view>
 
 #include "Common/CommonTypes.h"
 
@@ -27,11 +28,11 @@ public:
   SettingsHandler();
   explicit SettingsHandler(Buffer&& buffer);
 
-  void AddSetting(const std::string& key, const std::string& value);
+  void AddSetting(std::string_view key, std::string_view value);
 
   const Buffer& GetBytes() const;
   void SetBytes(Buffer&& buffer);
-  std::string GetValue(const std::string& key) const;
+  std::string GetValue(std::string_view key) const;
 
   void Decrypt();
   void Reset();

From 13a454d603d379d6980d501cf117f5a61264d548 Mon Sep 17 00:00:00 2001
From: Lioncash <mathew1800@gmail.com>
Date: Tue, 16 Jul 2019 04:23:18 -0400
Subject: [PATCH 2/2] Common/SettingsHandler: Use fmt for printing out the
 serial number

Same thing, less code.
---
 Source/Core/Common/SettingsHandler.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/Source/Core/Common/SettingsHandler.cpp b/Source/Core/Common/SettingsHandler.cpp
index 49e142daad..e7b3d5d066 100644
--- a/Source/Core/Common/SettingsHandler.cpp
+++ b/Source/Core/Common/SettingsHandler.cpp
@@ -9,9 +9,10 @@
 #include <cstddef>
 #include <ctime>
 #include <iomanip>
-#include <sstream>
 #include <string>
 
+#include <fmt/time.h>
+
 #include "Common/CommonTypes.h"
 
 namespace Common
@@ -124,8 +125,6 @@ std::string SettingsHandler::GenerateSerialNumber()
   // Must be 9 characters at most; otherwise the serial number will be rejected by SDK libraries,
   // as there is a check to ensure the string length is strictly lower than 10.
   // 3 for %j, 2 for %H, 2 for %M, 2 for %S.
-  std::stringstream stream;
-  stream << std::put_time(std::localtime(&t), "%j%H%M%S");
-  return stream.str();
+  return fmt::format("{:%j%H%M%S}", *std::localtime(&t));
 }
 }  // namespace Common