Using an early return as it is more clear; NFC.

llvm-svn: 246447

1 files changed, 12 insertions, 11 deletions

diff --git a/clang-tools-extra/clang-tidy/misc/MoveConstructorInitCheck.cpp b/clang-tools-extra/clang-tidy/misc/MoveConstructorInitCheck.cpp
index 5b2c789a6c8..5e513d82748 100644
--- a/clang-tools-extra/clang-tidy/misc/MoveConstructorInitCheck.cpp
+++ b/clang-tools-extra/clang-tidy/misc/MoveConstructorInitCheck.cpp
@@ -19,17 +19,18 @@ namespace tidy {
 void MoveConstructorInitCheck::registerMatchers(MatchFinder *Finder) {
   // Only register the matchers for C++11; the functionality currently does not
   // provide any benefit to other languages, despite being benign.
-  if (getLangOpts().CPlusPlus11) {
-    Finder->addMatcher(
-      constructorDecl(unless(isImplicit()), allOf(
-        isMoveConstructor(),
-        hasAnyConstructorInitializer(
-          ctorInitializer(withInitializer(constructExpr(hasDeclaration(
-            constructorDecl(isCopyConstructor()).bind("ctor")
-            )))).bind("init")
-          )
-        )), this);
-  }
+  if (!getLangOpts().CPlusPlus11)
+    return;
+
+  Finder->addMatcher(
+    constructorDecl(unless(isImplicit()), allOf(
+      isMoveConstructor(),
+      hasAnyConstructorInitializer(
+        ctorInitializer(withInitializer(constructExpr(hasDeclaration(
+          constructorDecl(isCopyConstructor()).bind("ctor")
+          )))).bind("init")
+        )
+      )), this); 
 }
 
 void MoveConstructorInitCheck::check(const MatchFinder::MatchResult &Result) {