LGTM

http://codereview.chromium.org/6594075/diff/1/src/handles.h
File src/handles.h (right):

http://codereview.chromium.org/6594075/diff/1/src/handles.h#newcode144
src/handles.h:144: Handle<T> Escape(Handle<T> handle_value) {
I find the name Escape a little bit misleading since it also closes the
scope. Can we call it something like CloseAndEscape(handle)?

http://codereview.chromium.org/6594075/

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

Reply via email to