<div dir="ltr">I think for this (and probably many future clang-tidy checks), the answer "Does this make sense as a compiler warning?" is always: YES. However, clang-tidy enables us to quickly develop and tune an implementation using AST matchers, not worrying about slowing down the compiler and testing it on a larger codebase in order establish whether a check/warning provides sufficient benefit and even let users play around with it to get more feedback. So, I see clang-tidy as a prototyping platform and good checks will be turned into compiler warnings. At some stage we might want to think about how we can simplify that process, i.e. how we can make porting a clang-tidy check into a compiler warning easier, but IMO that stage isn't necessarily now.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 10, 2014 at 5:38 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">Wouldn't this make sense as a regular warning?<br>
<br>
</div><a href="http://reviews.llvm.org/D4457" target="_blank">http://reviews.llvm.org/D4457</a><br>
<br>
<br>
</blockquote></div><br></div>