Browse Source

Merge pull request #155 from fvaneijk/newIsNaNFix

New isNaN fix
Sébastien Ros 10 years ago
parent
commit
965473e14a
2 changed files with 10 additions and 2 deletions
  1. 8 0
      Jint.Tests/Runtime/EngineTests.cs
  2. 2 2
      Jint/Native/Global/GlobalObject.cs

+ 8 - 0
Jint.Tests/Runtime/EngineTests.cs

@@ -1153,5 +1153,13 @@ namespace Jint.Tests.Runtime
                 assert(a[0] === 3);
             ");
         }
+
+        [Fact]
+        public void ShouldReturnTrueForEmptyIsNaNStatement()
+        {
+            RunTest(@"
+                assert(true === isNaN());
+            ");
+        }    
     }
 }

+ 2 - 2
Jint/Native/Global/GlobalObject.cs

@@ -317,7 +317,7 @@ namespace Jint.Native.Global
         /// </summary>
         public static JsValue IsNaN(JsValue thisObject, JsValue[] arguments)
         {
-            var x = TypeConverter.ToNumber(arguments[0]);
+            var x = TypeConverter.ToNumber(arguments.At(0));
             return double.IsNaN(x);
         }
 
@@ -331,7 +331,7 @@ namespace Jint.Native.Global
                 return false;
             }
 
-            var n = TypeConverter.ToNumber(arguments[0]);
+            var n = TypeConverter.ToNumber(arguments.At(0));
             if (double.IsNaN(n) || double.IsInfinity(n))
             {
                 return false;