Disable closures in initializers and fail instead of crashing.

frog/leg/ssa/closure.dart

Index: frog/leg/ssa/closure.dart
diff --git a/frog/leg/ssa/closure.dart b/frog/leg/ssa/closure.dart
index 7414a5698967148206c4615ca64c95ae0ab73655..ad364e758fd42b8202192e05921d335dd877646d 100644
--- a/frog/leg/ssa/closure.dart
+++ b/frog/leg/ssa/closure.dart
@@ -376,7 +376,10 @@ class ClosureTranslator extends AbstractVisitor {
     // TODO(ahe): This is problematic. The backend should not repeat
     // the work of the resolver. It is the resolver's job to create
     // parameters, etc. Other phases should only visit statements.
-    node.visitChildren(this);
+    // TODO(floitsch): we avoid visiting the initializers on purpose so that we
+    // get an error-message later in the builder.
+    if (node.parameters !== null) node.parameters.accept(this);

[ahe, 2012/03/20 21:12:30]

Visiting the parameters is also problematic. Long term, I hope we can use the
parameter elements.

+    if (node.body !== null) node.body.accept(this);
 
     attachCapturedScopeVariables(node);