Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(469)

Unified Diff: dart/frog/leg/scanner/listener.dart

Issue 9109015: AST nodes for function-typed parameters. (Closed) Base URL: ssh://aaricia.aar/home/ahe/Dart/all/dart.googlecode.com.git@master
Patch Set: Created 8 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: dart/frog/leg/scanner/listener.dart
diff --git a/dart/frog/leg/scanner/listener.dart b/dart/frog/leg/scanner/listener.dart
index 1c4559c111fc65c3fa2cea8836912ec217095871..adaeca2db309c2915150dabd8f02796a1fc028fb 100644
--- a/dart/frog/leg/scanner/listener.dart
+++ b/dart/frog/leg/scanner/listener.dart
@@ -1057,9 +1057,11 @@ class NodeListener extends ElementListener {
}
void handleFunctionTypedFormalParameter(Token token) {
- canceler.cancel('function typed formal parameters are not implemented',
- token: token);
- NodeList parameters = popNode();
+ NodeList formals = popNode();
+ Identifier name = popNode();
+ TypeAnnotation type = popNode();
ngeoffray 2012/01/06 08:20:46 Rename 'type' to 'returnType'?
ahe 2012/01/06 11:00:03 Done.
+ pushNode(null); // Signal "no return type" to endFormalParameter.
karlklose 2012/01/06 08:23:00 I think it should be "no type".
ahe 2012/01/06 11:00:03 Done.
+ pushNode(new FunctionExpression(name, formals, null, type, null, null));
}
void handleValuedFormalParameter(Token equals, Token token) {

Powered by Google App Engine
This is Rietveld 408576698