LGTM. I'm not too crazy about the inobject name, but I can live with
it until I come up with something better.

On Wed, Oct 15, 2008 at 7:44 AM,  <[EMAIL PROTECTED]> wrote:
> Reviewers: Kasper Lund, Feng Qian,
>
> Description:
> Allocate room for expected number of properties based on the
> constructor in the JSObject. This removes the need to allocate
> a properties array if the object is never assigned any extra
> properties.
>
>
> Please review this at http://codereview.chromium.org/7341
>
> Affected files:
>  M     src/bootstrapper.cc
>  M     src/builtins-ia32.cc
>  M     src/heap.cc
>  M     src/objects-debug.cc
>  M     src/objects-inl.h
>  M     src/objects.h
>  M     src/objects.cc
>  M     src/runtime.cc
>  M     src/string-stream.cc
>  M     src/stub-cache-arm.cc
>  M     src/stub-cache-ia32.cc
>
>
>

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

Reply via email to