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

Unified Diff: sdk/lib/_internal/compiler/implementation/mirrors/dart2js_mirror.dart

Issue 18670003: Remove support for conflicting constructors. (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/mirrors/dart2js_mirror.dart
diff --git a/sdk/lib/_internal/compiler/implementation/mirrors/dart2js_mirror.dart b/sdk/lib/_internal/compiler/implementation/mirrors/dart2js_mirror.dart
index 14934380c2eefe897c3adf54387cb698f95c116d..1be7087792fb73798bf35bc1296a668019b6a4bb 100644
--- a/sdk/lib/_internal/compiler/implementation/mirrors/dart2js_mirror.dart
+++ b/sdk/lib/_internal/compiler/implementation/mirrors/dart2js_mirror.dart
@@ -1272,13 +1272,11 @@ class Dart2JsMethodMirror extends Dart2JsMemberMirror
implements MethodMirror {
final Dart2JsContainerMirror _objectMirror;
final String simpleName;
- final String constructorName;
final Dart2JsMethodKind _kind;
Dart2JsMethodMirror._internal(Dart2JsContainerMirror objectMirror,
FunctionElement function,
String this.simpleName,
- String this.constructorName,
Dart2JsMethodKind this._kind)
: this._objectMirror = objectMirror,
super(objectMirror.mirrors, function);
@@ -1290,7 +1288,6 @@ class Dart2JsMethodMirror extends Dart2JsMemberMirror
// Elements.operatorNameToIdentifier.
String simpleName =
Elements.operatorNameToIdentifier(function.name).slowToString();
- String constructorName = null;
Dart2JsMethodKind kind;
if (function.kind == ElementKind.GETTER) {
kind = Dart2JsMethodKind.GETTER;
@@ -1299,16 +1296,6 @@ class Dart2JsMethodMirror extends Dart2JsMemberMirror
simpleName = '$simpleName=';
} else if (function.kind == ElementKind.GENERATIVE_CONSTRUCTOR) {
// TODO(johnniwinther): Support detection of redirecting constructors.
- constructorName = '';
- int dollarPos = simpleName.indexOf('\$');
- if (dollarPos != -1) {
- constructorName = simpleName.substring(dollarPos + 1);
- simpleName = simpleName.substring(0, dollarPos);
- // Simple name is TypeName.constructorName.
- simpleName = '$simpleName.$constructorName';
- } else {
- // Simple name is TypeName.
- }
if (function.modifiers.isConst()) {
kind = Dart2JsMethodKind.CONST;
} else {
@@ -1316,13 +1303,6 @@ class Dart2JsMethodMirror extends Dart2JsMemberMirror
}
} else if (function.modifiers.isFactory()) {
kind = Dart2JsMethodKind.FACTORY;
- constructorName = '';
- int dollarPos = simpleName.indexOf('\$');
- if (dollarPos != -1) {
- constructorName = simpleName.substring(dollarPos+1);
- simpleName = simpleName.substring(0, dollarPos);
- simpleName = '$simpleName.$constructorName';
- }
} else if (realName == 'unary-') {
kind = Dart2JsMethodKind.OPERATOR;
// Simple name is 'unary-'.
@@ -1336,7 +1316,7 @@ class Dart2JsMethodMirror extends Dart2JsMemberMirror
kind = Dart2JsMethodKind.REGULAR;
}
return new Dart2JsMethodMirror._internal(objectMirror, function,
- simpleName, constructorName, kind);
+ simpleName, kind);
}
FunctionElement get _function => _element;
@@ -1354,9 +1334,6 @@ class Dart2JsMethodMirror extends Dart2JsMemberMirror
bool get isMethod => !isConstructor;
- bool get isPrivate =>
- isConstructor ? _isPrivate(constructorName) : _isPrivate(simpleName);
-
bool get isStatic => _function.modifiers.isStatic();
List<ParameterMirror> get parameters {

Powered by Google App Engine
This is Rietveld 408576698