[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