LGTM

http://codereview.chromium.org/7219003/diff/13001/src/d8.cc
File src/d8.cc (right):

http://codereview.chromium.org/7219003/diff/13001/src/d8.cc#newcode491
src/d8.cc:491: HandleScope scope;
Maybe add a comment here that when the utility context is created all
three contexts needs access to one another.

http://codereview.chromium.org/7219003/diff/13001/src/d8.cc#newcode928
src/d8.cc:928: bool FLAG_executed = false;
FLAG_executed -> executed or script_executed. It is not a command line
flag.

http://codereview.chromium.org/7219003/

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

Reply via email to