[cfe-commits] [Patch][Review] Fix for error message PR12500

Richard Smith richard at metafoo.co.uk
Wed Apr 11 11:00:29 PDT 2012


As John noted on the bug, please drop the comma from the diagnostic. With
that change, LGTM.

On Wed, Apr 11, 2012 at 9:58 AM, Jonathan Sauer <jonathan.sauer at gmx.de>wrote:

> "Give me a ping, Vasily. One ping only, please."
>
> > Hello,
> >
> > as discussed in #llvm, I tried my hand at a patch for <
> http://llvm.org/bugs/show_bug.cgi?id=12500>.
> > I only changed the error message though, but did not implement the fixit
> suggestion. My local clang
> > tests pass with the changes.
> >
> > Please review and, if ok, commit.
> >
> >
> > Jonathan
> > <PR12500.diff>
> >
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120411/7e7d1cdd/attachment.html>


More information about the cfe-commits mailing list