LGTM.

http://codereview.chromium.org/7911/diff/1/3
File src/api.cc (right):

http://codereview.chromium.org/7911/diff/1/3#newcode2495
Line 2495: double v8::Date::Value() {
Is this function really necessary? You can get the same result by
converting it to a number by using NumbeValue, right? Maybe it's okay
for consistency.

http://codereview.chromium.org/7911

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

Reply via email to