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 <[email protected]>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 > > [email protected] > > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
