<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 16, 2015 at 3:40 PM, Gonzalo BG <span dir="ltr"><<a href="mailto:gonzalobg88@gmail.com" target="_blank">gonzalobg88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">I see, thanks for the explanation. <br><br>I just started toying with clang-tidy today and while setting it up with CMake as a build system a verbose mode would have helped me a bit. Things that I would like to see are:<div><div>- the path of the '.clang-tidy' file found,</div></div></div></blockquote><div><br></div><div>Some logs to that effect are available in the assertions-enabled build when running with -debug. But it might make sense to add some optional logging in the release build as well. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div>- if there are any unknown checks,</div></div></div></blockquote><div><br></div><div>The -checks= option is a filter, not a list of checks. So there's currently no way to tell whether there are unknown checks listed.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div>- error on checks='checkA,checkB, checkC' (the whitespace error is too easy to make). </div></div></div></blockquote><div><br></div><div>Fixed in r247812.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br>I'm not finished with the setup yet since I still have some problems with system paths being recognized as project path and with the HeaderFilterRegex: '...' (I assume it does the same as -header-filter but that is actually not documented anywhere). <br><br>I guess it would also be nice if there was a `-header-remove` option that takes a regex to remove some headers since mixing regexes for headers to accept and headers to remove inside -header-filter is messy. </div><div class="gmail_extra"><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">I would not need -header-remove if I would be able to get -isystem headers removed though. I somehow have the feeling that either clang-tidy decides on system headers depending on include <> vs include "" (instead of looking for -I or -isystem in the compilation_database.json) or just finding system headers in the compilation_database.json is broken. <br></div></div></blockquote><div><br></div><div>The ability to  use #include <> vs #include "" depends on whether the header can be found using -I paths or -isystem paths. If both can be used for some headers, it seems like something that should be addressed on the build-system (or build configuration) level to make sure code readers and tools don't get confused.</div></div>
</div></div>