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

Hans Wennborg hans at chromium.org
Wed Nov 19 14:02:23 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 caused test errors internally:
http://sponge/fd52baa1-9e70-4607-8be7-38c53da004c5

I'll revert to attempt to unblock the intergrate.



More information about the cfe-commits mailing list