Changeset 10252:ec524648dbaa in orange

Timestamp:

02/15/12 16:16:42 (15 months ago)

Author:

ales_erjavec

Branch:

default

Message:

A workaround for a strange behavior of 'operator >>' for floats (on Mac OSX gcc 4.2.1). Fixes #1104.

File:

• source/orange/tdidt_simple.cpp (modified) (1 diff)

source/orange/tdidt_simple.cpp

@@ -785 +785 @@
     int i;
     string lbracket, rbracket;
+    string split_string;
     SimpleTreeNode *node;
+
+    ss.exceptions(istream::failbit);
 
     ASSERT(node = (SimpleTreeNode *)malloc(sizeof *node));
     ss >> lbracket >> node->type >> node->children_size;
 
+
     if (node->type != PredictorNode)
-        ss >> node->split_attr >> node->split;
+    {
+        ss >> node->split_attr;
+
+        /* Read split into a string and use strtod to parse it.
+         * istream sometimes (on some platforms) seems to have problems
+         * reading formated floats.
+         */
+        ss >> split_string;
+        node->split = float(strtod(split_string.c_str(), NULL));
+    }
 
     if (node->children_size) {