Drive-by comments

http://codereview.chromium.org/6588130/diff/1/src/runtime.cc
File src/runtime.cc (right):

http://codereview.chromium.org/6588130/diff/1/src/runtime.cc#newcode1473
src/runtime.cc:1473: if (result.is_null()) return Failure::Exception();
you may use RETURN_IF_EMPTY_HANDLE

http://codereview.chromium.org/6588130/diff/1/src/runtime.cc#newcode8663
src/runtime.cc:8663: if (SetElement(jsobject, index1, tmp2).is_null()) {
Ditto

http://codereview.chromium.org/6588130/

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

Reply via email to