On Jun 10, 2013, at 17:12 , Arthur O'Dwyer <[email protected]> wrote:
> On Mon, Jun 10, 2013 at 4:51 PM, Fariborz Jahanian <[email protected]> > wrote: >> >> +def warn_missing_atsign_prefix : Warning< >> + "string literal must be prefixed by '@' ">, >> InGroup<ObjCLiteralMissingAtSign>; > > Shouldn't this say "NSString literal", not "string literal"? > Objective-C has the same > string literals as C and C++, and they don't need to be prefixed by > '@'. I feel like > overriding the term "string literal" in this message to refer > specifically to NSString > literals is actually more confusing than it is helpful. I thought about this in looking at the patch, but it doesn't match with similar diagnostics about "array literals" and "dictionary literals". Moreover, it's not actually guaranteed that the constant string class is called "NSString". If we wanted this, we could either look up the string class name, or refer to "Objective-C string literal". Jordan
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
