Changeset 10415 for OpenSceneGraph/trunk/src/osg/ArgumentParser.cpp

Timestamp:

06/25/09 18:07:49 (4 years ago)

Author:

robert

Message:

From Mathias Froehlich, "We are currently getting issues with locale settings and some osg plugins.
Therefore I have changed all the occurances of atof by asciiToFloat or
asciiToDouble.

I believe that it is safe to do so at least for all the plugins.
Included here are also asciiToFloat conversion of environment variables. One
might argue that these should be locale dependent. But IMO these should be
set and interpreted by osg independent of the current locale.
"

Files:

• OpenSceneGraph/trunk/src/osg/ArgumentParser.cpp (modified) (2 diffs)

OpenSceneGraph/trunk/src/osg/ArgumentParser.cpp

@@ -17 +17 @@
 #include <osg/ArgumentParser>
 #include <osg/ApplicationUsage>
+#include <osg/Math>
 #include <osg/Notify>
 
@@ -164 +165 @@
             break;
         }
-        case Parameter::FLOAT_PARAMETER:        *_value._float = atof(str); break;
-        case Parameter::DOUBLE_PARAMETER:       *_value._double = atof(str); break;
+        case Parameter::FLOAT_PARAMETER:        *_value._float = osg::asciiToFloat(str); break;
+        case Parameter::DOUBLE_PARAMETER:       *_value._double = osg::asciiToDouble(str); break;
         case Parameter::INT_PARAMETER:          *_value._int = atoi(str); break;
         case Parameter::UNSIGNED_INT_PARAMETER: *_value._uint = atoi(str); break;