[PATCH] Refactor: Simplify boolean expression in modularize

Richard legalize at xmission.com
Sun Mar 22 13:44:21 PDT 2015


Hi alexfh, jtsoftware, rafael, chapuni, dblaikie, bkramer, aprantl,

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

http://reviews.llvm.org/D8534

Files:
  modularize/CoverageChecker.cpp

Index: modularize/CoverageChecker.cpp
===================================================================
--- modularize/CoverageChecker.cpp
+++ modularize/CoverageChecker.cpp
@@ -288,7 +288,7 @@
   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

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8534.22443.patch
Type: text/x-patch
Size: 443 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150322/83787ffb/attachment.bin>


More information about the cfe-commits mailing list