[cfe-commits] r172110 - in /cfe/trunk: lib/Format/Format.cpp unittests/Format/FormatTest.cpp

Nico Weber thakis at chromium.org
Thu Jan 10 15:39:23 PST 2013


On Thu, Jan 10, 2013 at 1:50 PM, Jordan Rose <jordan_rose at apple.com> wrote:

>
> On Jan 10, 2013, at 13:45 , Nico Weber <thakis at chromium.org> wrote:
>
> On Thu, Jan 10, 2013 at 1:43 PM, Jordan Rose <jordan_rose at apple.com>wrote:
>
>> Looks like spaces are correct for @synthesize:
>>
>>
>> https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/ProgrammingWithObjectiveC/EncapsulatingData/EncapsulatingData.html#//apple_ref/doc/uid/TP40011210-CH5-SW6
>>
>> @implementation YourClass
>>
>> @synthesize propertyName = instanceVariableName;
>>
>> ...
>>
>> @end
>>
>>
>>
> https://developer.apple.com/library/mac/#documentation/Cocoa/Conceptual/CodingGuidelines/Articles/NamingIvarsAndTypes.htmlhas
>
>  @synthesize showsTitle=_showsTitle;
>
> though :-)
>
>
> Hm. Fair enough. I would still go with spaces in this case. (The first
> document has been updated more recently, and it seems to be the majority on
> developer.apple.com. Also I know the guy who wrote it.)
>

Sounds good. I removed the FIXME in r172141, thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130110/f6304a2b/attachment.html>


More information about the cfe-commits mailing list