Reviewers: jochen (slow),

https://codereview.chromium.org/962983002/diff/40001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/962983002/diff/40001/include/v8.h#newcode438
include/v8.h:438: V8_INLINE bool ToLocal(Local<S>* out) const {
On 2015/02/27 09:42:49, jochen (slow) wrote:
can we mark this as must use result?

Done.

Description:
Convert v8::Value::To* to use MaybeLocal

BUG=v8:3929
LOG=y

Please review this at https://codereview.chromium.org/962983002/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+209, -122 lines):
  M include/v8.h
  M include/v8config.h
  M src/api.h
  M src/api.cc


--
--
v8-dev mailing list
[email protected]
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 [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to