LGTM.

http://codereview.chromium.org/1529004/diff/29001/30003
File src/conversions.cc (right):

http://codereview.chromium.org/1529004/diff/29001/30003#newcode335
src/conversions.cc:335: // The buffer may contain up to
kMaxSignificantDigits + 1 digits and a zero
On 2010/03/31 17:18:08, SeRya wrote:
On 2010/03/31 16:50:00, Florian Loitsch wrote:
> Maybe add comment:
> A number with more than kMaxSignificantDigits digits will be read as
Infinity.

The comment 7 lines below says the same. I think that is place more
suitable
since digits are dropped there and it explaines why it's safe.
true. fine with me.

http://codereview.chromium.org/1529004

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

To unsubscribe, reply using "remove me" as the subject.

Reply via email to