Rework Unparser/unparse StringBuffer management.

lib/compiler/implementation/dart_backend/emitter.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 String emitCode(
       Compiler compiler,
       Unparser unparser,
       Map<LibraryElement, String> imports,
       Collection<Element> topLevelElements,
       Map<ClassElement, Collection<Element>> classMembers) {
-  final sb = new StringBuffer();
   final processedVariableLists = new Set<VariableListElement>();
 
   void outputElement(Element element) {
     if (element is SynthesizedConstructorElement) return;
     if (element.isField()) {
       assert(element is VariableElement);
       // Different VariableElement's may refer to the same VariableListElement.
       // Output this list only once.
       // TODO: only emit used variables.
       VariableElement variableElement = element;
       final variableList = variableElement.variables;
       if (!processedVariableLists.contains(variableList)) {
         processedVariableLists.add(variableList);
-        sb.add(unparser.unparse(variableList.parseNode(compiler)));
+        unparser.visit(variableList.parseNode(compiler));

[Roman, 2012/09/03 06:39:24]

Can you please make a function 'unparse' in Unparser that will just call
visit()?
Again it's a matter of client code vs implementation details. I can understand
"visit" when I'm inside Unparser, because I'm aware that it's a visitor. For
client the name 'visit' makes little sense, because it does not matter how
Unparser unparses code, whether it's a visitor or not.

[Anton Muhin, 2012/09/03 09:28:10]

Done.

       }
     } else {
-      sb.add(unparser.unparse(element.parseNode(compiler)));
+      unparser.visit(element.parseNode(compiler));
     }
   }
 
   void outputClass(ClassElement classElement, Collection<Element> members) {
     ClassNode classNode = classElement.parseNode(compiler);
     // classElement.beginToken is 'class', 'interface', or 'abstract'.
-    sb.add(classNode.beginToken.slowToString());
-    if (classNode.beginToken.slowToString() == 'abstract') {
-      sb.add(' ');
-      sb.add(classNode.beginToken.next.slowToString());  // 'class'
+    unparser.addToken(classNode.beginToken);
+    if (classNode.beginToken.stringValue == 'abstract') {
+      unparser.addToken(classNode.beginToken.next);
     }
-    sb.add(' ');
-    sb.add(unparser.unparse(classNode.name));
+    unparser.visit(classNode.name);
     if (classNode.typeParameters !== null) {
-      sb.add(unparser.unparse(classNode.typeParameters));
+      unparser.visit(classNode.typeParameters);
     }
     if (classNode.extendsKeyword !== null) {
-      sb.add(' ');
-      classNode.extendsKeyword.value.printOn(sb);
-      sb.add(' ');
-      sb.add(unparser.unparse(classNode.superclass));
+      unparser.addString(' ');
+      unparser.addToken(classNode.extendsKeyword);
+      unparser.visit(classNode.superclass);
     }
     if (!classNode.interfaces.isEmpty()) {
-      sb.add(' ');
-      sb.add(unparser.unparse(classNode.interfaces));
+      unparser.addString(' ');
+      unparser.visit(classNode.interfaces);
     }
     if (classNode.defaultClause !== null) {
-      sb.add(' default ');
-      sb.add(unparser.unparse(classNode.defaultClause));
+      unparser.addString(' default ');
+      unparser.visit(classNode.defaultClause);
     }
-    sb.add('{');
+    unparser.addString('{');
     members.forEach((element) {
       // TODO(smok): Filter out default constructors here.
       outputElement(element);
     });
-    sb.add('}');
+    unparser.addString('}');
   }
 
   imports.forEach((libraryElement, prefix) {
-    sb.add('#import("${libraryElement.uri}",prefix:"$prefix");');
+    unparser.addString('#import("${libraryElement.uri}",prefix:"$prefix");');
   });
 
   for (final element in topLevelElements) {
     if (element is ClassElement) {
       outputClass(element, classMembers[element]);
     } else {
       outputElement(element);
     }
   }
-
-  return sb.toString();
 }