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

Unified Diff: compiler/java/com/google/dart/compiler/type/InterfaceTypeImplementation.java

Issue 9270016: Issue 932. Checks for various named arguments cases. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Tweaks for comments, changes in tests. Created 8 years, 11 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
Index: compiler/java/com/google/dart/compiler/type/InterfaceTypeImplementation.java
diff --git a/compiler/java/com/google/dart/compiler/type/InterfaceTypeImplementation.java b/compiler/java/com/google/dart/compiler/type/InterfaceTypeImplementation.java
index 4331e2d0b2a2263a46df1bb373eea2ea28e1df0b..712ad098e7fcf9fc93547614994745f410e14da2 100644
--- a/compiler/java/com/google/dart/compiler/type/InterfaceTypeImplementation.java
+++ b/compiler/java/com/google/dart/compiler/type/InterfaceTypeImplementation.java
@@ -16,9 +16,9 @@ import java.util.List;
*/
class InterfaceTypeImplementation extends AbstractType implements InterfaceType {
private final ClassElement element;
- private final List<? extends Type> arguments;
+ private final List<Type> arguments;
- InterfaceTypeImplementation(ClassElement element, List<? extends Type> arguments) {
+ InterfaceTypeImplementation(ClassElement element, List<Type> arguments) {
this.element = element;
this.arguments = arguments;
}
@@ -29,7 +29,7 @@ class InterfaceTypeImplementation extends AbstractType implements InterfaceType
}
@Override
- public List<? extends Type> getArguments() {
+ public List<Type> getArguments() {
return arguments;
}
@@ -61,7 +61,7 @@ class InterfaceTypeImplementation extends AbstractType implements InterfaceType
}
@Override
- public InterfaceType subst(List<? extends Type> arguments, List<? extends Type> parameters) {
+ public InterfaceType subst(List<Type> arguments, List<Type> parameters) {
if (arguments.isEmpty() && parameters.isEmpty()) {
return this;
}
@@ -130,8 +130,8 @@ class InterfaceTypeImplementation extends AbstractType implements InterfaceType
private List<InterfaceType> getInterfaces() {
List<InterfaceType> interfaces = getElement().getInterfaces();
List<InterfaceType> result = new ArrayList<InterfaceType>(interfaces.size());
- List<? extends Type> typeArguments = getArguments();
- List<? extends Type> typeParameters = getElement().getTypeParameters();
+ List<Type> typeArguments = getArguments();
+ List<Type> typeParameters = getElement().getTypeParameters();
for (InterfaceType type : interfaces) {
result.add(type.subst(typeArguments, typeParameters));
}
@@ -159,8 +159,8 @@ class InterfaceTypeImplementation extends AbstractType implements InterfaceType
@Override
public Type getType() {
- List<? extends Type> typeArguments = getHolder().getArguments();
- List<? extends Type> typeParameters = getHolder().getElement().getTypeParameters();
+ List<Type> typeArguments = getHolder().getArguments();
+ List<Type> typeParameters = getHolder().getElement().getTypeParameters();
return getElement().getType().subst(typeArguments, typeParameters);
}
}

Powered by Google App Engine
This is Rietveld 408576698