Move member-iterating code into ClassElement.

lib/compiler/implementation/elements/elements.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.
 
 #library('elements');
 
 #import('../tree/tree.dart');
 #import('../scanner/scannerlib.dart');
 #import('../leg.dart');  // TODO(karlklose): we only need type.
 #import('../util/util.dart');
@@ skipping 761 matching lines @@
   /**
    * Returns the super class, if any.
    *
    * The returned element may not be resolved yet.
    */
   ClassElement get superclass() {
     assert(isResolved);
     return supertype === null ? null : supertype.element;
   }
 
+  /**
+   * Runs through all members of this class.
+   *
+   * The enclosing class is passed to the callback. This is useful when
+   * [includeSuperMembers] is [:true:].
+   */
+  void forEachMember([void f(ClassElement enclosingClass, Element member),

[ahe, 2012/04/19 10:02:31]

Why is f optional?

[floitsch, 2012/04/19 10:41:32]

So that you can give it a name and put it last.
It is completely unreadable when the function is first.

[ahe, 2012/04/19 10:43:58]

I see.

+                      includeBackendMembers = false,
+                      includeSuperMembers = false]) {
+    ClassElement classElement = this;
+    do {
+      for (Element element in classElement.members) {
+        f(classElement, element);
+      }
+      if (includeBackendMembers) {
+        for (Element element in classElement.backendMembers) {
+          f(classElement, element);
+        }
+      }
+      classElement = includeSuperMembers ? classElement.superclass : null;
+    } while(classElement !== null);    
+  }
+
+  /**
+   * Runs through all instance-field members of this class.
+   *
+   * The enclosing class is passed to the callback. This is useful when
+   * [includeSuperMembers] is [:true:].
+   *
+   * When [includeBackendMembers] and [includeSuperMembers] are both [:true:]
+   * then the fields are visited in the same order as they need to be given
+   * to the JavaScript constructor.
+   */
+  void forEachInstanceField([void f(ClassElement enclosingClass, Element field),

[ahe, 2012/04/19 10:02:31]

Ditto.

+                             includeBackendMembers = false,
+                             includeSuperMembers = false]) {
+    // Filters so that [f] is only invoked with instance fields. 
+    void fieldFilter(ClassElement enclosingClass, Element member) {
+      if (member.isInstanceMember() && member.kind == ElementKind.FIELD) {
+        f(enclosingClass, member);
+      }
+    }
+
+    forEachMember(fieldFilter, includeBackendMembers, includeSuperMembers);
+  }
+
   bool isInterface() => false;
   bool isNative() => nativeName != null;
   SourceString nativeName;
 }
 
 class Elements {
   static bool isLocal(Element element) {
     return ((element !== null)
             && !element.isInstanceMember()
             && !isStaticOrTopLevelField(element)
@@ skipping 168 matching lines @@
   final Node node;
   Type bound;
   Type type;
   TypeVariableElement(name, Element enclosing, this.node, this.type,
                       [this.bound])
     : super(name, ElementKind.TYPE_VARIABLE, enclosing);
   Type computeType(compiler) => type;
   Node parseNode(compiler) => node;
   toString() => "${enclosingElement.toString()}.${name.slowToString()}";
 }