LGTM with nit

http://codereview.chromium.org/7754014/diff/1/src/utils.h
File src/utils.h (right):

http://codereview.chromium.org/7754014/diff/1/src/utils.h#newcode617
src/utils.h:617: Vector<T> new_chunk = Vector<T>::New(new_capacity);
We don't need this new_chunk vector anymore right?

http://codereview.chromium.org/7754014/diff/1/test/cctest/test-utils.cc
File test/cctest/test-utils.cc (right):

http://codereview.chromium.org/7754014/diff/1/test/cctest/test-utils.cc#newcode205
test/cctest/test-utils.cc:205: i::Vector<const
char>("12345678901234567890123456789012", 32));
4 indent?

http://codereview.chromium.org/7754014/

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

Reply via email to