LGTM, much more readable!

http://codereview.chromium.org/2278002/diff/17001/3004
File src/runtime.cc (right):

http://codereview.chromium.org/2278002/diff/17001/3004#newcode594
src/runtime.cc:594: Handle<FixedArray> elms =
Factory::NewFixedArray(DESCRIPTOR_SIZE);
Remove extra space before Factory::...

http://codereview.chromium.org/2278002/diff/17001/3005
File src/v8natives.js (right):

http://codereview.chromium.org/2278002/diff/17001/3005#newcode495
src/v8natives.js:495: // GetOwnProperty returns and array indexed by the
constants
and -> an

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

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

Reply via email to