http://codereview.chromium.org/8776032/diff/1011/src/store-buffer.cc File src/store-buffer.cc (right):
http://codereview.chromium.org/8776032/diff/1011/src/store-buffer.cc#newcode73 src/store-buffer.cc:73: if (initial_length == 0) initial_length = 1; On 2011/12/02 13:38:52, Michael Starzinger wrote:
Can we change this into ASSERT(initial_length != 0)?
Refactored in a slightly different way. http://codereview.chromium.org/8776032/diff/1011/src/store-buffer.cc#newcode176 src/store-buffer.cc:176: reinterpret_cast<void*>(old_limit_), grow * kPointerSize, false)); On 2011/12/02 13:38:52, Michael Starzinger wrote:
Can we put each argument on a separate line like with the two
occurences above? Done. http://codereview.chromium.org/8776032/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
