LGTM
Please don't forget the postponed issues :) Maybe create a bug mentioning
them
all.
Thanks
Vitaly
http://codereview.chromium.org/2918001/diff/33002/12015
File src/scopeinfo.h (right):
http://codereview.chromium.org/2918001/diff/33002/12015#newcode156
src/scopeinfo.h:156: static Handle<Object> CreateHeapObject(Scope*
scope);
Since this uses ZoneScopeInfo only as a temporary storage and it's an
implementation detail really, I think CreateHeapObject and EmptyHeap
object functions should be in ScopeInfo.
http://codereview.chromium.org/2918001/show
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
