LGTM

https://chromiumcodereview.appspot.com/9605006/diff/1/src/elements.cc
File src/elements.cc (right):

https://chromiumcodereview.appspot.com/9605006/diff/1/src/elements.cc#newcode179
src/elements.cc:179: virtual MaybeObject* AddElementsToFixedArray(
The Chromium styleguide says that in function declarations (as opposed
to calls), each argument should be on its own line unless all arguments
fit on one line.

https://chromiumcodereview.appspot.com/9605006/diff/1/src/elements.h
File src/elements.h (right):

https://chromiumcodereview.appspot.com/9605006/diff/1/src/elements.h#newcode83
src/elements.h:83: // Deleted an element in an object, returning a new
elements backing store.
s/Deleted/Deletes/

https://chromiumcodereview.appspot.com/9605006/

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

Reply via email to