Support new getter syntax.

dart/lib/compiler/implementation/scanner/parser.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * An event generating parser of Dart programs. This parser expects
  * all tokens in a linked list (aka a token stream).
  *
  * The class [Scanner] is used to generate a token stream. See the
  * file scanner.dart.
@@ skipping 65 matching lines @@
 
   Token parseReturnTypeOpt(Token token) {
     if (token.stringValue === 'void') {
       listener.handleVoidKeyword(token);
       return token.next;
     } else {
       return parseTypeOpt(token);
     }
   }
 
+  Token parseFormalParametersOpt(Token token) {
+    if (optional('(', token)) {
+      return parseFormalParameters(token);
+    } else {
+      listener.handleNoFormalParameters(token);
+      return token;
+    }
+  }
+
   Token parseFormalParameters(Token token) {
     Token begin = token;
     listener.beginFormalParameters(begin);
     expect('(', token);
     int parameterCount = 0;
     if (optional(')', token.next)) {
       listener.endFormalParameters(parameterCount, begin, token.next);
       return token.next.next;
     }
     do {
@@ skipping 260 matching lines @@
       // Skip type.
       token = peek;
     }
     if (token === getOrSet) token = token.next;
     token = parseIdentifier(token);
     bool isField;
     while (true) {
       // Loop to allow the listener to rewrite the token stream for
       // error handling.
       final String value = token.stringValue;
-      if (value === '(') {
+      if (value === '(' || value === '{' || value === '=>') {
         isField = false;
         break;
       } else if ((value === '=') || (value === ';') || (value === ',')) {
         isField = true;
         break;
       } else {
         token = listener.unexpected(token);
         if (token.kind === EOF_TOKEN) {
           // TODO(ahe): This is a hack. It would be better to tell the
           // listener more explicitly that it must pop an identifier.
           listener.endTopLevelFields(1, start, token);
           return token;
         }
       }
     }
     if (isField) {
       int fieldCount = 1;
       token = parseVariableInitializerOpt(token);
       while (optional(',', token)) {
         token = parseIdentifier(token.next);
         token = parseVariableInitializerOpt(token);
         ++fieldCount;
       }
       expectSemicolon(token);
       listener.endTopLevelFields(fieldCount, start, token);
     } else {
-      token = parseFormalParameters(token);
+      token = parseFormalParametersOpt(token);
       token = parseFunctionBody(token, false);
       listener.endTopLevelMethod(start, getOrSet, token);
     }
     return token.next;
   }
 
   Token parseVariableInitializerOpt(Token token) {
     if (optional('=', token)) {
       Token assignment = token;
       listener.beginInitializer(token);
@@ skipping 192 matching lines @@
     if (optional('operator', token)) {
       token = parseOperatorName(token);
     } else {
       token = parseIdentifier(token);
     }
     bool isField;
     while (true) {
       // Loop to allow the listener to rewrite the token stream for
       // error handling.
       final String value = token.stringValue;
-      if ((value === '(') || (value === '.')) {
+      if ((value === '(') || (value === '.') || (value === '{') ||

[kasperl, 2012/08/20 12:55:31]

So here all the === comparisons are wrapped in () where as in the previous case
(line 375) they are not? Change one of them to be consistent?

[ahe, 2012/08/20 13:26:17]

Done.

+          (value === '=>')) {
         isField = false;
         break;
       } else if ((value === '=') || (value === ';') || (value === ',')) {
         isField = true;
         break;
       } else {
         token = listener.unexpected(token);
         if (token.kind === EOF_TOKEN) {
           // TODO(ahe): This is a hack, see parseTopLevelMember.
           listener.endFields(1, start, token);
@@ skipping 10 matching lines @@
       while (optional(',', token)) {
         // TODO(ahe): Count these.
         token = parseIdentifier(token.next);
         token = parseVariableInitializerOpt(token);
         ++fieldCount;
       }
       expectSemicolon(token);
       listener.endFields(fieldCount, start, token);
     } else {
       token = parseQualifiedRestOpt(token);
-      token = parseFormalParameters(token);
+      token = parseFormalParametersOpt(token);
       token = parseInitializersOpt(token);
       token = parseFunctionBody(token, false);
       listener.endMethod(getOrSet, start, token);
     }
     return token.next;
   }
 
   Token parseFactoryMethod(Token token) {
     assert((optional('external', token) && optional('factory', token.next)) ||
            optional('factory', token));
@@ skipping 41 matching lines @@
       if (getOrSet === token) token = token.next;
       listener.beginFunctionName(token);
       if (optional('operator', token)) {
         token = parseOperatorName(token);
       } else {
         token = parseIdentifier(token);
       }
     }
     token = parseQualifiedRestOpt(token);
     listener.endFunctionName(token);
-    token = parseFormalParameters(token);
+    token = parseFormalParametersOpt(token);
     token = parseInitializersOpt(token);
     token = parseFunctionBody(token, false);
     listener.endFunction(getOrSet, token);
     return token.next;
   }
 
   Token parseUnamedFunction(Token token) {
     listener.beginUnamedFunction(token);
     token = parseFormalParameters(token);
     bool isBlock = optional('{', token);
@@ skipping 1069 matching lines @@
     }
     listener.handleContinueStatement(hasTarget, continueKeyword, token);
     return expectSemicolon(token);
   }
 
   Token parseEmptyStatement(Token token) {
     listener.handleEmptyStatement(token);
     return expectSemicolon(token);
   }
 }