<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 14 December 2013 10:12, Kim Gräsman <span dir="ltr"><<a href="mailto:kim.grasman@gmail.com" target="_blank">kim.grasman@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">John Thompson built a tool like that a few weeks back:<br>
<a href="https://github.com/llvm-mirror/clang-tools-extra/tree/master/pp-trace" target="_blank">https://github.com/llvm-mirror/clang-tools-extra/tree/master/pp-trace</a><br>
<br>
for the specific purpose of testing PPCallbacks. He's been trying to<br>
call attention to getting it into Clang core.<br></blockquote><div><br></div><div>Hi Kim,</div><div><br></div><div>An alternative would be to make it to the test-suite, so we're sure it gets at least tested once every few commits.</div>
<div><br></div><div>There's a Regression directory which contains very quick tests that you (him) could include there.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm still on the fence whether it's a good idea to build a separate<br>
tool to test PPCallbacks, because failures would signal problems with<br>
the tool, not with PPCallbacks. Intuitively, I think unit tests would<br>
be better, but that was discouraged by someone else in the original<br>
review.<br></blockquote><div></div></div><br></div><div class="gmail_extra">I like unit tests, but not to the point where it makes things harder. Some of the comments that have been said in this list are very pertinent (and I had my share of problems with unit tests, code coverage, etc), so I won't defend any strategy in specific.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">If you could get your tests somewhere, it'd be already great. The test-suite seems like a good compromise, even if just temporarily.</div><div class="gmail_extra">
<br></div><div class="gmail_extra">cheers,</div><div class="gmail_extra">--renato</div></div>