Looks good, committed in r233000

On Sun, Mar 22, 2015 at 1:44 PM, Richard <[email protected]> wrote:

> 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/
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to