Change return type of get typeVariables and typeArguments in ClassMirror.

sdk/lib/mirrors/mirrors.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.
 
 // For the purposes of the mirrors library, we adopt a naming
 // convention with respect to getters and setters.  Specifically, for
 // some variable or field...
 //
 //   var myField;
 //
@@ skipping 730 matching lines @@
    */
   Map<Symbol, VariableMirror> get variables;
 
   /**
    * An immutable map from names to mirrors for all constructor
    * declarations for this type.
    */
   Map<Symbol, MethodMirror> get constructors;
 
   /**
-   * An immutable map from names to mirrors for all type variables for
-   * this type.
-   * If this type is a generic declaration or an invocation of
-   * a generic declaration, the returned map has the names of the formal
-   * type parameters of the original declaration as its keys, and
-   * each such key maps to a TypeVariableMirror on the corresponding
-   * type variable. Otherwise, the returned map is empty.
+   * A list with mirrors for all type variables for this type.

[ahe, 2013/09/18 15:08:27]

an immutable list

[zarah, 2013/09/19 07:00:24]

Done.

    *
-   * This map preserves the order of declaration of the type variables.
+   * If this type is a generic declaration or an invocation of a generic
+   * declaration, the returned list contains mirrors on the type variables.
+   * Otherwise, the returned list is empty.
+   *
+   * This list preserves the order of declaration of the type variables.
    */
-  Map<Symbol, TypeVariableMirror> get typeVariables;
+  List<TypeVariableMirror> get typeVariables;
 
   /**
-   * An immutable map from names to mirrors for all type arguments for
-   * this type.  The keys of the map are the names of the
-   * corresponding type variables.
+   * A list with mirrors for all type arguments for

[ahe, 2013/09/18 15:08:27]

an immutable list

[zarah, 2013/09/19 07:00:24]

Done.

+   * this type.
    *
    * If the the reflectee is an invocation of a generic class,
    * the type arguments are the bindings of its type parameters.
    * If the reflectee is the original declaration of a generic,
-   * it has no type arguments and this method returns an empty map.
+   * it has no type arguments and this method returns an empty list.
    * If the reflectee is a not generic, then
-   * it has no type arguments and this method returns an empty map.
-   * This map preserves the order of declaration of the type variables.
+   * it has no type arguments and this method returns an empty list.
+   *
+   * This list preserves the order of declaration of the type variables.
    */
-  Map<Symbol, TypeMirror> get typeArguments;
+  List<TypeVariableMirror> get typeArguments;

[gbracha, 2013/09/18 19:54:35]

List<TypeMirror>

[zarah, 2013/09/19 07:00:24]

Done.

 
   /**
    * Is this the original declaration of this type?
    *
    * For most classes, they are their own original declaration.  For
    * generic classes, however, there is a distinction between the
    * original class declaration, which has unbound type variables, and
    * the instantiations of generic classes, which have bound type
    * variables.
    */
@@ skipping 536 matching lines @@
    *
    * When used as metadata on an import of "dart:mirrors", this metadata does
    * not apply to the library in which the annotation is used, but instead
    * applies to the other libraries (all libraries if "*" is used).
    */
   final override;
 
   const MirrorsUsed(
       {this.symbols, this.targets, this.metaTargets, this.override});
 }