commit d88fb2205991b0fb011fdd5311d6730e294f1940
author Stephen Kelly <steveire@gmail.com> Thu Mar 25 14:11:38 2021 +0000
committer Copybara-Service <copybara-worker@google.com> Fri Apr 16 12:12:44 2021 -0700
tree b3ad44757c55c97b757f485245814e4938e7fdc9
parent 67ab3572180c5d214c08c4722ddf37417c784848

NFC: Remove condition to simplify code

The run method is only called if there is a match with a binding.

GitOrigin-RevId: c960c3836c6a28ba8f8c0b21db0474bf7d674a18

lib/Tooling/DumpTool/ASTSrcLocProcessor.cpp

diff --git a/lib/Tooling/DumpTool/ASTSrcLocProcessor.cpp b/lib/Tooling/DumpTool/ASTSrcLocProcessor.cpp
index 5121740..a19114a 100644
--- a/lib/Tooling/DumpTool/ASTSrcLocProcessor.cpp
+++ b/lib/Tooling/DumpTool/ASTSrcLocProcessor.cpp
@@ -157,29 +157,28 @@
 
 void ASTSrcLocProcessor::run(const MatchFinder::MatchResult &Result) {
 
-  if (const auto *ASTClass =
-          Result.Nodes.getNodeAs<clang::CXXRecordDecl>("className")) {
+  const auto *ASTClass =
+      Result.Nodes.getNodeAs<clang::CXXRecordDecl>("className");
 
-    StringRef ClassName = ASTClass->getName();
+  StringRef ClassName = ASTClass->getName();
 
-    ClassData CD;
+  ClassData CD;
 
-    const auto *NodeClade =
-        Result.Nodes.getNodeAs<clang::CXXRecordDecl>("nodeClade");
-    StringRef CladeName = NodeClade->getName();
+  const auto *NodeClade =
+      Result.Nodes.getNodeAs<clang::CXXRecordDecl>("nodeClade");
+  StringRef CladeName = NodeClade->getName();
 
-    if (const auto *DerivedFrom =
-            Result.Nodes.getNodeAs<clang::CXXRecordDecl>("derivedFrom"))
-      ClassInheritance[ClassName] = DerivedFrom->getName();
+  if (const auto *DerivedFrom =
+          Result.Nodes.getNodeAs<clang::CXXRecordDecl>("derivedFrom"))
+    ClassInheritance[ClassName] = DerivedFrom->getName();
 
-    CD.ASTClassLocations =
-        CaptureMethods("class clang::SourceLocation", ASTClass, Result);
-    CD.ASTClassRanges =
-        CaptureMethods("class clang::SourceRange", ASTClass, Result);
+  CD.ASTClassLocations =
+      CaptureMethods("class clang::SourceLocation", ASTClass, Result);
+  CD.ASTClassRanges =
+      CaptureMethods("class clang::SourceRange", ASTClass, Result);
 
-    if (!CD.isEmpty()) {
-      ClassEntries[ClassName] = CD;
-      ClassesInClade[CladeName].push_back(ClassName);
-    }
+  if (!CD.isEmpty()) {
+    ClassEntries[ClassName] = CD;
+    ClassesInClade[CladeName].push_back(ClassName);
   }
 }