http://codereview.chromium.org/131107/diff/1/2
File src/mirror-delay.js (right):

http://codereview.chromium.org/131107/diff/1/2#newcode2240
Line 2240: type: scope.scopeType(),
On 2009/06/22 06:42:28, Søren Gjesse wrote:
> I think you ahould add
>    index: i,
> even though it will be equal to the array index.

Done.

http://codereview.chromium.org/131107/diff/1/2#newcode2241
Line 2241: ref: scope.scopeObject().handle()
On 2009/06/22 06:42:28, Søren Gjesse wrote:
> For transient scope objects (negative handles) you should skip the ref
to the
> handle, as it cannot be retrieved through that id anyway.

Done.

http://codereview.chromium.org/131107

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

Reply via email to