[cfe-commits] r109024 - in /cfe/trunk/lib: Rewrite/RewriteObjC.cpp Sema/SemaInit.cpp Sema/SemaInit.h

Fariborz Jahanian fjahanian at apple.com
Wed Jul 21 11:41:23 PDT 2010


On Jul 21, 2010, at 11:35 AM, Douglas Gregor wrote:

>>
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =====================================================================
>> --- cfe/trunk/lib/Sema/SemaInit.cpp (original)
>> +++ cfe/trunk/lib/Sema/SemaInit.cpp Wed Jul 21 13:31:47 2010
>> @@ -3803,7 +3803,8 @@
>>
>>      // Build a call to the selected constructor.
>>      ASTOwningVector<&ActionBase::DeleteExpr> ConstructorArgs(S);
>> -      SourceLocation Loc = Kind.getLocation();
>> +      SourceLocation Loc = Kind.isCopyInit() ? Kind.getEqualLoc()
>> +                           : Kind.getLocation();
>
> Copy-initialization doesn't always involve an '=' sign, so you  
> probably want to check Kind.isCopyInit() &&  
> Kind.getEqualLoc().isValid().
Thanks. In r109025.

- fariborz

>
>
> 	- Doug
>




More information about the cfe-commits mailing list