Add preliminary support for methods to AstBuilder.

pkg/front_end/lib/src/fasta/analyzer/ast_builder.dart

 // Copyright (c) 2016, 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.
 
 library fasta.analyzer.ast_builder;
 
 import 'package:front_end/src/fasta/scanner/token.dart'
     show BeginGroupToken, Token;
 
 import 'package:analyzer/analyzer.dart';
@@ skipping 1000 matching lines @@
         comment, metadata, name, toAnalyzerToken(extendsOrSuper), bound));
   }
 
   @override
   void endTypeVariables(int count, Token beginToken, Token endToken) {
     debugEvent("TypeVariables");
     List<TypeParameter> typeParameters = popList(count);
     push(ast.typeParameterList(toAnalyzerToken(beginToken), typeParameters,
         toAnalyzerToken(endToken)));
   }
+
+  void endMethod(Token getOrSet, Token beginToken, Token endToken) {

[scheglov, 2017/02/21 20:17:27]

@override

[Paul Berry, 2017/02/21 20:24:03]

Done.

+    debugEvent("Method");
+    FunctionBody body = _endFunctionBody();
+    ConstructorName redirectedConstructor = null; // TODO(paulberry)
+    List<ConstructorInitializer> initializers = null; // TODO(paulberry)
+    Token separator = null; // TODO(paulberry)
+    FormalParameterList parameters = pop();
+    TypeParameterList typeParameters = pop();
+    var name = pop();
+    analyzer.Token propertyKeyword = toAnalyzerToken(getOrSet);
+    TypeAnnotation returnType = pop();
+    // TODO(paulberry): handle modifiers.
+    var modifiers = pop();
+    assert(modifiers == null);
+    Token externalKeyword = null; // TODO(paulberry)
+    Token constKeyword = null; // TODO(paulberry)
+    Token factoryKeyword = null; // TODO(paulberry)
+    List<Annotation> metadata = pop();
+    // TODO(paulberry): capture doc comments.
+    Comment comment = null;
+    SimpleIdentifier returnType2;
+    Token period;
+    SimpleIdentifier name2;
+    if (name is SimpleIdentifier) {
+      SimpleIdentifier returnType2 = name;

[ahe, 2017/02/22 11:30:38]

There's a few hints introduced with this change. Could we agree on keeping the
code free of hints?

[Paul Berry, 2017/02/22 16:37:12]

Fair enough.  I will follow up with a fix.

+    }
+    push(ast.constructorDeclaration(
+        comment,
+        metadata,
+        toAnalyzerToken(externalKeyword),
+        toAnalyzerToken(constKeyword),
+        toAnalyzerToken(factoryKeyword),
+        returnType2,
+        toAnalyzerToken(period),
+        name2,
+        parameters,
+        toAnalyzerToken(separator),
+        initializers,
+        redirectedConstructor,
+        body));
+  }
+
+  void endMember() {

[scheglov, 2017/02/21 20:17:27]

@override

[Paul Berry, 2017/02/21 20:24:03]

Done.

+    debugEvent("Member");
+  }
 }
 
 /// Data structure placed on the stack to represent a class body.
 ///
 /// This is needed because analyzer has no separate AST representation of a
 /// class body; it simply stores all of the relevant data in the
 /// [ClassDeclaration] object.
 class _ClassBody {
   final Token beginToken;
 
@@ skipping 12 matching lines @@
 /// [ClassDeclaration] or [ClassTypeAlias] object.
 class _MixinApplication {
   final TypeName supertype;
 
   final Token withKeyword;
 
   final List<TypeName> mixinTypes;
 
   _MixinApplication(this.supertype, this.withKeyword, this.mixinTypes);
 }