Lgtm.

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

http://codereview.chromium.org/260002/diff/1/3#newcode3023
Line 3023: Local<Integer> Integer::New(uint32_t value) {
We should treat the signed and unsigned versions the same -- so either
move this into api.cc or move the signed version here.

http://codereview.chromium.org/260002

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

Reply via email to