[clang-tools-extra] r222363 - [clang-tidy] Count errors in ClangTidyDiagnosticConsumer

Hans Wennborg hans at chromium.org
Wed Nov 19 14:03:08 PST 2014


On Wed, Nov 19, 2014 at 8:07 AM, Alexander Kornienko <alexfh at google.com> wrote:
> Author: alexfh
> Date: Wed Nov 19 10:07:26 2014
> New Revision: 222363
>
> URL: http://llvm.org/viewvc/llvm-project?rev=222363&view=rev
> Log:
> [clang-tidy] Count errors in ClangTidyDiagnosticConsumer
>
> Modified:
>     clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp
>     clang-tools-extra/trunk/test/clang-tidy/clang-tidy-diff.cpp
>
> Modified: clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp
> URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp?rev=222363&r1=222362&r2=222363&view=diff
> ==============================================================================
> --- clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp (original)
> +++ clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp Wed Nov 19 10:07:26 2014
> @@ -277,6 +277,9 @@ void ClangTidyDiagnosticConsumer::finali
>
>  void ClangTidyDiagnosticConsumer::HandleDiagnostic(
>      DiagnosticsEngine::Level DiagLevel, const Diagnostic &Info) {
> +  // Count warnings/errors.
> +  DiagnosticConsumer::HandleDiagnostic(DiagLevel, Info);

This was causing clang-tidy invocations that would previously pass
despite source errors to fail.

I've reverted in r222390.



More information about the cfe-commits mailing list