Fix "illegal access" when calling parseInt with a radix that is not a smi.

src/v8natives.js

Index: src/v8natives.js
diff --git a/src/v8natives.js b/src/v8natives.js
index 6ad4a0c75bdb1648492c56ced8170c61faa7749c..da12ccd907ce88c6556874681d27ee8cd4f8724f 100644
--- a/src/v8natives.js
+++ b/src/v8natives.js
@@ -106,7 +106,7 @@ function GlobalParseInt(string, radix) {
       // Truncate number.
       return string | 0;
     }
-    if (IS_UNDEFINED(radix)) radix = 0;
+    radix = radix || 0;

[Lasse Reichstein, 2011/06/28 08:32:21]

This seems silly to begin with. Just set 
 radix = 0;
It means the same as radix 10, so the rest of the function will work the same,
and it seems we optimize the code for 0 over 10.
 

[Steven, 2011/06/28 10:10:35]

It seems so but it's not: 'radix = 0' will trigger radix detection what you
don't want in the case 'radix == 10'.
On 2011/06/28 08:32:21, Lasse Reichstein wrote:

[Lasse Reichstein, 2011/06/28 10:28:17]

Ack, my bad.
In that case, use
 radix = TO_INT32(radix);
Or, if it's faster (it probably is), just:
 radix = radix | 0;
Both will convert to Int32, which converts both undefined and -0 to 0.
Using || 0 is likely to be slower, since it contains an implicity branch.

[Lasse Reichstein, 2011/06/28 10:31:48]

and because  radix = radix || 0  would not convert a HeapNumber containing 10 to
a smi.

If possible, add a test where the number is a heap-number value equal to 10 (but
it's hard to guarantee the underlying representation, try something like
Math.ln(Math.exp(10)) and cross your fingers).

   } else {
     radix = TO_INT32(radix);
     if (!(radix == 0 || (2 <= radix && radix <= 36)))