As John noted on the bug, please drop the comma from the diagnostic. With that change, LGTM.<br><br><div class="gmail_quote">On Wed, Apr 11, 2012 at 9:58 AM, Jonathan Sauer <span dir="ltr"><<a href="mailto:jonathan.sauer@gmx.de">jonathan.sauer@gmx.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">"Give me a ping, Vasily. One ping only, please."<br>
<div><div class="h5"><br>
> Hello,<br>
><br>
> as discussed in #llvm, I tried my hand at a patch for <<a href="http://llvm.org/bugs/show_bug.cgi?id=12500" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=12500</a>>.<br>
> I only changed the error message though, but did not implement the fixit suggestion. My local clang<br>
> tests pass with the changes.<br>
><br>
> Please review and, if ok, commit.<br>
><br>
><br>
> Jonathan<br>
</div></div>> <PR12500.diff><br>
><br>
> _______________________________________________<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>_______________________________________________<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>