From 88a973131cd3f865651bd4169a6e3bf6826fdecc Mon Sep 17 00:00:00 2001 From: Lioncash Date: Mon, 11 Dec 2023 07:53:10 -0500 Subject: Common/SettingsHandler: Use std::string_view more We don't need to enforce the use of std::string instances with AddSetting(). We can accept views and only construct one string, rather than three temporaries. --- Source/Core/Common/SettingsHandler.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'Source/Core/Common/SettingsHandler.cpp') diff --git a/Source/Core/Common/SettingsHandler.cpp b/Source/Core/Common/SettingsHandler.cpp index 81e5da5989..dd84b6967b 100644 --- a/Source/Core/Common/SettingsHandler.cpp +++ b/Source/Core/Common/SettingsHandler.cpp @@ -92,12 +92,12 @@ 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) { - WriteLine(key + '=' + value + "\r\n"); + WriteLine(fmt::format("{}={}\r\n", key, value)); } -void SettingsHandler::WriteLine(const std::string& str) +void SettingsHandler::WriteLine(std::string_view str) { const u32 old_position = m_position; const u32 old_key = m_key; -- cgit v1.2.3