LGTM with a nit.

https://codereview.chromium.org/11896060/diff/3001/src/objects-inl.h
File src/objects-inl.h (right):

https://codereview.chromium.org/11896060/diff/3001/src/objects-inl.h#newcode5018
src/objects-inl.h:5018: ::NONE,
Do we really need to make the top level namespace explicit here?

https://codereview.chromium.org/11896060/

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

Reply via email to