Feedback addressed, landing.

http://codereview.chromium.org/9605006/diff/1/src/elements.cc
File src/elements.cc (right):

http://codereview.chromium.org/9605006/diff/1/src/elements.cc#newcode179
src/elements.cc:179: virtual MaybeObject* AddElementsToFixedArray(
On 2012/03/06 11:57:43, Jakob wrote:
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.

Done.

http://codereview.chromium.org/9605006/diff/1/src/elements.h
File src/elements.h (right):

http://codereview.chromium.org/9605006/diff/1/src/elements.h#newcode83
src/elements.h:83: // Deleted an element in an object, returning a new
elements backing store.
On 2012/03/06 11:57:43, Jakob wrote:
s/Deleted/Deletes/

Done.

http://codereview.chromium.org/9605006/

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

Reply via email to