[cfe-commits] r85060 - in /cfe/trunk: include/clang/Parse/Action.h lib/Sema/SemaDeclCXX.cpp

Chris Lattner clattner at apple.com
Sun Oct 25 14:19:20 PDT 2009


On Oct 25, 2009, at 1:13 PM, John McCall wrote:
>> @@ -4282,9 +4281,11 @@
>>   // Try to convert the decl specifier to a type.  This works for
>>   // friend templates because ActOnTag never produces a  
>> ClassTemplateDecl
>>   // for a TUK_Friend.
>> -  bool invalid = false;
>> -  QualType T = ConvertDeclSpecToType(DS, Loc, invalid);
>> -  if (invalid) return DeclPtrTy();
>> +  Declarator TheDeclarator(DS, Declarator::MemberContext);
>> +  // TODO: Should use D.SetIdentifier() to specify where the  
>> identifier is?
>> +  QualType T = GetTypeForDeclarator(TheDeclarator, S);
>> +  if (TheDeclarator.isInvalidType())
>> +    return DeclPtrTy()
>
> I'm not sure what you're getting at with this TODO;  there is never an
> identifier in these declarations.

Duh, right.  Removed, thanks!

-Chris



More information about the cfe-commits mailing list