LGTM, but:
http://codereview.chromium.org/160382/diff/1011/15 File src/handles.h (right): http://codereview.chromium.org/160382/diff/1011/15#newcode185 Line 185: int expected_additional_properties = 0); I would get rid of default values for expected_additional_properties and just make it explicit everywhere (and >= 0). http://codereview.chromium.org/160382/diff/1011/12 File src/objects.cc (right): http://codereview.chromium.org/160382/diff/1011/12#newcode2139 Line 2139: property_count += 2; // Make spece for two more properties. spece -> space http://codereview.chromium.org/160382 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
