On 2013/09/10 06:09:56, Sven Panne wrote:
LGTM with a nit.
https://codereview.chromium.org/23464022/diff/21001/include/v8.h
File include/v8.h (right):
https://codereview.chromium.org/23464022/diff/21001/include/v8.h#newcode3816
include/v8.h:3816: Maybe<bool> is_memory_constrained() { return
is_memory_constrained_; }
Missing const.
Thanks Sven, updated the CL to address your comments.
Could you please land this CL for me (and the Maybe template moving CL which
this CL depends upon).
https://codereview.chromium.org/23464022/
--
--
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/groups/opt_out.
