From: Bruno Haible <bruno@clisp.org>
Date: Mon, 23 Nov 2009 01:43:23 +0000 (+0100)
Subject: vasnprintf: Tiny optimization.
X-Git-Tag: v0.1~5143
X-Git-Url: http://erislabs.org.uk/gitweb/?a=commitdiff_plain;h=0581dde596c645aab197339a2468f4f16253315e;p=gnulib.git

vasnprintf: Tiny optimization.
---

diff --git a/ChangeLog b/ChangeLog
index ff3d5e526..3bb6922c4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2009-11-22  Bruno Haible  <bruno@clisp.org>
 
+	vasnprintf: Tiny optimization.
+	* lib/vasnprintf.c (decimal_point_char): Choose the fast path also on
+	MacOS X.
+
+2009-11-22  Bruno Haible  <bruno@clisp.org>
+
 	Tests for module 'duplocale'.
 	* modules/duplocale-tests: New file.
 	* tests/test-duplocale.c: New file.
diff --git a/lib/vasnprintf.c b/lib/vasnprintf.c
index ab662888b..faffd9ee9 100644
--- a/lib/vasnprintf.c
+++ b/lib/vasnprintf.c
@@ -261,10 +261,10 @@ decimal_point_char (void)
 {
   const char *point;
   /* Determine it in a multithread-safe way.  We know nl_langinfo is
-     multithread-safe on glibc systems, but is not required to be multithread-
-     safe by POSIX.  sprintf(), however, is multithread-safe.  localeconv()
-     is rarely multithread-safe.  */
-#  if HAVE_NL_LANGINFO && __GLIBC__
+     multithread-safe on glibc systems and MacOS X systems, but is not required
+     to be multithread-safe by POSIX.  sprintf(), however, is multithread-safe.
+     localeconv() is rarely multithread-safe.  */
+#  if HAVE_NL_LANGINFO && (__GLIBC__ || (defined __APPLE__ && defined __MACH__))
   point = nl_langinfo (RADIXCHAR);
 #  elif 1
   char pointbuf[5];