chiark / gitweb /
test: fix strtod() test
authorDavid Herrmann <dh.herrmann@gmail.com>
Fri, 15 Aug 2014 08:44:43 +0000 (10:44 +0200)
committerDavid Herrmann <dh.herrmann@gmail.com>
Fri, 15 Aug 2014 08:44:43 +0000 (10:44 +0200)
One strtod() test is broken since:

commit 8e211000025940b770794abf5754de61b4add0af
Author: Thomas Hindoe Paaboel Andersen <phomes@gmail.com>
Date:   Mon Aug 4 23:13:31 2014 +0200

    test: use fabs on doubles

The commit was right, so no reason to revert it, but the test was broken
before and only worked by coincidence. Convert "0,5" to "0.5" so we don't
depend on locales for double conversion (or well, we depend on "C" which
seems reasonable).

src/test/test-util.c

index 69e3f5d3ae2dab5da42fa3ce1715cbf6d98e7789..1850f97723b6641d67992f5d07bb4f388a298c75 100644 (file)
@@ -212,7 +212,7 @@ static void test_safe_atod(void) {
         assert_se(r == -EINVAL);
 
         errno = 0;
-        assert_se(fabs(strtod("0,5", &e) - 0.5) < 0.00001);
+        assert_se(fabs(strtod("0.5", &e) - 0.5) < 0.00001);
 
         /* And check again, reset */
         setlocale(LC_NUMERIC, "C");