https://codereview.chromium.org/225283005/diff/1/src/api.cc
File src/api.cc (right):
https://codereview.chromium.org/225283005/diff/1/src/api.cc#newcode6984
src/api.cc:6984: EXCEPTION_BAILOUT_CHECK(isolate, Local<Value>());
This second EXCEPTION_BAILOUT_CHECK will decrement call depth for the
second time which is not good. I think you should keep the second
EXCEPTION_PREAMBLE.
https://codereview.chromium.org/225283005/diff/1/src/isolate.h
File src/isolate.h (right):
https://codereview.chromium.org/225283005/diff/1/src/isolate.h#newcode193
src/isolate.h:193: isolate, call, MaybeHandle<T>())
Now it fits one line.
https://codereview.chromium.org/225283005/diff/1/src/runtime.cc
File src/runtime.cc (right):
https://codereview.chromium.org/225283005/diff/1/src/runtime.cc#newcode216
src/runtime.cc:216: static MaybeHandle<Object> CreateLiteralBoilerplate(
I think you should add MUST_USE_RESULT here and for every static
MaybeHandle<> function below.
https://codereview.chromium.org/225283005/diff/1/src/runtime.h
File src/runtime.h (right):
https://codereview.chromium.org/225283005/diff/1/src/runtime.h#newcode901
src/runtime.h:901: static MaybeHandle<Object>
CreateArrayLiteralBoilerplate(
MUST_USE_RESULT
https://codereview.chromium.org/225283005/diff/1/test/cctest/test-heap.cc
File test/cctest/test-heap.cc (right):
https://codereview.chromium.org/225283005/diff/1/test/cctest/test-heap.cc#newcode288
test/cctest/test-heap.cc:288: JSReceiver::SetProperty(global, name,
function, NONE, SLOPPY).Assert();
Shouldn't we use .Check() here and in all other tests?
https://codereview.chromium.org/225283005/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.