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

Unified Diff: sdk/lib/mirrors/mirrors.dart

Issue 23890036: Change return type of get typeVariables and typeArguments in ClassMirror. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Addressed comments. Created 7 years, 3 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
« no previous file with comments | « runtime/lib/mirrors_impl.dart ('k') | tests/lib/mirrors/class_mirror_type_variables_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sdk/lib/mirrors/mirrors.dart
diff --git a/sdk/lib/mirrors/mirrors.dart b/sdk/lib/mirrors/mirrors.dart
index 313026f26ee74379a00199a0e78258661820fd51..f6fc7faf243d2bf5e0aa720dc40889f92e78a1e5 100644
--- a/sdk/lib/mirrors/mirrors.dart
+++ b/sdk/lib/mirrors/mirrors.dart
@@ -748,32 +748,30 @@ abstract class ClassMirror implements TypeMirror, ObjectMirror {
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.
+ * An immutable list with mirrors for all type variables for this type.
+ *
+ * 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 map preserves the order of declaration of the type variables.
+ * 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.
+ * An immutable list with mirrors for all type arguments for
+ * 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<TypeMirror> get typeArguments;
/**
* Is this the original declaration of this type?
« no previous file with comments | « runtime/lib/mirrors_impl.dart ('k') | tests/lib/mirrors/class_mirror_type_variables_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698