Looks good to me, but since it changes the V8 API, I would like to ask Daniel
for the final approval.

http://codereview.chromium.org/9139018/diff/4001/src/api.cc
File src/api.cc (left):

http://codereview.chromium.org/9139018/diff/4001/src/api.cc#oldcode6032
src/api.cc:6032:
nit: please, leave 2 lines here, too

http://codereview.chromium.org/9139018/diff/4001/src/api.cc
File src/api.cc (right):

http://codereview.chromium.org/9139018/diff/4001/src/api.cc#newcode4050
src/api.cc:4050:
nit: 2 empty lines between function definitions.

http://codereview.chromium.org/9139018/

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

Reply via email to