[cfe-commits] Implement 'mutable'

Douglas Gregor dgregor at apple.com
Thu Nov 13 11:18:50 PST 2008

On Nov 8, 2008, at 12:32 PM, Sebastian Redl wrote:
> This patch implements parsing and semantic analysis of the mutable  
> keyword in C++.


+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

More information about the cfe-commits mailing list