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

John McCall rjmccall at apple.com
Fri Sep 10 18:24:58 PDT 2010


On Sep 10, 2010, at 4:48 PM, Argyrios Kyrtzidis wrote:

> 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.

Well said.

John.



More information about the cfe-commits mailing list