On Aug 5, 2011, at 2:24 PM, Caitlin Sadowski wrote:

> Doug,
> 
> Thanks for reviewing the patch! Here is an updated patch, with
> comments inline. Let me know if it is ok to submit, or if you have any
> other edits.

+      if (ArgExpr.isInvalid()) {
+        ArgExprsOk = false;
+        SkipUntil(tok::r_paren);

Wouldn't MatchRHSPunctuation provide better recovery diagnostics?

This looks fine to commit.

        - Doug
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to