On Nov 19, 2008, at 2:25 AM, Daniel Dunbar wrote: > On Tue, Nov 18, 2008 at 11:41 PM, Chris Lattner <[EMAIL PROTECTED]> > 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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
