From b92871111a400cb8cf834db59366400ed9568960 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Wed, 22 Mar 2017 19:09:22 -0400 Subject: [PATCH] IniFile: std::move a std::string in GetLines Also gets rid of an unnecessary string copy. --- Source/Core/Common/IniFile.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Source/Core/Common/IniFile.cpp b/Source/Core/Common/IniFile.cpp index 1fd0f7c343..c58b3cf50e 100644 --- a/Source/Core/Common/IniFile.cpp +++ b/Source/Core/Common/IniFile.cpp @@ -274,13 +274,13 @@ void IniFile::Section::SetLines(std::vector&& lines) bool IniFile::Section::GetLines(std::vector* lines, const bool remove_comments) const { - for (std::string line : m_lines) + for (const std::string& line : m_lines) { - line = StripSpaces(line); + std::string stripped_line = StripSpaces(line); if (remove_comments) { - size_t commentPos = line.find('#'); + size_t commentPos = stripped_line.find('#'); if (commentPos == 0) { continue; @@ -288,11 +288,11 @@ bool IniFile::Section::GetLines(std::vector* lines, const bool remo if (commentPos != std::string::npos) { - line = StripSpaces(line.substr(0, commentPos)); + stripped_line = StripSpaces(stripped_line.substr(0, commentPos)); } } - lines->push_back(line); + lines->push_back(std::move(stripped_line)); } return true;