LGTM
http://codereview.chromium.org/113634/diff/1/8 File src/builtins.cc (right): http://codereview.chromium.org/113634/diff/1/8#newcode398 Line 398: // API that support calls. The object can be called as either a constructor In all of these comments, I would remove the "that supports calls". http://codereview.chromium.org/113634/diff/1/9 File src/contexts.h (right): http://codereview.chromium.org/113634/diff/1/9#newcode94 Line 94: call_as_constructor_delegate) \ Indentation seems slightly off here? http://codereview.chromium.org/113634/diff/1/12 File src/execution.h (right): http://codereview.chromium.org/113634/diff/1/12#newcode133 Line 133: static Handle<Object> GetConstructorDelegate(Handle<Object> object); Add comment like the one above? http://codereview.chromium.org/113634 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
