<div dir="ltr">On Wed, Mar 20, 2013 at 5:16 PM, Tareq A. Siraj <span dir="ltr"><<a href="mailto:tareq.a.siraj@intel.com" target="_blank">tareq.a.siraj@intel.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"><br>
<br>
================<br>
Comment at: cpp11-migrate/UseNullptr/NullptrActions.cpp:125<br>
@@ +124,3 @@<br>
</div><div class="im">+    // coming from a macro called NULL.<br>
+    if (SM.isMacroArgExpansion(StartLoc) && SM.isMacroArgExpansion(EndLoc)) {<br>
+      StartLoc = SM.getFileLoc(StartLoc);<br>
</div>----------------<br>
<div class="im">Manuel Klimek wrote:<br>
> I assume we don't care that those can both be true, but start and end can still be produced from different macro expansions? Or do we check that somewhere else?<br>
</div>Can you provide an example for start/end coming from different macro expansions in case of ImplicitCastNodes? I don't think we do this check but is it really necessary for ImplicitCastNodes? Thanks.<br></blockquote>
<div><br></div><div style>Well, I'm not sure. What about:</div><div style>#define B(X) X ## .742</div><div style><br></div><div style>void f(int i) {</div><div style>  f(B(3));</div><div style>}</div><div style><br></div>
</div></div></div>