Reviewers: Mads Ager,
Description:
A few small changes to --print-ast behavior:
* Remove two horizontal spaces per indent
* Don't print an extra space between "VAR" and "(" for parameters
* Call the name on named property accesses "NAME"
Please review this at http://codereview.chromium.org/67124
SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/
Affected files:
M src/prettyprinter.cc
Index: src/prettyprinter.cc
===================================================================
--- src/prettyprinter.cc (revision 1697)
+++ src/prettyprinter.cc (working copy)
@@ -646,7 +646,7 @@
void AstPrinter::PrintIndented(const char* txt) {
for (int i = 0; i < indent_; i++) {
- Print(". ");
+ Print(". ");
}
Print(txt);
}
@@ -732,7 +732,7 @@
if (scope->num_parameters() > 0) {
IndentedScope indent("PARAMS");
for (int i = 0; i < scope->num_parameters(); i++) {
- PrintLiteralWithModeIndented("VAR ", scope->parameter(i),
+ PrintLiteralWithModeIndented("VAR", scope->parameter(i),
scope->parameter(i)->name(),
scope->parameter(i)->type());
}
@@ -1024,7 +1024,7 @@
Visit(node->obj());
Literal* literal = node->key()->AsLiteral();
if (literal != NULL && literal->handle()->IsSymbol()) {
- PrintLiteralIndented("LITERAL", literal->handle(), false);
+ PrintLiteralIndented("NAME", literal->handle(), false);
} else {
PrintIndentedVisit("KEY", node->key());
}
--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---
