[cfe-commits] r76758 - in /cfe/trunk/lib: AST/DeclCXX.cpp Sema/SemaDeclCXX.cpp Sema/SemaTemplateInstantiate.cpp

[Fariborz Jahanian]

On Jul 22, 2009, at 1:56 PM, Douglas Gregor wrote:

>
> On Jul 22, 2009, at 1:45 PM, Fariborz Jahanian wrote:
>
>>
>> On Jul 22, 2009, at 1:32 PM, John McCall wrote:
>>
>>>>
>>>
>>> Is there some crazy reason this needs to involve an allocation at  
>>> all,
>>> rather than just
>>>  this->VBases[i] = CXXBaseSpecifier(...);
>>
>> This creates a temporary which will go out of scope when we return
>> from this function.
>
> The copy is stored in VBases[i], and that's fine. I've made this fix  
> in the tree (along with fixing a few other memory-allocation-related  
> issues).

Oops. Yes, I missed that one  :).

- fj

>
>
> 	- Doug