[clang] bcbea2a - [NFC] Remove redundant string copy
Nathan James via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 12 07:14:13 PDT 2021
Author: Nathan James
Date: 2021-04-12T15:13:59+01:00
New Revision: bcbea2ab84df87983fa9b30e94ea25eda0eb2338
URL: https://github.com/llvm/llvm-project/commit/bcbea2ab84df87983fa9b30e94ea25eda0eb2338
DIFF: https://github.com/llvm/llvm-project/commit/bcbea2ab84df87983fa9b30e94ea25eda0eb2338.diff
LOG: [NFC] Remove redundant string copy
Added:
Modified:
clang/include/clang/AST/TextNodeDumper.h
Removed:
################################################################################
diff --git a/clang/include/clang/AST/TextNodeDumper.h b/clang/include/clang/AST/TextNodeDumper.h
index 15ca348f47667..0955c494a74ad 100644
--- a/clang/include/clang/AST/TextNodeDumper.h
+++ b/clang/include/clang/AST/TextNodeDumper.h
@@ -69,10 +69,8 @@ class TextTreeStructure {
return;
}
- // We need to capture an owning-string in the lambda because the lambda
- // is invoked in a deferred manner.
- std::string LabelStr(Label);
- auto DumpWithIndent = [this, DoAddChild, LabelStr](bool IsLastChild) {
+ auto DumpWithIndent = [this, DoAddChild,
+ Label(Label.str())](bool IsLastChild) {
// Print out the appropriate tree structure and work out the prefix for
// children of this node. For instance:
//
@@ -89,8 +87,8 @@ class TextTreeStructure {
OS << '\n';
ColorScope Color(OS, ShowColors, IndentColor);
OS << Prefix << (IsLastChild ? '`' : '|') << '-';
- if (!LabelStr.empty())
- OS << LabelStr << ": ";
+ if (!Label.empty())
+ OS << Label << ": ";
this->Prefix.push_back(IsLastChild ? ' ' : '|');
this->Prefix.push_back(' ');
More information about the cfe-commits
mailing list