[cfe-commits] Implement 'mutable'

[Douglas Gregor]

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