Fix Infinity and NaN to int cast

Author: adazem009

src/scratch/value_functions.cpp

@@ -192,7 +192,7 @@ extern "C"
     long value_toLong(const libscratchcpp::ValueData *v)
     {
         if (v->type == ValueType::Number) {
-            return v->numberValue;
+            return std::isnan(v->numberValue) || std::isinf(v->numberValue) ? 0 : v->numberValue;
         } else if (v->type == ValueType::Bool)
             return v->boolValue;
         else if (v->type == ValueType::String)
@@ -205,7 +205,7 @@ extern "C"
     int value_toInt(const libscratchcpp::ValueData *v)
     {
         if (v->type == ValueType::Number)
-            return v->numberValue;
+            return std::isnan(v->numberValue) || std::isinf(v->numberValue) ? 0 : v->numberValue;
         else if (v->type == ValueType::Bool)
             return v->boolValue;
         else if (v->type == ValueType::String)

test/scratch_classes/value_test.cpp

@@ -653,6 +653,13 @@ TEST(ValueTest, ToInt)
     v = "NaN";
     ASSERT_EQ(v.toInt(), 0);
 
+    v = std::numeric_limits<double>::infinity();
+    ASSERT_EQ(v.toInt(), 0);
+    v = -std::numeric_limits<double>::infinity();
+    ASSERT_EQ(v.toInt(), 0);
+    v = std::numeric_limits<double>::quiet_NaN();
+    ASSERT_EQ(v.toInt(), 0);
+
     v = "something";
     ASSERT_EQ(v.toInt(), 0);
 
@@ -795,6 +802,13 @@ TEST(ValueTest, ToLong)
     v = "NaN";
     ASSERT_EQ(v.toLong(), 0);
 
+    v = std::numeric_limits<double>::infinity();
+    ASSERT_EQ(v.toLong(), 0);
+    v = -std::numeric_limits<double>::infinity();
+    ASSERT_EQ(v.toLong(), 0);
+    v = std::numeric_limits<double>::quiet_NaN();
+    ASSERT_EQ(v.toLong(), 0);
+
     v = "something";
     ASSERT_EQ(v.toLong(), 0);