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

Unified Diff: pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart

Issue 1068233002: Revert "Extract CallStructure from Selector." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 5 years, 8 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/compiler/lib/src/js_emitter/old_emitter/emitter.dart
diff --git a/pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart b/pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart
index 4b2d79c170b5b94a445703d27e1d974a44c67992..d67650b2db706c23f43e06c1111c877cd81d49ce 100644
--- a/pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart
+++ b/pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart
@@ -337,7 +337,8 @@ class OldEmitter implements Emitter {
}
String getReflectionNameInternal(elementOrSelector, String mangledName) {
- String name = namer.privateName(elementOrSelector.memberName);
+ String name =
+ namer.privateName(elementOrSelector.library, elementOrSelector.name);
if (elementOrSelector.isGetter) return name;
if (elementOrSelector.isSetter) {
if (!mangledName.startsWith(namer.setterPrefix)) return '$name=';
@@ -364,9 +365,9 @@ class OldEmitter implements Emitter {
String namedArguments = '';
bool isConstructor = false;
if (elementOrSelector is Selector) {
- CallStructure callStructure = elementOrSelector.callStructure;
- positionalParameterCount = callStructure.positionalArgumentCount;
- namedArguments = namedParametersAsReflectionNames(callStructure);
+ Selector selector = elementOrSelector;
+ positionalParameterCount = selector.positionalArgumentCount;
+ namedArguments = namedParametersAsReflectionNames(selector);
} else {
FunctionElement function = elementOrSelector;
if (function.isConstructor) {
@@ -380,9 +381,12 @@ class OldEmitter implements Emitter {
for (Element e in signature.optionalParameters) {
names.add(e.name);
}
- CallStructure callStructure =
- new CallStructure(positionalParameterCount, names);
- namedArguments = namedParametersAsReflectionNames(callStructure);
+ Selector selector = new Selector.call(
+ function.name,
+ function.library,
+ positionalParameterCount,
+ names);
+ namedArguments = namedParametersAsReflectionNames(selector);
} else {
// Named parameters are handled differently by mirrors. For unnamed
// parameters, they are actually required if invoked
@@ -409,9 +413,9 @@ class OldEmitter implements Emitter {
'Do not know how to reflect on this $element.');
}
- String namedParametersAsReflectionNames(CallStructure structure) {
- if (structure.isUnnamed) return '';
- String names = structure.getOrderedNamedArguments().join(':');
+ String namedParametersAsReflectionNames(Selector selector) {
+ if (selector.getOrderedNamedArguments().isEmpty) return '';
+ String names = selector.getOrderedNamedArguments().join(':');
return ':$names';
}

Powered by Google App Engine
This is Rietveld 408576698