Sent from my iPhone

On Mar 5, 2013, at 3:53 PM, Jordan Rose <[email protected]> wrote:

> 
> On Mar 5, 2013, at 15:51 , jahanian <[email protected]> wrote:
> 
>> 
>> On Mar 5, 2013, at 3:37 PM, Jordan Rose <[email protected]> wrote:
>> 
>>> I hate to continue bikeshedding here, but why is this not just a single 
>>> warning with a %select and associated enum { Function, Callback, Method }?
>>> 
>>> Also, "Objective-C" should not be lowercase.
>> 
>> No particular reason. I see that objective-c is being used throughout clang 
>> source in comments.
> 
> I meant the diagnostic text:
> 
> +def warn_doc_method_not_attached_to_a_objc_method_decl : Warning<
> +  "'%select{\\|@}0method' command should be used in a comment attached to an 
> "
> +  "objective-c method declaration">,
> 
> 
> I'm not really happy about it being lowercase in the source either, but 
> you're right that that's hardly uncommon.

Still, we should just fix it. The language is spelled Objective-C. 
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to