<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Apr 13, 2011, at 2:25 PM, jahanian wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Apr 13, 2011, at 2:18 PM, Douglas Gregor wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div><br>On Apr 13, 2011, at 2:02 PM, Eli Friedman wrote:<br><br><blockquote type="cite">On Wed, Apr 13, 2011 at 1:31 PM, Fariborz Jahanian <<a href="mailto:fjahanian@apple.com">fjahanian@apple.com</a>> wrote:<br></blockquote><font class="Apple-style-span" color="#540000"><br></font>Actually, this is strange: I thought this code already existed. I wonder why the Fix-Its with macro-instantiation source locations aren't getting implicitly dropped appropriately?<br></div></blockquote><div><br></div>Because, it turns out to eventually become a null SourceLocation. I think the right fix is to pass the macro-instantiation source locations  and see if it magically</div><div>drops the fixit. I will take a look.</div></div></blockquote><div><br></div>In r129481.</div><div><br></div><div>- fariborz</div><div><br></div><br></body></html>