]> Savannah Git Hosting - gnulib.git/commitdiff
hard-locale: Make multithread-safe.
authorBruno Haible <bruno@clisp.org>
Wed, 18 Dec 2019 10:42:21 +0000 (11:42 +0100)
committerBruno Haible <bruno@clisp.org>
Wed, 18 Dec 2019 10:42:21 +0000 (11:42 +0100)
* lib/hard-locale.h (hard_locale): Move documentation to here.
* lib/hard-locale.c: Don't include <stdlib.h>.
(GLIBC_VERSION): Remove macro.
(hard_locale): Assume that all systems name the "C" and "POSIX" locales
"C" or "POSIX". Invoke setlocale_null instead of setlocale.
* modules/hard-locale (Depends-on): Remove strdup. Add setlocale-null.
(configure.ac): Require gl_FUNC_SETLOCALE_NULL. Set LIB_HARD_LOCALE.
(Link): New section.
* modules/hard-locale-tests (Makefile.am): Link test-hard-locale against
$(LIB_HARD_LOCALE).

ChangeLog
lib/hard-locale.c
lib/hard-locale.h
modules/hard-locale
modules/hard-locale-tests

index 2f41e5e8a711d715fcbdbcd19512c4403df381b9..e2c33fb163b2674d4f6c82fde564dbdd78aa6e61 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2019-12-18  Bruno Haible  <bruno@clisp.org>
+
+       hard-locale: Make multithread-safe.
+       * lib/hard-locale.h (hard_locale): Move documentation to here.
+       * lib/hard-locale.c: Don't include <stdlib.h>.
+       (GLIBC_VERSION): Remove macro.
+       (hard_locale): Assume that all systems name the "C" and "POSIX" locales
+       "C" or "POSIX". Invoke setlocale_null instead of setlocale.
+       * modules/hard-locale (Depends-on): Remove strdup. Add setlocale-null.
+       (configure.ac): Require gl_FUNC_SETLOCALE_NULL. Set LIB_HARD_LOCALE.
+       (Link): New section.
+       * modules/hard-locale-tests (Makefile.am): Link test-hard-locale against
+       $(LIB_HARD_LOCALE).
+
 2019-12-18  Bruno Haible  <bruno@clisp.org>
 
        hard-locale: Avoid test failure on Haiku.
index dcfcad62ee7cf86204109b07a7c358e829e9ba94..4a2adabb7e07a3e42da9a6bf1bd4b6e34397d9e0 100644 (file)
 #include "hard-locale.h"
 
 #include <locale.h>
-#include <stdlib.h>
 #include <string.h>
 
-#ifdef __GLIBC__
-# define GLIBC_VERSION __GLIBC__
-#elif defined __UCLIBC__
-# define GLIBC_VERSION 2
-#else
-# define GLIBC_VERSION 0
-#endif
-
-/* Return true if the current CATEGORY locale is hard, i.e. if you
-   can't get away with assuming traditional C or POSIX behavior.  */
 bool
 hard_locale (int category)
 {
-  bool hard = true;
-  char const *p = setlocale (category, NULL);
-
-  if (p)
-    {
-      if (2 <= GLIBC_VERSION)
-        {
-          if (strcmp (p, "C") == 0 || strcmp (p, "POSIX") == 0)
-            hard = false;
-        }
-      else
-        {
-          char *locale = strdup (p);
-          if (locale)
-            {
-              /* Temporarily set the locale to the "C" and "POSIX" locales
-                 to find their names, so that we can determine whether one
-                 or the other is the caller's locale.  */
-              if (((p = setlocale (category, "C"))
-                   && strcmp (p, locale) == 0)
-                  || ((p = setlocale (category, "POSIX"))
-                      && strcmp (p, locale) == 0))
-                hard = false;
+  char locale[SETLOCALE_NULL_MAX];
 
-              /* Restore the caller's locale.  */
-              setlocale (category, locale);
-              free (locale);
-            }
-        }
-    }
+  if (setlocale_null (category, locale, sizeof (locale)))
+    return false;
 
-  return hard;
+  return !(strcmp (locale, "C") == 0 || strcmp (locale, "POSIX") == 0);
 }
index 8f1da96e2f252abf418f41c721b732724f1fcb23..160674ba7a572df7d7338fe75dfcd2c8f0c5b00f 100644 (file)
@@ -20,6 +20,9 @@
 
 # include <stdbool.h>
 
-bool hard_locale (int);
+/* Return true if the specified CATEGORY of the current locale is hard, i.e.
+   different from the C or POSIX locale that has a fixed behavior.
+   CATEGORY must be one of the LC_* values, but not LC_ALL.  */
+extern bool hard_locale (int category);
 
 #endif /* HARD_LOCALE_H_ */
index d4463b74bd90856fb412022d4200f8fa1333a79e..df07d4a6ed5df27ad8aa4bfc913202c2c97726df 100644 (file)
@@ -7,9 +7,12 @@ lib/hard-locale.c
 
 Depends-on:
 stdbool
-strdup
+setlocale-null
 
 configure.ac:
+AC_REQUIRE([gl_FUNC_SETLOCALE_NULL])
+LIB_HARD_LOCALE="$LIB_SETLOCALE_NULL"
+AC_SUBST([LIB_HARD_LOCALE])
 
 Makefile.am:
 lib_SOURCES += hard-locale.c
@@ -17,6 +20,9 @@ lib_SOURCES += hard-locale.c
 Include:
 "hard-locale.h"
 
+Link:
+$(LIB_HARD_LOCALE)
+
 License:
 LGPLv2+
 
index cdc084eba4ef04c4ee7fb563a448946b9f0f5bfd..a501781d8d50ba266290e1767b7f14a735eed494 100644 (file)
@@ -16,3 +16,4 @@ Makefile.am:
 TESTS += test-hard-locale
 check_PROGRAMS += test-hard-locale
 noinst_PROGRAMS += locale
+test_hard_locale_LDADD = $(LDADD) @LIB_HARD_LOCALE@