On Fri, Oct 5, 2012 at 4:44 PM, Philip Craig <span dir="ltr"><<a href="mailto:philipjcraig@gmail.com" target="_blank" class="cremed">philipjcraig@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Fri, Oct 5, 2012 at 2:40 PM, Abramo Bagnara<br>
<<a href="mailto:abramo.bagnara@bugseng.com" class="cremed">abramo.bagnara@bugseng.com</a>> wrote:<br>
> Il 05/10/2012 00:15, Chandler Carruth ha scritto:<br>
>> On Thu, Oct 4, 2012 at 3:12 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a><br>
>> <mailto:<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>>> wrote:<br>
>><br>
>>     Can we get a test case for this?<br>
>><br>
>><br>
>> Also, not just for this, but for all of the source location fixes you're<br>
>> making?<br>
><br>
> This problem has been discussed a few time in past, but after discussion<br>
> on the list we failed to find a nice (and easy) solution for testing.<br>
><br>
> If things are now changed with tooling infrastructure, this would be<br>
> nice to hear. If someone is so kind to write a test for this patch we<br>
> will be happy to add the others replicating the same mechanism shown.<br>
><br>
> The testcase if very simple "void f(i) { }", what it should be tested is<br>
> that source range for ParamDecl matches the 'i'.<br>
<br>
</div>Here's a patch for the test. I don't know whether you want to keep<br>
these tests from the other RAV tests or not.<br></blockquote><div><br></div><div>My suggestion would be to create an ASTSourceLocationTest.cpp file to hold these. We can split them up by root node kind if they ever get too big, but I want a place where we can stash increasing numbers of utilities for digging out and testing source locations.</div>
<div><br></div><div>-Chandler</div><div><br></div></div></div>