LGTM.

http://codereview.chromium.org/122038/diff/1/2
File include/v8.h (right):

http://codereview.chromium.org/122038/diff/1/2#newcode215
Line 215: T* operator->() const {
I would write this on a single line just like the IsEmpty case.

http://codereview.chromium.org/122038/diff/1/2#newcode219
Line 219: T* operator*() const {
Ditto.

http://codereview.chromium.org/122038

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

Reply via email to