LGTM

http://codereview.chromium.org/2944016/diff/6001/7003
File test/mjsunit/object-freeze.js (right):

http://codereview.chromium.org/2944016/diff/6001/7003#newcode32
test/mjsunit/object-freeze.js:32: // Test that we throw an error if the
an object is not passed as argument.
the an -> an

http://codereview.chromium.org/2944016/diff/6001/7003#newcode53
test/mjsunit/object-freeze.js:53: var obj = {x: 42, z: 'foobar'};
Space after { and before }?

Similarly in the rest of the file.

http://codereview.chromium.org/2944016/diff/6001/7003#newcode97
test/mjsunit/object-freeze.js:97: // // Test on accessors.
Remove extra //

http://codereview.chromium.org/2944016/diff/6001/7003#newcode122
test/mjsunit/object-freeze.js:122: obj2.foo = 42;
Identation off.

http://codereview.chromium.org/2944016/show

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to