Updating a comment in InternalStringToInt.

Review URL: http://codereview.chromium.org/1547017

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@4340 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

diff --git a/src/conversions.cc b/src/conversions.cc
index 651bc55..2929191 100644 (file)
--- a/src/conversions.cc
+++ b/src/conversions.cc
@@ -357,9 +357,10 @@ static double InternalStringToInt(Iterator current, EndMark end, int radix) {
     return sign ? -gay_strtod(buffer, NULL) : gay_strtod(buffer, NULL);
   }
 
-  // TODO(serya): The following legacy code causes accumulating rounding
-  // error for number greater than ~2^56. It should be rewritten using long
-  // arithmetic.
+  // The following code causes accumulating rounding error for numbers greater
+  // than ~2^56. It's explicitly allowed in the spec: "if R is not 2, 4, 8, 10,
+  // 16, or 32, then mathInt may be an implementation-dependent approximation to
+  // the mathematical integer value" (15.1.2.2).
 
   int lim_0 = '0' + (radix < 10 ? radix : 10);
   int lim_a = 'a' + (radix - 10);