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

Unified Diff: pkg/analyzer/lib/src/generated/resolver.dart

Issue 207853002: Optimizations for hashCode. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 9 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: pkg/analyzer/lib/src/generated/resolver.dart
diff --git a/pkg/analyzer/lib/src/generated/resolver.dart b/pkg/analyzer/lib/src/generated/resolver.dart
index e53ecda8b1fb725aa205415af5b3d87274f14443..ff1e7ee86291a5cad89b0b2d2f006809f827d43a 100644
--- a/pkg/analyzer/lib/src/generated/resolver.dart
+++ b/pkg/analyzer/lib/src/generated/resolver.dart
@@ -15945,34 +15945,28 @@ class TypeResolverVisitor extends ScopedVisitor {
int argumentCount = arguments.length;
List<DartType> parameters = _getTypeArguments(type);
int parameterCount = parameters.length;
- int count = Math.min(argumentCount, parameterCount);
- List<DartType> typeArguments = new List<DartType>();
- for (int i = 0; i < count; i++) {
- DartType argumentType = _getType(arguments[i]);
- if (argumentType != null) {
- typeArguments.add(argumentType);
+ List<DartType> typeArguments = new List<DartType>(parameterCount);
+ if (argumentCount == parameterCount) {
+ for (int i = 0; i < parameterCount; i++) {
+ TypeName argumentTypeName = arguments[i];
+ DartType argumentType = _getType(argumentTypeName);
+ if (argumentType == null) {
+ argumentType = _dynamicType;
+ }
+ typeArguments[i] = argumentType;
}
- }
- if (argumentCount != parameterCount) {
+ } else {
reportErrorForNode(_getInvalidTypeParametersErrorCode(node), node, [typeName.name, parameterCount, argumentCount]);
- }
- argumentCount = typeArguments.length;
- if (argumentCount < parameterCount) {
- //
- // If there were too many arguments, we already handled it by not adding the values of the
- // extra arguments to the list. If there are too few, we handle it by adding 'dynamic'
- // enough times to make the count equal.
- //
- for (int i = argumentCount; i < parameterCount; i++) {
- typeArguments.add(_dynamicType);
+ for (int i = 0; i < parameterCount; i++) {
+ typeArguments[i] = _dynamicType;
}
}
if (type is InterfaceTypeImpl) {
InterfaceTypeImpl interfaceType = type as InterfaceTypeImpl;
- type = interfaceType.substitute4(new List.from(typeArguments));
+ type = interfaceType.substitute4(typeArguments);
} else if (type is FunctionTypeImpl) {
FunctionTypeImpl functionType = type as FunctionTypeImpl;
- type = functionType.substitute3(new List.from(typeArguments));
+ type = functionType.substitute3(typeArguments);
} else {
}
} else {

Powered by Google App Engine
This is Rietveld 408576698