No longer accept and silently convert invalid REAL tokens to zero

aothms · aothms · commit f3531276c84d · 2014-03-22T13:02:21.000Z
diff --git a/src/ifcparse/IfcParse.cpp b/src/ifcparse/IfcParse.cpp
@@ -348,7 +348,11 @@ bool TokenFunc::asBool(const Token& t) {
 }
 double TokenFunc::asFloat(const Token& t) {
 	const std::string str = asString(t);
-	return (double) atof(str.c_str());
+	const char* start = str.c_str();
+	char* end;
+	double result = strtod(start,&end);
+	if ( start == end ) throw IfcException("Token is not a real");
+	return result;
 }
 std::string TokenFunc::asString(const Token& t) {
 	if ( isOperator(t,'$') ) return "";

Original file line number	Diff line number	Diff line change
`@@ -348,7 +348,11 @@ bool TokenFunc::asBool(const Token& t) {`
`348`	`348`	`}`
`349`	`349`	`double TokenFunc::asFloat(const Token& t) {`
`350`	`350`	`const std::string str = asString(t);`
`351`		`- return (double) atof(str.c_str());`
	`351`	`+ const char* start = str.c_str();`
	`352`	`+ char* end;`
	`353`	`+ double result = strtod(start,&end);`
	`354`	`+ if ( start == end ) throw IfcException("Token is not a real");`
	`355`	`+ return result;`
`352`	`356`	`}`
`353`	`357`	`std::string TokenFunc::asString(const Token& t) {`
`354`	`358`	`if ( isOperator(t,'$') ) return "";`