Remove printDebugInfo from Unparser

lib/compiler/implementation/tree/unparser.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.
 
 class Unparser implements Visitor {
   StringBuffer sb;
-  final bool printDebugInfo;
 
-  Unparser([this.printDebugInfo = false]);
+  Unparser();
 
   String unparse(Node node) {
     sb = new StringBuffer();
     visit(node);
     return sb.toString();
   }
 
   void add(SourceString string) {
     string.printOn(sb);
   }
 
   visit(Node node) {
     if (node !== null) {

[Anton Muhin, 2012/06/05 09:35:19]

probably it's time to go single line now: if (node !== null) node.accept(this);

[Roman, 2012/06/05 11:09:07]

Done.

By the way, Dart style guide is ambiguous:
http://www.dartlang.org/articles/style-guide/

On one hand, it says: "DO use curly braces for all flow control structures.",
but later there is "green example":
if (obj is! SomeType) print('not SomeType');

[Anton Muhin, 2012/06/05 11:39:21]

You may want to file a bug.

Historically, at least AAR people use single line variants a lot.

On 2012/06/05 11:09:07, Roman wrote:

-      if (printDebugInfo) sb.add('[${node.getObjectDescription()}: ');
       node.accept(this);
-      if (printDebugInfo) sb.add(']');
-    } else if (printDebugInfo) {
-      sb.add('[null]');
     }
   }
 
   visitBlock(Block node) {
     visit(node.statements);
   }
 
   visitCascade(Cascade node) {
     visit(node.expression);
   }
@@ skipping 387 matching lines @@
       sb.add(' ');
     }
     visit(node.name);
     if (node.typeParameters !== null) {
       visit(node.typeParameters);
     }
     visit(node.formals);
     add(node.endToken.value);
   }
 }