Re: [C++ Patch] PR 58126

2013-10-06 Thread Jason Merrill
On 10/06/2013 03:35 PM, Paolo Carlini wrote: PS: I struggled a bit with "%qT" vs "%q#T": I would slightly prefer simply talking about, eg, "base class 'A'", we do that in other cases, but here we are already using # which automatically picks class vs struct. Thus I think we should use # for the b

[C++ Patch] PR 58126

2013-10-06 Thread Paolo Carlini
Hi, it seems to me that in order to fix this accepts-invalid (for the testcase we don't reject the declaration 'B b;') we have simply to propagate from bases to derived the CLASSTYPE_READONLY_FIELDS_NEED_INIT and CLASSTYPE_REF_FIELDS_NEED_INIT flags. Thus, the few class.c lines in the patch b