<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt">Committed at r167268.<br><br><div class="gmail_quote">On Fri, Oct 5, 2012 at 3:18 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Minor point: misspelled "ColonColon" as "ColocColon".<br>
"GetLookAheadToken(0)" is synonymous with "getCurToken" as far as I<br>
know (& the latter's infinitely more common)<br>
<br>
Other than that it all looks reasonable - I assume creating a removal<br>
for a default SourceLocation (in the case where there is no right<br>
paren to remove) isn't problematic (& just does nothing/is silent)?<br>
(judging by your test cases I assume it works fine, but I didn't<br>
build/run it myself to check)<br>
<br>
Otherwise the only issue to me is whether this is the best error<br>
recovery, but that's probably best left to experimentation.<br>
<div><div class="h5"><br>
On Fri, Oct 5, 2012 at 2:18 PM, Richard Trieu <<a href="mailto:rtrieu@google.com">rtrieu@google.com</a>> wrote:<br>
> Clang was crashing when encountering a '(' after '::'.  This patch will<br>
> allow Clang to detect this code, remove the parens from the token stream,<br>
> and emit an error with a fix it.  This will fix PR11852.<br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br></div>