<div dir="ltr">I am fine with leaving those out until it causes a problem. On the other hand, adding them seems cheap. Up to you.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 8, 2014 at 9:12 AM, 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"><p dir="ltr">The tests are not included in the compilation database, so clang-tidy won't run on them. But I could add .clang-tidy files to the test/ directories, if you like.</p><div class="HOEnZb"><div class="h5">
<div class="gmail_quote">On 8 Sep 2014 09:08, "Chandler Carruth" <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Sep 7, 2014 at 11:23 PM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="overflow:hidden">Do we really need a separate file for Clang? Shouldn't D5236 suffice as Clang is always checked out into LLVM?<br></div></blockquote><div><br></div><div>There are folks that still develop without Clang checked out under LLVM. Seems cheap to support that here.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="overflow:hidden">
<br>
On the other hand, I would add files containing "Checks: '-*'" into the test/ directories of both. Mosts test won't be (and don't need to be) clang-tidy clean.</div></blockquote></div><div class="gmail_extra"><br></div>Agreed.<br><br></div></div>
</blockquote></div>
</div></div></blockquote></div><br></div>