<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 19, 2014 at 5:35 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi klimek,<br>
<br>
This patch implements filtering of clang-tidy diagnostic messages by<br>
the check name, so that "clang-tidy -checks=^llvm-" won't output any clang<br>
warnings, for example. This is also helpful to run specific static-analyzer<br>
checks: static analyzer always needs core checks to be enabled, but the user may<br>
be interested only in the checks he asked for.<br>
<br>
This patch also exposes warning option names for built-in diagnostics. We need<br>
to have a namespace for these names to avoid collisions and to allow convenient<br>
filtering, so I prefix them with "-W". I'm not sure it's the best thing to do,<br>
and maybe "W" or "clang-diagnostic-" or something like this would be better.<br></blockquote><div><br></div><div>+1 for clang-diagnostic...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
<a href="http://llvm-reviews.chandlerc.com/D3121" target="_blank">http://llvm-reviews.chandlerc.com/D3121</a><br>
<br>
Files:<br>
  clang-tidy/ClangTidy.cpp<br>
  clang-tidy/ClangTidy.h<br>
  clang-tidy/ClangTidyDiagnosticConsumer.cpp<br>
  clang-tidy/ClangTidyDiagnosticConsumer.h<br>
  clang-tidy/ClangTidyModule.cpp<br>
  test/clang-tidy/diagnostic.cpp<br>
  unittests/clang-tidy/ClangTidyTest.h<br>
</blockquote></div><br></div></div>