https://chromiumcodereview.appspot.com/9373023/diff/2001/src/ast.h File src/ast.h (right):
https://chromiumcodereview.appspot.com/9373023/diff/2001/src/ast.h#newcode2520 src/ast.h:2520: VISIT_AND_RETURN(ModuleDeclaration, decl) On 2012/02/09 13:18:35, Jakob wrote:
You'll want to implement the matching Visit____ methods in AstConstructionVisitor to make this work.
Done. https://chromiumcodereview.appspot.com/9373023/diff/2001/src/hydrogen.cc File src/hydrogen.cc (right): https://chromiumcodereview.appspot.com/9373023/diff/2001/src/hydrogen.cc#newcode6593 src/hydrogen.cc:6593: On 2012/02/09 13:18:35, Jakob wrote:
nit: we generally have two empty lines between top-level things.
Done. https://chromiumcodereview.appspot.com/9373023/diff/2001/src/prettyprinter.cc File src/prettyprinter.cc (right): https://chromiumcodereview.appspot.com/9373023/diff/2001/src/prettyprinter.cc#newcode84 src/prettyprinter.cc:84: On 2012/02/09 13:18:35, Jakob wrote:
nit: two empty lines
Done. https://chromiumcodereview.appspot.com/9373023/diff/2001/src/prettyprinter.cc#newcode770 src/prettyprinter.cc:770: On 2012/02/09 13:18:35, Jakob wrote:
same here
Done. https://chromiumcodereview.appspot.com/9373023/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
