LGTM

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(),
I think you ahould add
   index: i,
even though it will be equal to the array index.

http://codereview.chromium.org/131107/diff/1/2#newcode2241
Line 2241: ref: scope.scopeObject().handle()
For transient scope objects (negative handles) you should skip the ref
to the handle, as it cannot be retrieved through that id anyway.

http://codereview.chromium.org/131107

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

Reply via email to