[cfe-commits] r164047 - in /cfe/trunk: include/clang/Basic/DiagnosticParseKinds.td lib/Parse/ParseObjc.cpp test/SemaObjC/unused.m test/SemaObjC/warning-missing-selector-name.m

Richard Smith richard at metafoo.co.uk
Mon Sep 17 15:38:19 PDT 2012


On Mon, Sep 17, 2012 at 12:15 PM, Fariborz Jahanian <fjahanian at apple.com>wrote:

> Author: fjahanian
> Date: Mon Sep 17 14:15:26 2012
> New Revision: 164047
>
> URL: http://llvm.org/viewvc/llvm-project?rev=164047&view=rev
> Log:
> objective-C: issue warning when there is no whitespace
> between objc method parameter name and colon.
> // rdar://12263549
>
> Added:
>     cfe/trunk/test/SemaObjC/warning-missing-selector-name.m
> Modified:
>     cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
>     cfe/trunk/lib/Parse/ParseObjc.cpp
>     cfe/trunk/test/SemaObjC/unused.m
>
> Modified: cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td?rev=164047&r1=164046&r2=164047&view=diff
>
> ==============================================================================
> --- cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td (original)
> +++ cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td Mon Sep 17
> 14:15:26 2012
> @@ -213,6 +213,12 @@
>    "expected ';' after static_assert">;
>  def err_expected_semi_for : Error<"expected ';' in 'for' statement
> specifier">;
>  def err_expected_colon_after : Error<"expected ':' after %0">;
> +def missing_selector_name : Warning<
> +  "parameter name used as selector"
> +  " may result in incomplete method selector name">,
> +  InGroup<DiagGroup<"missing-argument-name-in-selector">>;
>

You're missing the "warn_" from this warning name.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120917/2f8cbfac/attachment.html>


More information about the cfe-commits mailing list