[cfe-commits] r113634 - /cfe/trunk/include/clang/AST/ExprCXX.h

Argyrios Kyrtzidis kyrtzidis at apple.com
Fri Sep 10 16:48:55 PDT 2010


On Sep 10, 2010, at 10:59 PM, Douglas Gregor wrote:

> 
> On Sep 10, 2010, at 2:15 PM, Sebastian Redl wrote:
> 
>> Author: cornedbee
>> Date: Fri Sep 10 16:15:56 2010
>> New Revision: 113634
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=113634&view=rev
>> Log:
>> Follow Ted's example and hide CXXNoexceptExpr's setters from all but serialization.
> 
> I've actually been leaning toward removing the setters entirely, then letting serialization poke at the fields directly.

Yes, serialization using setters is pointless; the less trivial the setter the more dangerous it is for serialization to use it.

-Argiris

> 
> 	- Doug
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits





More information about the cfe-commits mailing list