[cfe-commits] r164559 - in /cfe/trunk: include/clang/Basic/ test/FixIt/ test/SemaObjC/ test/SemaObjCXX/

[jahanian]

On Sep 24, 2012, at 3:08 PM, Jordan Rose <jordan_rose at apple.com> wrote:

> 
> On Sep 24, 2012, at 15:00 , Fariborz Jahanian <fjahanian at apple.com> wrote:
> 
>> 
>> def err_misplaced_ivar : Error<
>> -  "ivars may not be placed in %select{categories|class extension}0">;
>> +  "instance variable may not be placed in %select{categories|class extension}0">;
> 
> 
>> def err_arc_autoreleasing_var : Error<
>> -  "%select{__block variables|global variables|fields|ivars}0 cannot have "
>> +  "%select{__block variables|global variables|fields|instance variable}0 cannot have "
>>  "__autoreleasing ownership">;
> 
> I'm not sure if the plural was changed to singular intentionally here, but I want to say I prefer the plural. In the singular form, I feel like there's something wrong with this particular instance variable that maybe I could fix, but what the warning is trying to say is that there's a rule that applies to all instance variables.

Makes sense. Will change shortly. 
- fariborz
> 
> Jordan