Good. LGTM. On Wed, Jul 29, 2009 at 10:32 AM, <[email protected]> wrote:
> Reviewers: Kevin Millikin, > > Description: > Add missing case in AST printer. > > Please review this at http://codereview.chromium.org/159580 > > SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ > > Affected files: > M src/prettyprinter.cc > > > Index: src/prettyprinter.cc > =================================================================== > --- src/prettyprinter.cc (revision 2569) > +++ src/prettyprinter.cc (working copy) > @@ -934,6 +934,9 @@ > case ObjectLiteral::Property::COMPUTED: > prop_kind = "PROPERTY - COMPUTED"; > break; > + case ObjectLiteral::Property::MATERIALIZED_LITERAL: > + prop_kind = "PROPERTY - MATERIALIZED_LITERAL"; > + break; > case ObjectLiteral::Property::PROTOTYPE: > prop_kind = "PROPERTY - PROTOTYPE"; > break; > @@ -945,7 +948,6 @@ > break; > default: > UNREACHABLE(); > - break; > } > IndentedScope prop(prop_kind); > PrintIndentedVisit("KEY", node->properties()->at(i)->key()); > > > --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
