Disable closures in initializers and fail instead of crashing.

frog/leg/ssa/closure.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.
 
 class ClosureFieldElement extends Element {
   ClosureFieldElement(SourceString name, ClassElement enclosing)
       : super(name, ElementKind.FIELD, enclosing);
 
   bool isInstanceMember() => true;
   bool isAssignable() => false;
@@ skipping 358 matching lines @@
     // If we are inside a named closure we have to declare ourselve. For
     // simplicity we declare the local even if the closure does not have a name
     // It will simply not be used.
     if (insideClosure) {
       declareLocal(element);
     }
 
     // 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);
 
     closureDataCache[node] = closureData;
 
     ClosureData savedClosureData = closureData;
     bool savedInsideClosure = insideClosure;
 
     // Restore old values.
     scopeVariables = oldScopeVariables;
@@ skipping 20 matching lines @@
     declareLocal(elements[node]);
   }
 
   visitTryStatement(TryStatement node) {
     // TODO(ngeoffray): implement finer grain state.
     inTryCatchOrFinally = true;
     node.visitChildren(this);
     inTryCatchOrFinally = false;
   }
 }