On Nov 8, 2008, at 12:32 PM, Sebastian Redl wrote:
> This patch implements parsing and semantic analysis of the mutable
> keyword in C++.
Cool.
+DIAG(err_bad_mutable, ERROR,
+ "'mutable' can only be applied to non-const, non-static, non-
reference "
+ "class data members")
That "non-const, non-static, non-reference" is a real mouthful. I
think it would be clearer if you split this into separate error
messages for "const", one for "static", references, and for things
that aren't data members.
Other than that, it looks good. Are you planning to submit a follow-up
patch that deals with the constness of MemberExprs for mutable members?
- Doug
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
