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

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

Issue 21013004: Remove support for getters with parameters. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 5 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 cea260dac883e35ab42b2a75dc43a0386350761a..b5532b2c5610c061092b1cc6e013570a4fe3871e 100644
--- a/sdk/lib/_internal/compiler/implementation/resolution/members.dart
+++ b/sdk/lib/_internal/compiler/implementation/resolution/members.dart
@@ -3862,14 +3862,9 @@ class SignatureResolver extends CommonResolverVisitor<Element> {
if (!identical(formalParameters.getEndToken().next.stringValue,
// TODO(ahe): Remove the check for native keyword.
'native')) {
- if (compiler.rejectDeprecatedFeatures &&
- // TODO(ahe): Remove isPlatformLibrary check.
- !element.getLibrary().isPlatformLibrary) {
- compiler.reportError(formalParameters,
- MessageKind.EXTRA_FORMALS);
- } else {
- compiler.onDeprecatedFeature(formalParameters, 'getter parameters');
- }
+ trace('$element');
ahe 2013/08/06 14:25:06 Remove trace.
Johnni Winther 2013/08/07 06:35:17 Done.
+ compiler.reportError(formalParameters,
+ MessageKind.EXTRA_FORMALS);
}
}
LinkBuilder<Element> parametersBuilder =
@@ -3898,10 +3893,6 @@ class SignatureResolver extends CommonResolverVisitor<Element> {
MessageKind.ILLEGAL_SETTER_FORMALS);
}
}
- if (element.isGetter() && (requiredParameterCount != 0
- || visitor.optionalParameterCount != 0)) {
- compiler.reportError(formalParameters, MessageKind.EXTRA_FORMALS);
- }
return new FunctionSignatureX(parameters,
visitor.optionalParameters,
requiredParameterCount,
« no previous file with comments | « no previous file | tests/compiler/dart2js/deprecated_features_test.dart » ('j') | tests/language/getter_parameters_test.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698