http://codereview.chromium.org/115814/diff/1/3 File src/log-utils.cc (right):
http://codereview.chromium.org/115814/diff/1/3#newcode43 Line 43: blocks_[0] = NewArray<char>(block_size_); On 2009/05/27 12:00:40, Søren Gjesse wrote: > Maybe add a private method AllocateBlock. Done. http://codereview.chromium.org/115814/diff/1/3#newcode93 Line 93: blocks_[++block_index_] = NewArray<char>(block_size_); On 2009/05/27 12:00:40, Søren Gjesse wrote: > Ditto. Done. http://codereview.chromium.org/115814/diff/1/4 File src/log-utils.h (right): http://codereview.chromium.org/115814/diff/1/4#newcode49 Line 49: // filled is returned, it is <= that 'buf_size'. On 2009/05/27 12:00:40, Søren Gjesse wrote: > Remove that Done. http://codereview.chromium.org/115814/diff/1/7 File test/cctest/test-log-utils.cc (right): http://codereview.chromium.org/115814/diff/1/7#newcode16 Line 16: // Writes ref_buffer.length chars into dynabuf. On 2009/05/27 12:00:40, Søren Gjesse wrote: > Please add to comment that this also actually fills the ref_buffer before > writing it to the dynabuf. Done. http://codereview.chromium.org/115814 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
