Fixed issue where evaluating tagged bools always returned NaN.

fholm · Jul 9, 2011 · 161945d · 161945d
1 parent 7b8fd39
commit 161945d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/Src/IronJS.Runtime/TaggedBools.cs b/Src/IronJS.Runtime/TaggedBools.cs
@@ -17,5 +17,15 @@ public static double ToTagged(bool value)
         {
             return value ? True : False;
         }
+
+        internal static bool IsTrue(double d)
+        {
+            return double.IsNaN(d) && (BitConverter.DoubleToInt64Bits(d) == TrueBitPattern);
+        }
+
+        internal static bool IsFalse(double d)
+        {
+            return double.IsNaN(d) && (BitConverter.DoubleToInt64Bits(d) == TrueBitPattern);
+        }
     }
 }
diff --git a/Src/IronJS.Runtime/TypeConverter.cs b/Src/IronJS.Runtime/TypeConverter.cs
@@ -509,12 +509,12 @@ public static double ToNumber(string s)
 
         public static double ToNumber(double d)
         {
-            if (double.IsNaN(d) && (TaggedBools.True == BitConverter.DoubleToInt64Bits(d)))
+            if (TaggedBools.IsTrue(d))
             {
                 return 1.0;
             }
 
-            if (double.IsNaN(d) && (TaggedBools.False == BitConverter.DoubleToInt64Bits(d)))
+            if (TaggedBools.IsFalse(d))
             {
                 return 0.0;
             }