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

Unified Diff: sdk/lib/_internal/compiler/implementation/resolution/members.dart

Issue 17825002: Remove support for 'argument definitions test' from dart2js. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Remove SentinelConstant. Created 7 years, 6 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: sdk/lib/_internal/compiler/implementation/resolution/members.dart
diff --git a/sdk/lib/_internal/compiler/implementation/resolution/members.dart b/sdk/lib/_internal/compiler/implementation/resolution/members.dart
index 4f619ce409d95af65fd427461bbbb1af53082e63..a835942131fbbd2fc8b9add34c7786cb9080cf2e 100644
--- a/sdk/lib/_internal/compiler/implementation/resolution/members.dart
+++ b/sdk/lib/_internal/compiler/implementation/resolution/members.dart
@@ -16,7 +16,6 @@ abstract class TreeElements {
Selector getGetterSelectorInComplexSendSet(SendSet node);
Selector getOperatorSelectorInComplexSendSet(SendSet node);
DartType getType(Node node);
- bool isParameterChecked(Element element);
void setSelector(Node node, Selector selector);
void setGetterSelectorInComplexSendSet(SendSet node, Selector selector);
void setOperatorSelectorInComplexSendSet(SendSet node, Selector selector);
@@ -37,7 +36,6 @@ class TreeElementMapping implements TreeElements {
final Map<Spannable, Selector> selectors =
new LinkedHashMap<Spannable, Selector>();
final Map<Node, DartType> types = new LinkedHashMap<Node, DartType>();
- final Set<Element> checkedParameters = new LinkedHashSet<Element>();
final Set<Node> superUses = new LinkedHashSet<Node>();
final Set<Element> otherDependencies = new LinkedHashSet<Element>();
final int hashCode = ++hashCodeCounter;
@@ -134,10 +132,6 @@ class TreeElementMapping implements TreeElements {
return selectors[node.inToken];
}
- bool isParameterChecked(Element element) {
- return checkedParameters.contains(element);
- }
-
void registerDependency(Element element) {
otherDependencies.add(element.implementation);
}
@@ -2285,14 +2279,6 @@ class ResolverVisitor extends MappingVisitor<Element> {
compiler.enqueuer.resolution.registerAsCheck(type, mapping);
}
resolvedArguments = true;
- } else if (identical(operatorString, '?')) {
- Element parameter = mapping[node.receiver];
- if (parameter == null
- || !identical(parameter.kind, ElementKind.PARAMETER)) {
- error(node.receiver, MessageKind.PARAMETER_NAME_EXPECTED);
- } else {
- mapping.checkedParameters.add(parameter);
- }
}
}

Powered by Google App Engine
This is Rietveld 408576698