[pulseaudio-discuss] [PATCH 1/2] build-sys: check strtod_l instead of strtof_l
Tanu Kaskinen
tanu.kaskinen at linux.intel.com
Wed May 20 12:35:27 PDT 2015
We have no strtof_l calls in the code, so it doesn't make sense to
check that function's availability. We have one strtod_l call, so
let's check that instead.
I don't know if this change makes any practical difference. I just
wondered why we had HAVE_STRTOF_L ifdefs in core-util.c for code that
didn't use strtof_l.
---
configure.ac | 2 +-
src/pulsecore/core-util.c | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/configure.ac b/configure.ac
index b2fcd1e..00a1951 100644
--- a/configure.ac
+++ b/configure.ac
@@ -572,7 +572,7 @@ AC_CHECK_FUNCS_ONCE([strerror_r])
AC_CHECK_FUNCS_ONCE([lstat])
# Non-standard
-AC_CHECK_FUNCS_ONCE([setresuid setresgid setreuid setregid seteuid setegid ppoll strsignal sig2str strtof_l pipe2 accept4])
+AC_CHECK_FUNCS_ONCE([setresuid setresgid setreuid setregid seteuid setegid ppoll strsignal sig2str strtod_l pipe2 accept4])
AC_FUNC_ALLOCA
diff --git a/src/pulsecore/core-util.c b/src/pulsecore/core-util.c
index ad5b2d2..824368a 100644
--- a/src/pulsecore/core-util.c
+++ b/src/pulsecore/core-util.c
@@ -52,7 +52,7 @@
#include <pcreposix.h>
#endif
-#ifdef HAVE_STRTOF_L
+#ifdef HAVE_STRTOD_L
#include <locale.h>
#endif
@@ -2401,7 +2401,7 @@ int pa_atol(const char *s, long *ret_l) {
return 0;
}
-#ifdef HAVE_STRTOF_L
+#ifdef HAVE_STRTOD_L
static locale_t c_locale = NULL;
static void c_locale_destroy(void) {
@@ -2431,7 +2431,7 @@ int pa_atod(const char *s, double *ret_d) {
/* This should be locale independent */
-#ifdef HAVE_STRTOF_L
+#ifdef HAVE_STRTOD_L
PA_ONCE_BEGIN {
--
1.9.3
More information about the pulseaudio-discuss
mailing list