diff options
author | Pavel Labath <labath@google.com> | 2015-07-28 09:18:32 +0000 |
---|---|---|
committer | Pavel Labath <labath@google.com> | 2015-07-28 09:18:32 +0000 |
commit | 3a29f8b9ec905f27829b6180c10f0f41fa883f0f (patch) | |
tree | 13efc3f7b775406d65ef3b5aa43271699eb82479 /lldb/source/Core/FormatEntity.cpp | |
parent | e5b47640ee437b49313657aeee7f4e8c08bb0019 (diff) | |
download | bcm5719-llvm-3a29f8b9ec905f27829b6180c10f0f41fa883f0f.tar.gz bcm5719-llvm-3a29f8b9ec905f27829b6180c10f0f41fa883f0f.zip |
Fix warnings detected by -Wpessimizing-move
patch by Eugene Zelenko
Differential Revision: http://reviews.llvm.org/D11429
llvm-svn: 243399
Diffstat (limited to 'lldb/source/Core/FormatEntity.cpp')
-rw-r--r-- | lldb/source/Core/FormatEntity.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/lldb/source/Core/FormatEntity.cpp b/lldb/source/Core/FormatEntity.cpp index e89d6c9cb4a..b960007251f 100644 --- a/lldb/source/Core/FormatEntity.cpp +++ b/lldb/source/Core/FormatEntity.cpp @@ -1976,7 +1976,7 @@ ParseEntry (const llvm::StringRef &format_str, switch (entry_def->type) { case FormatEntity::Entry::Type::ParentString: - entry.string = std::move(format_str.str()); + entry.string = format_str.str(); return error; // Success case FormatEntity::Entry::Type::ParentNumber: @@ -2026,7 +2026,7 @@ ParseEntry (const llvm::StringRef &format_str, { // Any value whose separator is a with a ':' means this value has a string argument // that needs to be stored in the entry (like "${script.var:modulename.function}") - entry.string = std::move(value.str()); + entry.string = value.str(); } else { @@ -2247,7 +2247,7 @@ FormatEntity::ParseInternal (llvm::StringRef &format, Entry &parent_entry, uint3 Entry entry; if (!variable_format.empty()) { - entry.printf_format = std::move(variable_format.str()); + entry.printf_format = variable_format.str(); // If the format contains a '%' we are going to assume this is // a printf style format. So if you want to format your thread ID @@ -2449,7 +2449,7 @@ MakeMatch (const llvm::StringRef &prefix, const char *suffix) { std::string match(prefix.str()); match.append(suffix); - return std::move(match); + return match; } static void @@ -2463,7 +2463,7 @@ AddMatches (const FormatEntity::Entry::Definition *def, { for (size_t i=0; i<n; ++i) { - std::string match = std::move(prefix.str()); + std::string match = prefix.str(); if (match_prefix.empty()) matches.AppendString(MakeMatch (prefix, def->children[i].name)); else if (strncmp(def->children[i].name, match_prefix.data(), match_prefix.size()) == 0) @@ -2488,7 +2488,7 @@ FormatEntity::AutoComplete (const char *s, // Hitting TAB after $ at the end of the string add a "{" if (dollar_pos == str.size() - 1) { - std::string match = std::move(str.str()); + std::string match = str.str(); match.append("{"); matches.AppendString(std::move(match)); } @@ -2521,12 +2521,12 @@ FormatEntity::AutoComplete (const char *s, if (n > 0) { // "${thread.info" <TAB> - matches.AppendString(std::move(MakeMatch (str, "."))); + matches.AppendString(MakeMatch(str, ".")); } else { // "${thread.id" <TAB> - matches.AppendString(std::move(MakeMatch (str, "}"))); + matches.AppendString(MakeMatch (str, "}")); word_complete = true; } } |