Start addressing massive arrays on the stack. There is hardly ever...

Start addressing massive arrays on the stack. There is hardly ever
a reason to stack allocate large chunks of stack space.
- Runtime_GetCFrames used to allocate a frame size of 52040 bytes.
- PreallocatedMemoryThread::Run used to allocate 32784 bytes.
- Fixed StringStream overflow conditions.


Committed: http://code.google.com/p/v8/source/detail?r=1729

[Mikhail Naganov, 2009-04-16 01:20:54]

http://codereview.chromium.org/67197/diff/1/4
File src/runtime.cc (right):

http://codereview.chromium.org/67197/diff/1/4#newcode5934
Line 5934: frames.Dispose();
I think we should use a stack-allocated auto disposer for a Vector here.

[Søren Thygesen Gjesse, 2009-04-16 05:25:44]

LGTM

[iposva, 2009-04-16 17:39:05]

Addressed review comments (added ScopedVector) and fixed StringStream overflow
conditions. Please take another look.

-Ivan

[Mikhail Naganov, 2009-04-16 17:41:41]

LGTM

Cool, thanks!

Please fix lint errors prior to submitting.