r205024 - Objective-C. revert r204965. This will make
jahanian
fjahanian at apple.com
Fri Mar 28 16:16:07 PDT 2014
Thanks Nico for our quick feedback. Our thinking is currently to allow extra paren around the selector name
to suppress the warning. In addition, the warning will have the fix-it to suggest how to suppress the warning.
For example:
@selector((“selector”))
This is precedented by -Wparenthesis:
if ((p = foo()) { … }
In the meantime, I will revert the revert till I get the time to implement this (or any other proposal).
- Fariborz
On Mar 28, 2014, at 11:59 AM, Nico Weber <thakis at chromium.org> wrote:
> Can you share the idea for the fix-it? And maybe this shouldn't be on by default until the fix-it is implemented?
>
> As is, this will warn if someone writes @selector(length) and includes Cocoa, which doesn't seem very unlikely – and warning on that without a workaround doesn't seem useful.
>
> (Also, it'd be nice if the discussion was on-list.)
>
>
More information about the cfe-commits
mailing list