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) {
On 2009/10/02 13:22:07, Christian Plesner Hansen wrote:
> We should treat the signed and unsigned versions the same -- so either
move this
> into api.cc or move the signed version here.

Christian, I'm planning to inline Integer::New(int32_t) as well, but
wanted to have a separate CL for it.  Is it fine with you?  If not, I'd
move into api.cc for now.

http://codereview.chromium.org/260002

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

Reply via email to