From bf89109013bc51868173de3d8c71d28989afd39e Mon Sep 17 00:00:00 2001 From: Aaron Ballman Date: Mon, 31 Aug 2015 15:28:57 +0000 Subject: Using an early return as it is more clear; NFC. llvm-svn: 246447 --- .../clang-tidy/misc/UnusedAliasDeclsCheck.cpp | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) (limited to 'clang-tools-extra/clang-tidy/misc/UnusedAliasDeclsCheck.cpp') diff --git a/clang-tools-extra/clang-tidy/misc/UnusedAliasDeclsCheck.cpp b/clang-tools-extra/clang-tidy/misc/UnusedAliasDeclsCheck.cpp index ffa0873fb14..3af988a647a 100644 --- a/clang-tools-extra/clang-tidy/misc/UnusedAliasDeclsCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/UnusedAliasDeclsCheck.cpp @@ -20,13 +20,14 @@ namespace tidy { void UnusedAliasDeclsCheck::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) { - // We cannot do anything about headers (yet), as the alias declarations - // used in one header could be used by some other translation unit. - Finder->addMatcher( - namespaceAliasDecl(isExpansionInMainFile()).bind("alias"), this); - Finder->addMatcher(nestedNameSpecifier().bind("nns"), this); - } + if (!getLangOpts().CPlusPlus11) + return; + + // We cannot do anything about headers (yet), as the alias declarations + // used in one header could be used by some other translation unit. + Finder->addMatcher(namespaceAliasDecl(isExpansionInMainFile()).bind("alias"), + this); + Finder->addMatcher(nestedNameSpecifier().bind("nns"), this); } void UnusedAliasDeclsCheck::check(const MatchFinder::MatchResult &Result) { -- cgit v1.2.3