implement private members, fixes #74

lib/src/js/printer.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.
 
 part of js_ast;
 
 
 class JavaScriptPrintingOptions {
   final bool shouldCompressOutput;
   final bool minifyLocalVariables;
@@ skipping 849 matching lines @@
       case "yield":
         return true;
     }
     return false;
   }
 
   visitAccess(PropertyAccess access) {
     visitNestedExpression(access.receiver, CALL,
                           newInForInit: inForInit,
                           newAtStatementBegin: atStatementBegin);
-    Node selector = access.selector;
-    if (selector is LiteralString) {
-      LiteralString selectorString = selector;
-      String fieldWithQuotes = selectorString.value;
-      if (isValidJavaScriptId(fieldWithQuotes)) {
-        if (access.receiver is LiteralNumber) out(" ");
-        out(".");
-        out(fieldWithQuotes.substring(1, fieldWithQuotes.length - 1));
-        return;
-      }
-    }
-    out("[");
-    visitNestedExpression(selector, EXPRESSION,
-                          newInForInit: false, newAtStatementBegin: false);
-    out("]");
+    propertyNameOut(access.selector, inAccess: true);
   }
 
   visitNamedFunction(NamedFunction namedFunction) {
     VarCollector vars = new VarCollector();
     vars.visitNamedFunction(namedFunction);
     functionOut(namedFunction.function, namedFunction.name, vars);
   }
 
   visitFun(Fun fun) {
     VarCollector vars = new VarCollector();
@@ skipping 182 matching lines @@
       spaceOut();
       out("{}");
     } else {
       blockBody(fun.body, needsSeparation: false, needsNewline: false);
     }
     localNamer.leaveScope();
   }
 
   visitPropertyName(PropertyName node) => propertyNameOut(node);
 
-  void propertyNameOut(Expression node, {bool inMethod: false}) {
-    inForInit = false;
-    atStatementBegin = false;
+  void propertyNameOut(Expression node, {bool inMethod: false,
+      bool inAccess: false}) {
 
     if (node is LiteralNumber) {
       LiteralNumber nameNumber = node;
+      if (inAccess) out('[');
       out(nameNumber.value);
+      if (inAccess) out(']');
     } else {
       String quotedName;
       if (node is PropertyName) {
         quotedName = "'${node.name}'";
       } else if (node is LiteralString) {
         quotedName = node.value;
       }
       if (quotedName != null) {
         if (isValidJavaScriptId(quotedName)) {
+          if (inAccess) out('.');
           out(quotedName.substring(1, quotedName.length - 1));
         } else {
-          if (inMethod) out("[");
+          if (inMethod || inAccess) out("[");
           out(quotedName);
-          if (inMethod) out("]");
+          if (inMethod || inAccess) out("]");
         }
       } else {
         // ComputedPropertyName
         out("[");
-        visit(node);
+        visitNestedExpression(node, EXPRESSION,
+                              newInForInit: false, newAtStatementBegin: false);
         out("]");
       }
     }
   }
 
   visitLiteralExpression(LiteralExpression node) {
     String template = node.template;
     List<Expression> inputs = node.inputs;
 
     List<String> parts = template.split('#');
@@ skipping 335 matching lines @@
         codes.add(nthLetter((n ~/ nameSpaceSize) % LETTERS));
       }
       codes.add(charCodes.$0 + digit);
       newName = new String.fromCharCodes(codes);
     }
     assert(new RegExp(r'[a-zA-Z][a-zA-Z0-9]*').hasMatch(newName));
     maps.last[oldName] = newName;
     return newName;
   }
 }