svn commit: r204537 - in projects/clangbsd/contrib/libstdc++: include/ext src

Bruce Simpson bms at incunabulum.net
Tue Mar 2 13:00:51 UTC 2010


On 03/02/10 08:22, Roman Divacky wrote:
> <disclaimer>
> I am not a C++ expert but these changes were suggested by a member
> of C++ committee. ie. I trust them :)
>>> @@ -382,7 +382,7 @@ _GLIBCXX_BEGIN_NAMESPACE(__gnu_cxx)
>>>   	  {
>>>   	    _Rope_RopeLeaf<_CharT, _Alloc>* __l
>>>   	      = (_Rope_RopeLeaf<_CharT, _Alloc>*)this;
>>> -	    __l->_Rope_RopeLeaf<_CharT, _Alloc>::~_Rope_RopeLeaf();
>>> +	    __l->template _Rope_RopeLeaf<_CharT, _Alloc>::~_Rope_RopeLeaf();
>>>   	    _L_deallocate(__l, 1);
>>>   	    break;
>>>   	  }
>>>        
>> Hmm, this hurts my brain to have 'template ' in the middle of a dereference.
>> I also don't see why it should be needed.
>>      
> C++ seems to work this way... it's ugly yes, but correct
>    

+1. You do need to qualify the use of templates in situations like this.




More information about the svn-src-projects mailing list