[cfe-commits] r59604 - /cfe/trunk/lib/Parse/ParseObjc.cpp

[Chris Lattner]

On Nov 19, 2008, at 2:25 AM, Daniel Dunbar wrote:

> On Tue, Nov 18, 2008 at 11:41 PM, Chris Lattner <sabre at nondot.org>  
> wrote:
>> +  llvm::SmallString<100> SelectorName;
>> +  SelectorName += "set";
>
> This '+=' is confusing, '=' is ok?

Is it really confusing?  SmallString doesn't have operator= for  
input.  I could add that of course, but I don't really find this  
confusing.  To me the worse bit is that there is a copy of this code  
in ParseObjC.cpp and in SemaExpr.cpp. :(

-Chris