<div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 1, 2012 at 10:51 AM, Argyrios Kyrtzidis <span dir="ltr"><<a href="mailto:kyrtzidis@apple.com" target="_blank" class="cremed">kyrtzidis@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Kim,<br>
<div class="im"><br>
On Sep 30, 2012, at 12:29 AM, Kim Gräsman <<a href="mailto:kim.grasman@gmail.com" class="cremed">kim.grasman@gmail.com</a>> wrote:<br>
<br>
> On Sat, Sep 29, 2012 at 11:50 PM, Kim Gräsman <<a href="mailto:kim.grasman@gmail.com" class="cremed">kim.grasman@gmail.com</a>> wrote:<br>
>><br>
>> I would expect either '<stdio.h>' or<br>
>> 'HEADER_NAME', but I think getting the expanded range ('<stdio.h>')<br>
>> would be the most natural.<br>
><br>
> Having looked into our callback implementation, we already handle<br>
> macro expansion, and I think passing down the macro name range<br>
> unexpanded gives more fidelity. So I change my mind -- I think getting<br>
> the range as-written would be better ('HEADER_NAME'). Of course,<br>
> assuming that FilenameRange.getBegin().isMacroID() is still true so<br>
> callback sinks can expand it.<br>
<br>
</div>Your suggestion is a bit contradictory, if we get the range 'HEADER_NAME', then FilenameRange.getBegin().isMacroID() will be false (since it's pointing at the beginning of the macro expansion).<br>
The most fidelity option would be to give a range for  '<stdio.h>' (MacroIDs for begin and end), from which a callback sink can get at the 'HEADER_NAME' file-level range.<br>
Is this your recommendation ?<br></blockquote><div><br></div><div>FWIW, I think it is, and it makes sense. ;] The source location structure is... confusing though.</div><div><br></div><div>Richard, thoughts?</div></div></div>