2008-11-05 Francisco Figueiredo Jr. <francisco@npgsql.org>
[mono.git] / mcs / mcs / cs-parser.jay
index e62e03d1f8e71c5c10ffde2c2301016e3549c708..82665c5e9d1b92c107b51f89c22c0e58908ec6ab 100644 (file)
@@ -148,7 +148,6 @@ namespace Mono.CSharp
 %token ABSTRACT        
 %token AS
 %token ADD
-%token ASSEMBLY
 %token BASE    
 %token BOOL    
 %token BREAK   
@@ -229,7 +228,6 @@ namespace Mono.CSharp
 %token ARGLIST
 %token PARTIAL
 %token ARROW
-%token QUERY_FIRST_TOKEN
 %token FROM
 %token JOIN
 %token ON
@@ -242,8 +240,13 @@ namespace Mono.CSharp
 %token ASCENDING
 %token DESCENDING
 %token INTO
-%token QUERY_LAST_TOKEN
 %token INTERR_NULLABLE
+%token EXTERN_ALIAS
+
+/* Generics <,> tokens */
+%token OP_GENERICS_LT
+%token OP_GENERICS_LT_DECL
+%token OP_GENERICS_GT
 
 /* C# keywords which are not really keywords */
 %token GET
@@ -258,6 +261,7 @@ namespace Mono.CSharp
 %token CLOSE_BRACKET
 %token OPEN_PARENS
 %token CLOSE_PARENS
+
 %token DOT
 %token COMMA
 %token COLON
@@ -269,10 +273,7 @@ namespace Mono.CSharp
 %token BANG
 %token ASSIGN
 %token OP_LT
-%token OP_GENERICS_LT
-%token OP_GENERICS_LT_DECL
 %token OP_GT
-%token OP_GENERICS_GT
 %token BITWISE_AND
 %token BITWISE_OR
 %token STAR
@@ -316,11 +317,7 @@ namespace Mono.CSharp
 
 %token IDENTIFIER
 %token OPEN_PARENS_LAMBDA
-%token CLOSE_PARENS_CAST
-%token CLOSE_PARENS_NO_CAST
-%token CLOSE_PARENS_OPEN_PARENS
-%token CLOSE_PARENS_MINUS
-%token DEFAULT_OPEN_PARENS
+%token OPEN_PARENS_CAST
 %token GENERIC_DIMENSION
 %token DEFAULT_COLON
 
@@ -387,12 +384,12 @@ extern_alias_directives
        ;
 
 extern_alias_directive
-       : EXTERN IDENTIFIER IDENTIFIER SEMICOLON
+       : EXTERN_ALIAS IDENTIFIER IDENTIFIER SEMICOLON
          {
                LocatedToken lt = (LocatedToken) $2;
                string s = lt.Value;
                if (s != "alias"){
-                       Report.Error (1003, lt.Location, "'alias' expected");
+                       syntax_error (lt.Location, "`alias' expected");
                } else if (RootContext.Version == LanguageVersion.ISO_1) {
                        Report.FeatureIsNotAvailable (lt.Location, "external alias");
                } else {
@@ -400,6 +397,10 @@ extern_alias_directive
                        current_namespace.AddUsingExternalAlias (lt.Value, lt.Location);
                }
          }
+       | EXTERN_ALIAS error
+         {
+               syntax_error (GetLocation ($1), "`alias' expected");   // TODO: better
+         }
        ;
  
 using_directives
@@ -529,7 +530,7 @@ namespace_body_body
          opt_namespace_member_declarations
          EOF
          {
-               Report.Error (1513, lexer.Location, "} expected");
+               Report.Error (1513, lexer.Location, "Expected `}'");
          }
        ;
 
@@ -580,8 +581,8 @@ namespace_member_declaration
 
 type_declaration
        : class_declaration             
-       | struct_declaration            
-       | interface_declaration         
+       | struct_declaration
+       | interface_declaration
        | enum_declaration              
        | delegate_declaration
 //
@@ -697,7 +698,7 @@ attribute_section
          {
                $$ = $3;
          }
-         | OPEN_BRACKET attribute_list opt_comma CLOSE_BRACKET
+       | OPEN_BRACKET attribute_list opt_comma CLOSE_BRACKET
          {
                $$ = $2;
          }
@@ -845,12 +846,12 @@ named_argument_list
 
                $$ = args;
          }
-         | named_argument_list COMMA expression
-           {
-                 Report.Error (1016, ((Expression) $3).Location, "Named attribute argument expected");
-                 $$ = null;
-               }
-        ;
+       | named_argument_list COMMA expression
+         {
+               Report.Error (1016, ((Expression) $3).Location, "Named attribute argument expected");
+               $$ = null;
+         }
+       ;
 
 named_argument
        : IDENTIFIER ASSIGN expression
@@ -889,6 +890,12 @@ class_member_declaration
        | constructor_declaration               // done
        | destructor_declaration                // done
        | type_declaration
+       | error
+         {
+               Report.Error (1519, lexer.Location, "Unexpected symbol `{0}' in class, struct, or interface member declaration",
+                       GetSymbolName (yyToken));
+               $$ = null;
+         }
        ;
 
 struct_declaration
@@ -916,6 +923,7 @@ struct_declaration
          }
          struct_body
          {
+               --lexer.parsing_declaration;      
                if (RootContext.Documentation != null)
                        Lexer.doc_state = XmlCommentState.Allowed;
          }
@@ -1151,10 +1159,8 @@ local_variable_declarator
          {
                $$ = new VariableDeclaration ((LocatedToken) $1, null);
          }
-       | IDENTIFIER OPEN_BRACKET opt_expression CLOSE_BRACKET
+       | IDENTIFIER variable_bad_array
          {
-               Report.Error (650, ((LocatedToken) $1).Location, "Syntax error, bad array declarator. To declare a managed array the rank specifier precedes the variable's identifier. " +
-                       "To declare a fixed size buffer field, use the fixed keyword before the field type");
                $$ = null;
          }
        ;
@@ -1162,7 +1168,7 @@ local_variable_declarator
 local_variable_initializer
        : expression
        | array_initializer
-       | STACKALLOC type_expression OPEN_BRACKET expression CLOSE_BRACKET
+       | STACKALLOC simple_type OPEN_BRACKET expression CLOSE_BRACKET
          {
                $$ = new StackAlloc ((Expression) $2, (Expression) $4, (Location) $1);
          }
@@ -1170,14 +1176,13 @@ local_variable_initializer
          {
                $$ = new ArglistAccess ((Location) $1);
          }
-       | STACKALLOC type
+       | STACKALLOC simple_type
          {
                Report.Error (1575, (Location) $1, "A stackalloc expression requires [] after type");
-                $$ = null;
+               $$ = new StackAlloc ((Expression) $2, null, (Location) $1);             
          }
        ;
 
-
 variable_declarators
        : variable_declarator 
          {
@@ -1210,14 +1215,20 @@ variable_declarator
                lexer.parsing_generic_declaration = false;
                $$ = new VariableMemberDeclaration ((MemberName) $1, null);
          }
-       | member_declaration_name OPEN_BRACKET opt_expression CLOSE_BRACKET
+       | member_declaration_name variable_bad_array
          {
                lexer.parsing_generic_declaration = false;        
-               Report.Error (650, ((LocatedToken) $1).Location, "Syntax error, bad array declarator. To declare a managed array the rank specifier precedes the variable's identifier. " +
-                       "To declare a fixed size buffer field, use the fixed keyword before the field type");
                $$ = null;
          }
        ;
+       
+variable_bad_array
+       : OPEN_BRACKET opt_expression CLOSE_BRACKET
+         {
+               Report.Error (650, GetLocation ($1), "Syntax error, bad array declarator. To declare a managed array the rank specifier precedes the variable's identifier. " +
+                       "To declare a fixed size buffer field, use the fixed keyword before the field type");
+         }
+       ;
 
 variable_initializer
        : expression
@@ -1497,7 +1508,7 @@ fixed_parameter
          type
          {
                Report.Error (1001, GetLocation ($3), "Identifier expected");
-               $$ = null;
+               $$ = new Parameter ((FullNamedExpression) $3, "NeedSomeGeneratorHere", (Parameter.Modifier) $2, (Attributes) $1, lexer.Location);               
          }
        | opt_attributes
          opt_parameter_modifier
@@ -1700,8 +1711,16 @@ accessor_declarations
         }
        | error
          {
-               Report.Error (1014, GetLocation ($1), "A get or set accessor expected");
-               $$ = null;
+               if (yyToken == Token.CLOSE_BRACE) {
+                       $$ = null;
+               } else {
+                       if (yyToken == Token.SEMICOLON)
+                               Report.Error (1597, lexer.Location, "Semicolon after method or accessor block is not valid");
+                       else
+                               Report.Error (1014, GetLocation ($1), "A get or set accessor expected");
+
+                       $$ = new Accessors (null, null);
+               }
          }
        ;
 
@@ -1719,7 +1738,7 @@ get_accessor_declaration
          accessor_body
          {
                if (has_get) {
-                       Report.Error (1007, (Location) $3, "Property accessor already defined");
+                       Report.Error (1007, GetLocation ($3), "Property accessor already defined");
                        break;
                }
                Accessor accessor = new Accessor ((ToplevelBlock) $5, (int) $2, (Attributes) $1, current_local_parameters, (Location) $3);
@@ -1754,7 +1773,7 @@ set_accessor_declaration
          accessor_body
          {
                if (has_set) {
-                       Report.Error (1007, ((LocatedToken) $3).Location, "Property accessor already defined");
+                       Report.Error (1007, GetLocation ($3), "Property accessor already defined");
                        break;
                }
                Accessor accessor = new Accessor ((ToplevelBlock) $5, (int) $2, (Attributes) $1, current_local_parameters, (Location) $3);
@@ -1772,7 +1791,15 @@ set_accessor_declaration
 
 accessor_body
        : block 
-       | SEMICOLON             { $$ = null; }
+       | SEMICOLON
+         {
+               $$ = null;
+         }
+       | error
+         {
+               Error_SyntaxError (1043, yyToken);
+               $$ = null;
+         }
        ;
 
 interface_declaration
@@ -1802,6 +1829,7 @@ interface_declaration
          }
          interface_body
          {
+               --lexer.parsing_declaration;      
                if (RootContext.Documentation != null)
                        Lexer.doc_state = XmlCommentState.Allowed;
          }
@@ -2012,11 +2040,13 @@ conversion_operator_declarator
          }
        | IMPLICIT error 
          {
-               syntax_error ((Location) $1, "'operator' expected");
+               Error_SyntaxError (yyToken);
+               $$ = new OperatorDeclaration (Operator.OpType.Implicit, null, GetLocation ($1));
          }
        | EXPLICIT error 
          {
-               syntax_error ((Location) $1, "'operator' expected");
+               Error_SyntaxError (yyToken);
+               $$ = new OperatorDeclaration (Operator.OpType.Explicit, null, GetLocation ($1));
          }
        ;
 
@@ -2241,7 +2271,7 @@ event_declaration
                if (RootContext.Documentation != null)
                        Lexer.doc_state = XmlCommentState.Allowed;
 
-               Report.Error (1002, GetLocation ($6), "Expecting {0}, got {1}", GetExpecting (), GetTokenName (yyToken));
+               Error_SyntaxError (yyToken);
                $$ = null;
          }
        ;
@@ -2489,6 +2519,7 @@ enum_member_declaration
          }
        | opt_attributes IDENTIFIER
          {
+               ++lexer.parsing_block;
                if (RootContext.Documentation != null) {
                        tmpComment = Lexer.consume_doc_comment ();
                        Lexer.doc_state = XmlCommentState.NotAllowed;
@@ -2496,6 +2527,7 @@ enum_member_declaration
          }
          ASSIGN constant_expression
          { 
+               --lexer.parsing_block;    
                VariableDeclaration vd = new VariableDeclaration (
                        (LocatedToken) $2, $5, (Attributes) $1);
 
@@ -2598,7 +2630,7 @@ opt_type_argument_list
        | OP_GENERICS_LT error
          {
                Error_TypeExpected (lexer.Location);
-               $$ = new TypeParameterName ("", null, lexer.Location);
+               $$ = new TypeArguments (lexer.Location);
          }
        ;
 
@@ -2720,10 +2752,10 @@ type_parameter
          }
        | error
          {
-               if (GetTokenName (yyToken) == "<type>")
+               if (GetTokenName (yyToken) == "type")
                        Report.Error (81, GetLocation ($1), "Type parameter declaration must be an identifier not a type");
                else
-                       Report.Error (1002, GetLocation ($1), "Expecting {0}, got {1}", GetExpecting (), GetTokenName (yyToken));
+                       Error_SyntaxError (yyToken);
                        
                $$ = new TypeParameterName ("", null, lexer.Location);
          }
@@ -2758,10 +2790,23 @@ type
                $$ = TypeManager.system_void_expr;
          }     
        ;
+       
+simple_type
+       : type_expression
+       | VOID
+         {
+               Expression.Error_VoidInvalidInTheContext (lexer.Location);
+               $$ = TypeManager.system_void_expr;
+         }     
+       ;               
 
 type_expression_or_array
        : type_expression
-       | array_type
+       | type_expression rank_specifiers
+         {
+               string rank_specifiers = (string) $2;
+               $$ = current_array_type = new ComposedCast ((FullNamedExpression) $1, rank_specifiers);
+         }
        ;
        
 type_expression
@@ -2799,44 +2844,6 @@ type_expression
          }     
        ;
 
-non_expression_type
-       : builtin_types opt_nullable
-         {
-               if ($2 != null)
-                       $$ = new ComposedCast ((FullNamedExpression) $1, "?", lexer.Location);
-         }
-       | VOID
-         {
-               $$ = TypeManager.system_void_expr;
-         }
-       | non_expression_type rank_specifiers
-         {
-               Location loc = GetLocation ($1);
-               if (loc.IsNull)
-                       loc = lexer.Location;
-               $$ = new ComposedCast ((FullNamedExpression) $1, (string) $2, loc);
-         }
-       | non_expression_type STAR
-         {
-               Location loc = GetLocation ($1);
-               if (loc.IsNull)
-                       loc = lexer.Location;
-               $$ = new ComposedCast ((FullNamedExpression) $1, "*", loc);
-         }
-       //
-       // We need this because the parser will happily go and reduce IDENTIFIER STAR
-       // through this different path
-       //
-       | multiplicative_expression STAR 
-         {
-               FullNamedExpression e = $1 as FullNamedExpression;
-               if (e != null)
-                       $$ = new ComposedCast (e, "*");
-               else
-                       Error_TypeExpected (GetLocation ($1));
-         }
-       ;
-
 type_list
        : base_type_name
          {
@@ -2867,7 +2874,7 @@ base_type_name
        
 /*
  * replaces all the productions for isolating the various
- * simple types, but we need this to reuse it easily in local_variable_type
+ * simple types, but we need this to reuse it easily in variable_type
  */
 builtin_types
        : OBJECT        { $$ = TypeManager.system_object_expr; }
@@ -2891,14 +2898,6 @@ integral_type
        | CHAR          { $$ = TypeManager.system_char_expr; }
        ;
 
-array_type
-       : type_expression rank_specifiers
-         {
-               string rank_specifiers = (string) $2;
-               $$ = current_array_type = new ComposedCast ((FullNamedExpression) $1, rank_specifiers);
-         }
-       ;
-
 predefined_type
        : builtin_types
        | VOID
@@ -2980,36 +2979,25 @@ boolean_literal
        | FALSE                 { $$ = new BoolLiteral (false, lexer.Location); }
        ;
 
-parenthesized_expression_0
-       : OPEN_PARENS expression CLOSE_PARENS
-         {
-               $$ = $2;
-               lexer.Deambiguate_CloseParens ($$);
-               // After this, the next token returned is one of
-               // CLOSE_PARENS_CAST, CLOSE_PARENS_NO_CAST (CLOSE_PARENS), CLOSE_PARENS_OPEN_PARENS
-               // or CLOSE_PARENS_MINUS.
-         }
-       | OPEN_PARENS expression error { CheckToken (1026, yyToken, "Expecting ')'", lexer.Location); }
+
+//
+// Here is the trick, tokenizer may think that parens is a special but
+// parser is interested in open parens only, so we merge them.
+// Consider: if (a)foo ();
+//
+open_parens_any
+       : OPEN_PARENS
+       | OPEN_PARENS_CAST
+       | OPEN_PARENS_LAMBDA
        ;
 
 parenthesized_expression
-       : parenthesized_expression_0 CLOSE_PARENS_NO_CAST
-         {
-               $$ = $1;
-         }  
-       | parenthesized_expression_0 CLOSE_PARENS
-         {
-               $$ = $1;
-         }
-       | parenthesized_expression_0 CLOSE_PARENS_MINUS
+       : OPEN_PARENS expression CLOSE_PARENS
          {
-               // If a parenthesized expression is followed by a minus, we need to wrap
-               // the expression inside a ParenthesizedExpression for the CS0075 check
-               // in Binary.DoResolve().
-               $$ = new ParenthesizedExpression ((Expression) $1);
+               $$ = new ParenthesizedExpression ((Expression) $2);
          }
        ;
-
+       
 member_access
        : primary_expression DOT IDENTIFIER opt_type_argument_list
          {
@@ -3032,32 +3020,9 @@ member_access
        ;
 
 invocation_expression
-       : primary_expression OPEN_PARENS opt_argument_list CLOSE_PARENS
-         {
-               if ($1 == null)
-                       Report.Error (1, (Location) $2, "Parse error");
-               else
-                       $$ = new Invocation ((Expression) $1, (ArrayList) $3);
-         }
-       | parenthesized_expression_0 CLOSE_PARENS_OPEN_PARENS OPEN_PARENS CLOSE_PARENS
+       : primary_expression open_parens_any opt_argument_list CLOSE_PARENS
          {
-               $$ = new Invocation ((Expression) $1, new ArrayList ());
-         }
-       | parenthesized_expression_0 CLOSE_PARENS_OPEN_PARENS primary_expression
-         {
-               $$ = new InvocationOrCast ((Expression) $1, (Expression) $3);
-         }
-       | parenthesized_expression_0 CLOSE_PARENS_OPEN_PARENS OPEN_PARENS non_simple_argument CLOSE_PARENS
-         {
-               ArrayList args = new ArrayList (1);
-               args.Add ($4);
-               $$ = new Invocation ((Expression) $1, args);
-         }
-       | parenthesized_expression_0 CLOSE_PARENS_OPEN_PARENS OPEN_PARENS argument_list COMMA argument CLOSE_PARENS
-         {
-               ArrayList args = ((ArrayList) $4);
-               args.Add ($6);
-               $$ = new Invocation ((Expression) $1, args);
+               $$ = new Invocation ((Expression) $1, (ArrayList) $3);
          }
        ;
 
@@ -3178,7 +3143,7 @@ non_simple_argument
          { 
                $$ = new Argument ((Expression) $2, Argument.AType.Out);
          }
-       | ARGLIST OPEN_PARENS argument_list CLOSE_PARENS
+       | ARGLIST open_parens_any argument_list CLOSE_PARENS
          {
                ArrayList list = (ArrayList) $3;
                Argument[] args = new Argument [list.Count];
@@ -3187,7 +3152,7 @@ non_simple_argument
                Expression expr = new Arglist (args, (Location) $1);
                $$ = new Argument (expr, Argument.AType.Expression);
          }
-       | ARGLIST OPEN_PARENS CLOSE_PARENS
+       | ARGLIST open_parens_any CLOSE_PARENS
          {
                $$ = new Argument (new Arglist ((Location) $1), Argument.AType.Expression);
          }       
@@ -3279,21 +3244,19 @@ base_access
 post_increment_expression
        : primary_expression OP_INC
          {
-               $$ = new UnaryMutator (UnaryMutator.Mode.PostIncrement,
-                                      (Expression) $1, (Location) $2);
+               $$ = new UnaryMutator (UnaryMutator.Mode.PostIncrement, (Expression) $1);
          }
        ;
 
 post_decrement_expression
        : primary_expression OP_DEC
          {
-               $$ = new UnaryMutator (UnaryMutator.Mode.PostDecrement,
-                                      (Expression) $1, (Location) $2);
+               $$ = new UnaryMutator (UnaryMutator.Mode.PostDecrement, (Expression) $1);
          }
        ;
 
 object_or_delegate_creation_expression
-       : NEW type OPEN_PARENS opt_argument_list CLOSE_PARENS opt_object_or_collection_initializer
+       : NEW simple_type open_parens_any opt_argument_list CLOSE_PARENS opt_object_or_collection_initializer
          {
                if ($6 != null) {
                        if (RootContext.Version <= LanguageVersion.ISO_2)
@@ -3304,7 +3267,7 @@ object_or_delegate_creation_expression
                else
                        $$ = new New ((Expression) $2, (ArrayList) $4, (Location) $1);
          }
-       | NEW type object_or_collection_initializer
+       | NEW simple_type object_or_collection_initializer
          {
                if (RootContext.Version <= LanguageVersion.ISO_2)
                        Report.FeatureIsNotAvailable (GetLocation ($1), "collection initializers");
@@ -3313,27 +3276,26 @@ object_or_delegate_creation_expression
          }
        ;
 
+
 array_creation_expression
-       : NEW type_expression OPEN_BRACKET expression_list CLOSE_BRACKET 
-         opt_rank_specifier
+       : NEW simple_type OPEN_BRACKET expression_list CLOSE_BRACKET 
+         opt_rank_specifier    // shift/reduce on OPEN_BRACE
          opt_array_initializer
          {
                $$ = new ArrayCreation ((FullNamedExpression) $2, (ArrayList) $4, (string) $6, (ArrayList) $7, (Location) $1);
          }
-       | NEW type_expression rank_specifiers array_initializer
+       | NEW simple_type rank_specifiers opt_array_initializer
          {
+               if ($4 == null)
+                       Report.Error (1586, GetLocation ($1), "Array creation must have array size or array initializer");
+
                $$ = new ArrayCreation ((FullNamedExpression) $2, (string) $3, (ArrayList) $4, (Location) $1);
          }
        | NEW rank_specifiers array_initializer
          {
                $$ = new ImplicitlyTypedArrayCreation ((string) $2, (ArrayList) $3, (Location) $1);
          }
-       | NEW error
-         {
-               Report.Error (1031, (Location) $1, "Type expected");
-               $$ = null;
-         }          
-       | NEW type_expression error
+       | NEW simple_type error
          {
                Report.Error (1526, (Location) $1, "A new expression requires () or [] after type");
                $$ = null;
@@ -3408,11 +3370,11 @@ anonymous_type_parameter
 opt_rank_specifier
        : /* empty */
          {
-                 $$ = "";
+               $$ = "";
          }
        | rank_specifiers
          {
-                       $$ = $1;
+               $$ = $1;
          }
        ;
 
@@ -3434,28 +3396,27 @@ opt_rank_specifier_or_nullable
        ;
 
 rank_specifiers
-       : rank_specifier opt_rank_specifier
+       : rank_specifier
+       | rank_specifier rank_specifiers
          {
                $$ = (string) $2 + (string) $1;
          }
        ;
 
 rank_specifier
-       : OPEN_BRACKET opt_dim_separators CLOSE_BRACKET
+       : OPEN_BRACKET CLOSE_BRACKET
          {
-               $$ = "[" + (string) $2 + "]";
+               $$ = "[]";
          }
-       ;
-
-opt_dim_separators
-       : /* empty */
+       | OPEN_BRACKET dim_separators CLOSE_BRACKET
          {
-               $$ = "";
+               $$ = "[" + (string) $2 + "]";
          }
-       | dim_separators
+       | OPEN_BRACKET error CLOSE_BRACKET
          {
-                 $$ = $1;
-         }               
+               ArrayCreation.Error_IncorrectArrayInitializer (GetLocation ($1));
+               $$ = "[]";
+         }
        ;
 
 dim_separators
@@ -3505,6 +3466,11 @@ variable_initializer_list
                list.Add ($3);
                $$ = list;
          }
+       | error
+         {
+               Error_SyntaxError (yyToken);
+               $$ = new ArrayList ();
+         }
        ;
 
 typeof_expression
@@ -3513,7 +3479,7 @@ typeof_expression
                pushed_current_array_type = current_array_type;
                lexer.TypeOfParsing = true;
          }
-         OPEN_PARENS typeof_type_expression CLOSE_PARENS
+         open_parens_any typeof_type_expression CLOSE_PARENS
          {
                lexer.TypeOfParsing = false;
                Expression type = (Expression)$4;
@@ -3595,20 +3561,20 @@ qualified_alias_member
        ;
 
 sizeof_expression
-       : SIZEOF OPEN_PARENS type CLOSE_PARENS { 
+       : SIZEOF open_parens_any type CLOSE_PARENS { 
                $$ = new SizeOf ((Expression) $3, (Location) $1);
          }
        ;
 
 checked_expression
-       : CHECKED OPEN_PARENS expression CLOSE_PARENS
+       : CHECKED open_parens_any expression CLOSE_PARENS
          {
                $$ = new CheckedExpr ((Expression) $3, (Location) $1);
          }
        ;
 
 unchecked_expression
-       : UNCHECKED OPEN_PARENS expression CLOSE_PARENS
+       : UNCHECKED open_parens_any expression CLOSE_PARENS
          {
                $$ = new UnCheckedExpr ((Expression) $3, (Location) $1);
          }
@@ -3632,7 +3598,7 @@ anonymous_method_expression
          }
          block
          {
-               $$ = end_anonymous ((ToplevelBlock) $4, (Location) $1);
+               $$ = end_anonymous ((ToplevelBlock) $4);
        }
        ;
 
@@ -3654,12 +3620,12 @@ anonymous_method_signature
        ;
 
 default_value_expression
-       : DEFAULT_OPEN_PARENS type CLOSE_PARENS
+       : DEFAULT open_parens_any type CLOSE_PARENS
          {
                if (RootContext.Version < LanguageVersion.ISO_2)
                        Report.FeatureIsNotAvailable (lexer.Location, "default value expression");
 
-               $$ = new DefaultValueExpression ((Expression) $2, lexer.Location);
+               $$ = new DefaultValueExpression ((Expression) $3, GetLocation ($1));
          }
        ;
 
@@ -3667,39 +3633,23 @@ unary_expression
        : primary_expression
        | BANG prefixed_unary_expression
          {
-               $$ = new Unary (Unary.Operator.LogicalNot, (Expression) $2, (Location) $1);
+               $$ = new Unary (Unary.Operator.LogicalNot, (Expression) $2);
          }
        | TILDE prefixed_unary_expression
          {
-               $$ = new Unary (Unary.Operator.OnesComplement, (Expression) $2, (Location) $1);
+               $$ = new Unary (Unary.Operator.OnesComplement, (Expression) $2);
          }
        | cast_expression
        ;
 
-cast_list
-       : parenthesized_expression_0 CLOSE_PARENS_CAST unary_expression
+cast_expression
+       : OPEN_PARENS_CAST type CLOSE_PARENS prefixed_unary_expression
          {
-               $$ = new Cast ((Expression) $1, (Expression) $3);
+               $$ = new Cast ((Expression) $2, (Expression) $4, GetLocation ($1));
          }
-       | parenthesized_expression_0 CLOSE_PARENS_NO_CAST default_value_expression
+       | OPEN_PARENS predefined_type CLOSE_PARENS prefixed_unary_expression
          {
-               $$ = new Cast ((Expression) $1, (Expression) $3);
-         }
-       | parenthesized_expression_0 CLOSE_PARENS_OPEN_PARENS cast_expression
-         {
-               $$ = new Cast ((Expression) $1, (Expression) $3);
-         }     
-       ;
-
-cast_expression
-       : cast_list
-       | OPEN_PARENS non_expression_type CLOSE_PARENS prefixed_unary_expression
-         {
-               if ($2 == TypeManager.system_void_expr)
-                       Expression.Error_VoidInvalidInTheContext (GetLocation ($2));
-
-               // TODO: wrong location
-               $$ = new Cast ((Expression) $2, (Expression) $4, lexer.Location);
+               $$ = new Cast ((Expression) $2, (Expression) $4, GetLocation ($1));
          }
        ;
 
@@ -3711,29 +3661,27 @@ prefixed_unary_expression
        : unary_expression
        | PLUS prefixed_unary_expression
          { 
-               $$ = new Unary (Unary.Operator.UnaryPlus, (Expression) $2, (Location) $1);
+               $$ = new Unary (Unary.Operator.UnaryPlus, (Expression) $2);
          } 
        | MINUS prefixed_unary_expression 
          { 
-               $$ = new Unary (Unary.Operator.UnaryNegation, (Expression) $2, (Location) $1);
+               $$ = new Unary (Unary.Operator.UnaryNegation, (Expression) $2);
          }
        | OP_INC prefixed_unary_expression 
          {
-               $$ = new UnaryMutator (UnaryMutator.Mode.PreIncrement,
-                                      (Expression) $2, (Location) $1);
+               $$ = new UnaryMutator (UnaryMutator.Mode.PreIncrement, (Expression) $2);
          }
        | OP_DEC prefixed_unary_expression 
          {
-               $$ = new UnaryMutator (UnaryMutator.Mode.PreDecrement,
-                                      (Expression) $2, (Location) $1);
+               $$ = new UnaryMutator (UnaryMutator.Mode.PreDecrement, (Expression) $2);
          }
        | STAR prefixed_unary_expression
          {
-               $$ = new Indirection ((Expression) $2, (Location) $1);
+               $$ = new Indirection ((Expression) $2, GetLocation ($1));
          }
        | BITWISE_AND prefixed_unary_expression
          {
-               $$ = new Unary (Unary.Operator.AddressOf, (Expression) $2, (Location) $1);
+               $$ = new Unary (Unary.Operator.AddressOf, (Expression) $2);
          }
        ;
 
@@ -3765,9 +3713,13 @@ additive_expression
          }
        | additive_expression MINUS multiplicative_expression
          {
-               $$ = new Binary (Binary.Operator.Subtraction, 
-                                (Expression) $1, (Expression) $3);
+               $$ = new Binary (Binary.Operator.Subtraction, (Expression) $1, (Expression) $3);
          }
+       | parenthesized_expression MINUS multiplicative_expression
+         {
+               // Shift/Reduce conflict
+               $$ = new Binary (Binary.Operator.Subtraction, (Expression) $1, (Expression) $3);
+         }
        | additive_expression AS type
          {
                $$ = new As ((Expression) $1, (Expression) $3, (Location) $2);
@@ -4026,19 +3978,19 @@ lambda_expression
          {
                LocatedToken lt = (LocatedToken) $1;
                Parameter p = new ImplicitLambdaParameter (lt.Value, lt.Location);
-               start_anonymous (true, new Parameters (p), (Location) $2);
+               start_anonymous (true, new Parameters (p), GetLocation ($1));
          }
          lambda_expression_body
          {
-               $$ = end_anonymous ((ToplevelBlock) $4, (Location) $2);
+               $$ = end_anonymous ((ToplevelBlock) $4);
          }
        | OPEN_PARENS_LAMBDA opt_lambda_parameter_list CLOSE_PARENS ARROW 
          {
-               start_anonymous (true, (Parameters) $2, (Location) $4);
+               start_anonymous (true, (Parameters) $2, GetLocation ($1));
          }
          lambda_expression_body 
          {
-               $$ = end_anonymous ((ToplevelBlock) $6, (Location) $4);
+               $$ = end_anonymous ((ToplevelBlock) $6);
          }
        ;
 
@@ -4091,6 +4043,7 @@ class_declaration
          }
          class_body
          {
+               --lexer.parsing_declaration;      
                if (RootContext.Documentation != null)
                        Lexer.doc_state = XmlCommentState.Allowed;
          }
@@ -4404,7 +4357,7 @@ declaration_statement
  * > expressions to prevent reduce/reduce errors in the grammar.
  * > The expressions are converted into types during semantic analysis.
  */
-local_variable_type
+variable_type
        : primary_expression_no_array_creation opt_rank_specifier_or_nullable
          { 
                // FIXME: Do something smart here regarding the composition of the type.
@@ -4476,7 +4429,7 @@ local_variable_pointer_type
          }
        | builtin_types STAR
          {
-               $$ = new ComposedCast ((FullNamedExpression) $1, "*", lexer.Location);
+               $$ = new ComposedCast ((FullNamedExpression) $1, "*", GetLocation ($1));
          }
        | VOID STAR
          {
@@ -4488,6 +4441,23 @@ local_variable_pointer_type
          }
        ;
 
+local_variable_type
+       : variable_type
+       | local_variable_pointer_type opt_rank_specifier
+         {
+               if ($1 != null){
+                       string rank = (string)$2;
+
+                       if (rank == "")
+                               $$ = $1;
+                       else
+                               $$ = current_array_type = new ComposedCast ((FullNamedExpression) $1, rank);
+               } else {
+                       $$ = null;
+               }
+         }
+       ;
+
 local_variable_declaration
        : local_variable_type local_variable_declarators
          {
@@ -4500,23 +4470,10 @@ local_variable_declaration
                } else
                        $$ = null;
          }
-       | local_variable_pointer_type opt_rank_specifier local_variable_declarators
-         {
-               if ($1 != null){
-                       Expression t;
-
-                       if ((string) $2 == "")
-                               t = (Expression) $1;
-                       else
-                               t = new ComposedCast ((FullNamedExpression) $1, (string) $2);
-                       $$ = new DictionaryEntry (t, $3);
-               } else 
-                       $$ = null;
-         }
        ;
 
 local_constant_declaration
-       : CONST local_variable_type constant_declarators
+       : CONST variable_type constant_declarators
          {
                if ($2 != null)
                        $$ = new DictionaryEntry ($2, $3);
@@ -4547,14 +4504,13 @@ statement_expression
 
                if (s == null) {
                        expr.Error_InvalidExpressionStatement ();
-                       $$ = null;
-               } else {
-                       $$ = new StatementExpression (s);
                }
+
+               $$ = new StatementExpression (s);
          }
        | error
          {
-               Report.Error (1002, GetLocation ($1), "Expecting {0}, got {1}", GetExpecting (), GetTokenName (yyToken));
+               Error_SyntaxError (yyToken);
                $$ = null;
          }
        ;
@@ -4570,7 +4526,7 @@ interactive_statement_expression
          }
        | error
          {
-               Report.Error (1002, GetLocation ($1), "Expecting {0}, got {1}", GetExpecting (), GetTokenName (yyToken));
+               Error_SyntaxError (yyToken);
                $$ = null;
          }
        ;
@@ -4581,7 +4537,7 @@ selection_statement
        ; 
 
 if_statement
-       : IF OPEN_PARENS boolean_expression CLOSE_PARENS 
+       : IF open_parens_any boolean_expression CLOSE_PARENS 
          embedded_statement
          { 
                Location l = (Location) $1;
@@ -4593,7 +4549,7 @@ if_statement
                        Report.Warning (642, 3, l, "Possible mistaken empty statement");
 
          }
-       | IF OPEN_PARENS boolean_expression CLOSE_PARENS
+       | IF open_parens_any boolean_expression CLOSE_PARENS
          embedded_statement ELSE embedded_statement
          {
                Location l = (Location) $1;
@@ -4609,7 +4565,7 @@ if_statement
        ;
 
 switch_statement
-       : SWITCH OPEN_PARENS
+       : SWITCH open_parens_any
          { 
                if (switch_stack == null)
                        switch_stack = new Stack (2);
@@ -4705,7 +4661,7 @@ iteration_statement
        ;
 
 while_statement
-       : WHILE OPEN_PARENS boolean_expression CLOSE_PARENS embedded_statement
+       : WHILE open_parens_any boolean_expression CLOSE_PARENS embedded_statement
          {
                Location l = (Location) $1;
                $$ = new While ((Expression) $3, (Statement) $5, l);
@@ -4714,7 +4670,7 @@ while_statement
 
 do_statement
        : DO embedded_statement 
-         WHILE OPEN_PARENS boolean_expression CLOSE_PARENS SEMICOLON
+         WHILE open_parens_any boolean_expression CLOSE_PARENS SEMICOLON
          {
                Location l = (Location) $1;
 
@@ -4723,8 +4679,7 @@ do_statement
        ;
 
 for_statement
-       : FOR OPEN_PARENS 
-         opt_for_initializer SEMICOLON
+       : FOR open_parens_any opt_for_initializer SEMICOLON
          {
                Location l = lexer.Location;
                start_block (l);  
@@ -4821,12 +4776,12 @@ statement_expression_list
        ;
 
 foreach_statement
-       : FOREACH OPEN_PARENS type IN expression CLOSE_PARENS
+       : FOREACH open_parens_any type IN expression CLOSE_PARENS
          {
                Report.Error (230, (Location) $1, "Type and identifier are both required in a foreach statement");
                $$ = null;
          }
-       | FOREACH OPEN_PARENS type IDENTIFIER IN
+       | FOREACH open_parens_any type IDENTIFIER IN
          expression CLOSE_PARENS 
          {
                start_block (lexer.Location);
@@ -5045,11 +5000,11 @@ opt_catch_args
        ;         
 
 catch_args 
-       : OPEN_PARENS type opt_identifier CLOSE_PARENS 
+       : open_parens_any type opt_identifier CLOSE_PARENS 
          {
                $$ = new DictionaryEntry ($2, $3);
          }
-       | OPEN_PARENS CLOSE_PARENS 
+       | open_parens_any CLOSE_PARENS 
          {
                Report.Error (1015, GetLocation ($1), "A type that derives from `System.Exception', `object', or `string' expected");
          }
@@ -5079,7 +5034,7 @@ unsafe_statement
        ;
 
 fixed_statement
-       : FIXED OPEN_PARENS 
+       : FIXED open_parens_any 
          type_and_void fixed_pointer_declarators 
          CLOSE_PARENS
          {
@@ -5147,7 +5102,7 @@ fixed_pointer_declarator
        ;
 
 lock_statement
-       : LOCK OPEN_PARENS expression CLOSE_PARENS 
+       : LOCK open_parens_any expression CLOSE_PARENS 
          {
                //
          } 
@@ -5158,7 +5113,7 @@ lock_statement
        ;
 
 using_statement
-       : USING OPEN_PARENS local_variable_declaration CLOSE_PARENS
+       : USING open_parens_any local_variable_declaration CLOSE_PARENS
          {
                start_block (lexer.Location);
                Block assign_block = current_block;
@@ -5213,7 +5168,7 @@ using_statement
                current_block.AddStatement (stmt);
                $$ = end_block (lexer.Location);
          }
-       | USING OPEN_PARENS expression CLOSE_PARENS
+       | USING open_parens_any expression CLOSE_PARENS
          {
                start_block (lexer.Location);
          }
@@ -5533,6 +5488,7 @@ interactive_parsing
                method.Block = (ToplevelBlock) end_block(lexer.Location);
                current_container.AddMethod (method);
 
+               --lexer.parsing_declaration;
                InteractiveResult = pop_current_class ();
                current_local_parameters = null;
          } 
@@ -5649,7 +5605,7 @@ void push_current_class (TypeContainer tc, object partial_token)
        else
                current_container = current_container.AddTypeContainer (tc);
 
-
+       ++lexer.parsing_declaration;
        current_class = tc;
 }
 
@@ -5945,6 +5901,10 @@ end_block (Location loc)
 void
 start_anonymous (bool lambda, Parameters parameters, Location loc)
 {
+       if (RootContext.Version == LanguageVersion.ISO_1){
+               Report.FeatureIsNotAvailable (loc, "anonymous methods");
+       }
+
        oob_stack.Push (current_anonymous_method);
        oob_stack.Push (current_local_parameters);
 
@@ -5962,17 +5922,12 @@ start_anonymous (bool lambda, Parameters parameters, Location loc)
  * Completes the anonymous method processing, if lambda_expr is null, this
  * means that we have a Statement instead of an Expression embedded 
  */
-AnonymousMethodExpression end_anonymous (ToplevelBlock anon_block, Location loc)
+AnonymousMethodExpression end_anonymous (ToplevelBlock anon_block)
 {
        AnonymousMethodExpression retval;
 
-       if (RootContext.Version == LanguageVersion.ISO_1){
-               Report.FeatureIsNotAvailable (loc, "anonymous methods");
-               retval = null;
-       } else  {
-               current_anonymous_method.Block = anon_block;
-               retval = current_anonymous_method;
-       }
+       current_anonymous_method.Block = anon_block;
+       retval = current_anonymous_method;
 
        current_local_parameters = (Parameters) oob_stack.Pop ();
        current_anonymous_method = (AnonymousMethodExpression) oob_stack.Pop ();
@@ -5986,24 +5941,65 @@ public NamespaceEntry CurrentNamespace {
        }
 }
 
-public string GetExpecting ()
+
+void Error_SyntaxError (int token)
 {
-       int [] tokens = yyExpectingTokens (yyExpectingState);
-       if (tokens.Length == 1)
-               return "`" + GetTokenName (tokens [0]) + "'";
+       Error_SyntaxError (0, token);
+}
+
+void Error_SyntaxError (int error_code, int token)
+{
+       string symbol = GetSymbolName (token);
+       string expecting = GetExpecting ();
        
-       StringBuilder sb = new StringBuilder ();
-       ArrayList names = new ArrayList ();
+       if (error_code == 0) {
+               if (expecting == "`)'")
+                       error_code = 1026;
+               else
+                       error_code = 1525;
+       }
+       
+       if (expecting != null)
+               Report.Error (error_code, lexer.Location, "Unexpected symbol `{0}', expecting {1}", 
+                       symbol, expecting);       
+       else
+               Report.Error (error_code, lexer.Location, "Unexpected symbol `{0}'", symbol);
+}
+
+string GetExpecting ()
+{
+       int [] tokens = yyExpectingTokens (yyExpectingState);
+       ArrayList names = new ArrayList (tokens.Length);
+       bool has_type = false;
+       bool has_identifier = false;
        for (int i = 0; i < tokens.Length; i++){
-               string name = GetTokenName (tokens [i]);
+               int token = tokens [i];
+               has_identifier |= token == Token.IDENTIFIER;
+               
+               string name = GetTokenName (token);
                if (name == "<internal>")
                        continue;
+                       
+               has_type |= name == "type";
                if (names.Contains (name))
                        continue;
                
                names.Add (name);
        }
 
+       //
+       // Too many tokens to enumerate
+       //
+       if (names.Count > 8)
+               return null;
+
+       if (has_type && has_identifier)
+               names.Remove ("identifier");
+
+       if (names.Count == 1)
+               return "`" + GetTokenName (tokens [0]) + "'";
+       
+       StringBuilder sb = new StringBuilder ();
        names.Sort ();
        int count = names.Count;
        for (int i = 0; i < count; i++){
@@ -6017,7 +6013,126 @@ public string GetExpecting ()
        return sb.ToString ();
 }
 
-static public string GetTokenName (int token)
+
+string GetSymbolName (int token)
+{
+       switch (token){
+       case Token.LITERAL_FLOAT:
+       case Token.LITERAL_INTEGER:
+       case Token.LITERAL_DOUBLE:
+       case Token.LITERAL_DECIMAL:
+       case Token.LITERAL_CHARACTER:
+       case Token.LITERAL_STRING:
+               return lexer.Value.ToString ();
+       case Token.IDENTIFIER:
+               return ((LocatedToken)lexer.Value).Value;
+
+       case Token.BOOL:
+               return "bool";
+       case Token.BYTE:
+               return "byte";
+       case Token.CHAR:
+               return "char";
+       case Token.VOID:
+               return "void";
+       case Token.DECIMAL:
+               return "decimal";
+       case Token.DOUBLE:
+               return "double";
+       case Token.FLOAT:
+               return "float";
+       case Token.INT:
+               return "int";
+       case Token.LONG:
+               return "long";
+       case Token.SBYTE:
+               return "sbyte";
+       case Token.SHORT:
+               return "short";
+       case Token.STRING:
+               return "string";
+       case Token.UINT:
+               return "uint";
+       case Token.ULONG:
+               return "ulong";
+       case Token.USHORT:
+               return "ushort";
+       case Token.OBJECT:
+               return "object";
+               
+       case Token.PLUS:
+               return "+";
+       case Token.UMINUS:
+       case Token.MINUS:
+               return "-";
+       case Token.BANG:
+               return "!";
+       case Token.BITWISE_AND:
+               return "&";
+       case Token.BITWISE_OR:
+               return "|";
+       case Token.STAR:
+               return "*";
+       case Token.PERCENT:
+               return "%";
+       case Token.DIV:
+               return "/";
+       case Token.CARRET:
+               return "^";
+       case Token.OP_INC:
+               return "++";
+       case Token.OP_DEC:
+               return "--";
+       case Token.OP_SHIFT_LEFT:
+               return "<<";
+       case Token.OP_SHIFT_RIGHT:
+               return ">>";
+       case Token.OP_LT:
+               return "<";
+       case Token.OP_GT:
+               return ">";
+       case Token.OP_LE:
+               return "<=";
+       case Token.OP_GE:
+               return ">=";
+       case Token.OP_EQ:
+               return "==";
+       case Token.OP_NE:
+               return "!=";
+       case Token.OP_AND:
+               return "&&";
+       case Token.OP_OR:
+               return "||";
+       case Token.OP_PTR:
+               return "->";
+       case Token.OP_COALESCING:       
+               return "??";
+       case Token.OP_MULT_ASSIGN:
+               return "*=";
+       case Token.OP_DIV_ASSIGN:
+               return "/=";
+       case Token.OP_MOD_ASSIGN:
+               return "%=";
+       case Token.OP_ADD_ASSIGN:
+               return "+=";
+       case Token.OP_SUB_ASSIGN:
+               return "-=";
+       case Token.OP_SHIFT_LEFT_ASSIGN:
+               return "<<=";
+       case Token.OP_SHIFT_RIGHT_ASSIGN:
+               return ">>=";
+       case Token.OP_AND_ASSIGN:
+               return "&=";
+       case Token.OP_XOR_ASSIGN:
+               return "^=";
+       case Token.OP_OR_ASSIGN:
+               return "|=";
+       }
+
+       return GetTokenName (token);
+}
+
+static string GetTokenName (int token)
 {
        switch (token){
        case Token.ABSTRACT:
@@ -6026,8 +6141,6 @@ static public string GetTokenName (int token)
                return "as";
        case Token.ADD:
                return "add";
-       case Token.ASSEMBLY:
-               return "assembly";
        case Token.BASE:
                return "base";
        case Token.BREAK:
@@ -6092,8 +6205,6 @@ static public string GetTokenName (int token)
                return "new";
        case Token.NULL:
                return "null";
-       case Token.OBJECT:
-               return "object";
        case Token.OPERATOR:
                return "operator";
        case Token.OUT:
@@ -6194,14 +6305,10 @@ static public string GetTokenName (int token)
                return "[";
        case Token.CLOSE_BRACKET:
                return "]";
-       case Token.DEFAULT_OPEN_PARENS:
+       case Token.OPEN_PARENS_CAST:
        case Token.OPEN_PARENS_LAMBDA:
        case Token.OPEN_PARENS:
                return "(";
-       case Token.CLOSE_PARENS_CAST:
-       case Token.CLOSE_PARENS_NO_CAST:
-       case Token.CLOSE_PARENS_OPEN_PARENS:
-       case Token.CLOSE_PARENS_MINUS:
        case Token.CLOSE_PARENS:
                return ")";
        case Token.DOT:
@@ -6241,8 +6348,6 @@ static public string GetTokenName (int token)
        case Token.OP_OR:
        case Token.OP_PTR:
        case Token.OP_COALESCING:       
-               return "<operator>";
-       
        case Token.OP_MULT_ASSIGN:
        case Token.OP_DIV_ASSIGN:
        case Token.OP_MOD_ASSIGN:
@@ -6270,7 +6375,8 @@ static public string GetTokenName (int token)
        case Token.UINT:
        case Token.ULONG:
        case Token.USHORT:
-               return "<type>";
+       case Token.OBJECT:
+               return "type";
        
        case Token.ASSIGN:
                return "=";
@@ -6303,8 +6409,6 @@ static public string GetTokenName (int token)
        case Token.EVAL_USING_DECLARATIONS_UNIT_PARSER:
        case Token.EVAL_STATEMENT_PARSER:
        case Token.LOWPREC:
-       case Token.QUERY_LAST_TOKEN:
-       case Token.QUERY_FIRST_TOKEN:
        case Token.LAST_KEYWORD:        
                return "<internal>";