Turn some usages of NewArray with DeleteArray in the same scope into ScopedVector or SmartPointer.

Turn some usages of NewArray with DeleteArray in the same scope into ScopedVector or SmartPointer.

That makes it easier to maintain the code---one should care less
about releasing the memory as smart pointers would take care of
this.

Switch to ScopedVector instead Vector for the same semantics in
src/builtins.cc

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

[antonm, 2010-04-30 15:49:18]

Kasper, Mads and Misha,

may you have a look?

I'm just fulfilling the promise :)

[mnaganov (inactive), 2010-04-30 21:09:28]

On 2010/04/30 15:49:18, antonm wrote:
> Kasper, Mads and Misha,
> 
> may you have a look?
> 
> I'm just fulfilling the promise :)

LGTM! Thanks for doing this.

The only thing that worries me is that ScopedVector allows compiler to generate
copy constructor and assignment, which can lead to double frees. Could you
please disallow them?

[antonm, 2010-05-04 19:33:11]

Very nice idea, Michail, done.

I'll commit it tomorrow to track all the build breakages it might introduce.