[cfe-commits] r65639 - in /cfe/trunk: include/clang/Basic/DiagnosticParseKinds.def lib/Parse/Parser.cpp test/Lexer/block_cmt_end.c test/Parser/objc-foreach-error-1.m

Daniel Dunbar daniel at zuster.org
Fri Feb 27 09:20:17 PST 2009


Nice, I have always hated that particular diagnostic.

On Fri, Feb 27, 2009 at 9:15 AM, Chris Lattner <sabre at nondot.org> wrote:
> Author: lattner
> Date: Fri Feb 27 11:15:01 2009
> New Revision: 65639
>
> URL: http://llvm.org/viewvc/llvm-project?rev=65639&view=rev
> Log:
> change a diagnostic message from something pedantically correct but
> useless to something more vague but hopefully more clear.
> rdar://6624173
>
> Modified:
>    cfe/trunk/include/clang/Basic/DiagnosticParseKinds.def
>    cfe/trunk/lib/Parse/Parser.cpp
>    cfe/trunk/test/Lexer/block_cmt_end.c
>    cfe/trunk/test/Parser/objc-foreach-error-1.m
>
> Modified: cfe/trunk/include/clang/Basic/DiagnosticParseKinds.def
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticParseKinds.def?rev=65639&r1=65638&r2=65639&view=diff
>
> ==============================================================================
> --- cfe/trunk/include/clang/Basic/DiagnosticParseKinds.def (original)
> +++ cfe/trunk/include/clang/Basic/DiagnosticParseKinds.def Fri Feb 27 11:15:01 2009
> @@ -118,8 +118,8 @@
>      "expected function body after function declarator")
>  DIAG(err_expected_method_body, ERROR,
>      "expected method body")
> -DIAG(err_expected_after_declarator, ERROR,
> -     "expected '=', ',', ';', 'asm', or '__attribute__' after declarator")
> +DIAG(err_invalid_token_after_toplevel_declarator, ERROR,
> +     "invalid token after top level declarator")
>  DIAG(err_expected_statement, ERROR,
>      "expected statement")
>  DIAG(err_expected_lparen_after, ERROR,
>
> Modified: cfe/trunk/lib/Parse/Parser.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Parse/Parser.cpp?rev=65639&r1=65638&r2=65639&view=diff
>
> ==============================================================================
> --- cfe/trunk/lib/Parse/Parser.cpp (original)
> +++ cfe/trunk/lib/Parse/Parser.cpp Fri Feb 27 11:15:01 2009
> @@ -499,8 +499,8 @@
>     // FALL THROUGH.
>   } else if (DeclaratorInfo.isFunctionDeclarator() &&
>              (Tok.is(tok::l_brace) ||             // int X() {}
> -              ( !getLang().CPlusPlus &&
> -                isDeclarationSpecifier() ))) {    // int X(f) int f; {}
> +              (!getLang().CPlusPlus &&
> +               isDeclarationSpecifier()))) {    // int X(f) int f; {}
>     if (DS.getStorageClassSpec() == DeclSpec::SCS_typedef) {
>       Diag(Tok, diag::err_function_declared_typedef);
>
> @@ -520,7 +520,7 @@
>     if (DeclaratorInfo.isFunctionDeclarator())
>       Diag(Tok, diag::err_expected_fn_body);
>     else
> -      Diag(Tok, diag::err_expected_after_declarator);
> +      Diag(Tok, diag::err_invalid_token_after_toplevel_declarator);
>     SkipUntil(tok::semi);
>     return 0;
>   }
>
> Modified: cfe/trunk/test/Lexer/block_cmt_end.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Lexer/block_cmt_end.c?rev=65639&r1=65638&r2=65639&view=diff
>
> ==============================================================================
> --- cfe/trunk/test/Lexer/block_cmt_end.c (original)
> +++ cfe/trunk/test/Lexer/block_cmt_end.c Fri Feb 27 11:15:01 2009
> @@ -26,7 +26,7 @@
>  /* expected-warning {{escaped newline between}}   expected-warning {{backslash and newline separated by space}}    expected-warning {{trigraph ends block comment}}   *??/
>  /
>
> -foo /* expected-error {{expected '=', ',', ';', 'asm', or '__attribute__' after declarator}} */
> +foo /* expected-error {{invalid token after top level declarator}} */
>
>
>  // rdar://6060752 - We should not get warnings about trigraphs in comments:
>
> Modified: cfe/trunk/test/Parser/objc-foreach-error-1.m
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Parser/objc-foreach-error-1.m?rev=65639&r1=65638&r2=65639&view=diff
>
> ==============================================================================
> --- cfe/trunk/test/Parser/objc-foreach-error-1.m (original)
> +++ cfe/trunk/test/Parser/objc-foreach-error-1.m Fri Feb 27 11:15:01 2009
> @@ -1,6 +1,6 @@
>  // RUN: clang -fsyntax-only -verify %s
>
> -ce MyList // expected-error {{expected '=', ',', ';', 'asm', or '__attribute__' after declarator}}
> +ce MyList // expected-error {{invalid token after top level declarator}}
>  @end
>
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>




More information about the cfe-commits mailing list