<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 4, 2014 at 7:25 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><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"><div class="gmail_quote"><div><div class="h5">On Tue, Nov 4, 2014 at 10:25 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><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"><div class="gmail_quote"><span>On Tue, Nov 4, 2014 at 10:18 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"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span>On Tue, Nov 4, 2014 at 5:32 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><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"><div class="gmail_quote"><span>On Tue, Nov 4, 2014 at 7:25 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">Author: alexfh<br>
Date: Tue Nov  4 09:25:22 2014<br>
New Revision: 221272<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=221272&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=221272&view=rev</a><br>
Log:<br>
[clang-tidy] Don't print a message if there's no error.<br></blockquote><div><br></div></span><div>Test coverage?</div></div></div></div></blockquote><div> </div></span><div>It's a trivial change, and it needs a custom configuration format handler to be tested which makes it relatively expensive to test it here.</div></div></div></div></blockquote><div><br></div></span><div>Sometimes this is an indication of a chunk of functionality that isn't tested. Is there much other coverage missing in-tree for testing custom configuration format handlers? Perhaps all together it'd be worth adding a custom configuration format handler, even if this one case doesn't look worthwhile.<br></div></div></div></div></blockquote></div></div></div></div></div></blockquote><div><br></div><div>It may make sense. I'll try to add some tests later, if this code grows more features.</div><div> </div><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"><div class="gmail_quote"><div><div class="h5"><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"><div class="gmail_quote"><div><br>Just a thought</div><span><div> </div><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"><div class="gmail_quote"><div>But rest assured that the change is going to be tested on every invocation of our internal build ;)</div></div></div></div></blockquote></span></div></div></div></blockquote></div></div><div><br>Sure enough - but having the testing upstream means we don't have to triage it on an internal failure when someone else breaks it upstream. </div></div></div></div></blockquote></div></div></div>