[clang-tools-extra] r233000 - Refactor: Simplify boolean expression in modularize

David Blaikie dblaikie at gmail.com
Mon Mar 23 12:40:59 PDT 2015


Author: dblaikie
Date: Mon Mar 23 14:40:59 2015
New Revision: 233000

URL: http://llvm.org/viewvc/llvm-project?rev=233000&view=rev
Log:
Refactor: Simplify boolean expression in modularize

Simplify boolean expressions using `true` and `false` with `clang-tidy`

Patch by Richard Thomson.

Differential Revision: http://reviews.llvm.org/D8534

Modified:
    clang-tools-extra/trunk/modularize/CoverageChecker.cpp

Modified: clang-tools-extra/trunk/modularize/CoverageChecker.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/modularize/CoverageChecker.cpp?rev=233000&r1=232999&r2=233000&view=diff
==============================================================================
--- clang-tools-extra/trunk/modularize/CoverageChecker.cpp (original)
+++ clang-tools-extra/trunk/modularize/CoverageChecker.cpp Mon Mar 23 14:40:59 2015
@@ -288,7 +288,7 @@ CoverageChecker::collectUmbrellaHeaderHe
   int HadErrors = Tool.run(new CoverageCheckerFrontendActionFactory(*this));
 
   // If we had errors, exit early.
-  return HadErrors ? false : true;
+  return !HadErrors;
 }
 
 // Called from CoverageCheckerCallbacks to track a header included





More information about the cfe-commits mailing list