Do not put renaming logic into Unparser.

lib/compiler/implementation/dart_backend/backend.dart

Index: lib/compiler/implementation/dart_backend/backend.dart
diff --git a/lib/compiler/implementation/dart_backend/backend.dart b/lib/compiler/implementation/dart_backend/backend.dart
index c28f41669fb1b6a6459c321ba65ee648f22ee231..de00a48d13d1f34632cf1a65d1991ead3b6f332b 100644
--- a/lib/compiler/implementation/dart_backend/backend.dart
+++ b/lib/compiler/implementation/dart_backend/backend.dart
@@ -423,7 +423,7 @@ class DartBackend extends Backend {
       }
     }
 
-    final unparser = new Unparser.withRenamer((Node node) => renames[node]);
+    final unparser = new EmitterUnparser(renames);
     emitCode(unparser, imports, topLevelNodes, memberNodes);
     compiler.assembledCode = unparser.result;
 
@@ -450,6 +450,27 @@ class DartBackend extends Backend {
   log(String message) => compiler.log('[DartBackend] $message');
 }
 
+class EmitterUnparser extends Unparser {
+  final Map<Node, String> renames;
+
+  EmitterUnparser(this.renames);
+
+  visit(Node node) {
+    if (node !== null && renames.containsKey(node)) {
+      sb.add(renames[node]);
+    } else {
+      super.visit(node);
+    }
+  }
+
+  unparseSendReceiver(Send node, [bool spacesNeeded=false]) {
+    // TODO(smok): Remove ugly hack for library prefices.

[Roman, 2012/10/02 07:55:55]

prefixes ?

[Anton Muhin, 2012/10/02 08:48:42]

O tempora, o mores!

Let's wait for Peter's opinion.

On 2012/10/02 07:55:55, Roman wrote:

+    if (node.receiver !== null && renames[node.receiver] == '') return;

[ahe, 2012/10/02 14:27:39]

I'm not sure what problem this is solving. If you show me an example, perhaps I
have a suggestion for getting rid of it.

[Anton Muhin, 2012/10/02 14:32:15]

That'd be great, Peter.

The problem.  Imagine you use name with a prefix.  When we bundle the
application in a single file in most of the cases we want to delete this prefix
and emit the name w/o the prefix and that's the best way we came up by to
achieve that.

If notion of the name would have its own Send node, that is prefix.value = 1
would have translated into SendSet { target: Send { receiver: prefix, selector:
value }, value = 1 }, we could have renamed it at the level of nodes.

I'll submit this CL in ~20 mins after I ran the test.

On 2012/10/02 14:27:39, ahe wrote:

[ahe, 2012/10/09 07:40:59]

Anton, could you give me an example with actual code?

On 2012/10/02 14:32:15, Anton Muhin wrote:

[Anton Muhin, 2012/10/10 07:21:40]

I assume you're asking for code being transformed.  If not, I'll show how we do
it.

Imagine the code

import 'mylib.dart' as mylib;

main() {
  mylib.topLevelField = 13;
}

where mylib.dart is

int topLevelField = 42;

we'd like to emit (omitting renames):

topLevelField = 42;

main() {
  topLevelField = 13;
}

omitting the prefix.

Our whole renamer can rename the whole AST nodes---we have a map from a node to
string where most of those nodes are identifiers and sometimes it sends, Roma is
currently unifying it all.

However, in this case this would be a send set and there is no node to rename
to, so we need this special trick to mark AST node as one that should be
actually removed the input.

Does that make sense?

Is there simpler approach?

We're considering AST rewrites for those, fyi.


On 2012/10/09 07:40:59, ahe wrote:

+    super.unparseSendReceiver(node, spacesNeeded);
+  }
+}
+
+
 /**
  * Some elements are not recorded by resolver now,
  * for example, typedefs or classes which are only