On 2009/08/19 09:16:51, Kasper Lund wrote:
> LGTM.

> http://codereview.chromium.org/174052/diff/1/4
> File src/spaces.cc (right):

> http://codereview.chromium.org/174052/diff/1/4#newcode1083
> Line 1083: int extra = RoundUp(capacity_ / 2,
OS::AllocateAlignment());
> How about using Min(RoundUp(...), maximum_capacity - capacity)?

Good idea.  Done.

http://codereview.chromium.org/174052

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

Reply via email to