I think you should add some test cases to test-api.cc. Other than that, the change LGTM.
http://codereview.chromium.org/165044/diff/1/4 File src/objects-inl.h (right): http://codereview.chromium.org/165044/diff/1/4#newcode104 Line 104: // There is a constraint on the object; check This comment seems incomplete. Terminate with . http://codereview.chromium.org/165044/diff/1/4#newcode106 Line 106: // Fetch the constructor function of the object Terminate comment with . http://codereview.chromium.org/165044 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
