Use pthread_once to call bindtextdomain only once

CMakeLists.txt

@@ -152,6 +152,25 @@ ENDIF()
 STRING(REPLACE ";" " " libdiscid_OSDEP_STR "${libdiscid_OSDEP_SRCS}")
 MESSAGE(STATUS "Using discid implementation ${libdiscid_OSDEP_STR}")
 
+# check if pthread is available
+IF(ENABLE_NLS)
+	SET(CMAKE_THREAD_PREFER_PTHREAD TRUE)
+	IF(CMAKE_COMPILER_IS_GNUCC OR CMAKE_C_COMPILER MATCHES "clang")
+		# FIND_PACKAGE(Threads) prefers -lpthread over -pthread, -pthread is
+		# recommend though
+		SET(THREADS_HAVE_PTHREAD_ARG "-pthread")
+	ENDIF()
+	FIND_PACKAGE(Threads)
+	IF(CMAKE_USE_PTHREADS_INIT)
+		SET(HAVE_PTHREAD TRUE)
+		SET(libdiscid_LIBS ${LIBDISCID_LIBS} ${CMAKE_THREAD_LIBS_INIT})
+		IF(${CMAKE_THREAD_LIBS_INIT} MATCHES "-pthread")
+			# if we have -pthread, also pass -pthread to CC
+			SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -pthread")
+		ENDIF()
+	ENDIF()
+ENDIF()
+
 ADD_LIBRARY(libdiscid SHARED ${libdiscid_OSDEP_SRCS} ${libdiscid_RCS} src/base64.c src/disc.c src/sha1.c)
 TARGET_LINK_LIBRARIES(libdiscid ${libdiscid_OSDEP_LIBS} ${libdiscid_LIBS})
 SET_TARGET_PROPERTIES(libdiscid PROPERTIES

config-cmake.h.in

@@ -12,6 +12,9 @@
 /* enable string translation (gettext) */
 #cmakedefine ENABLE_NLS
 
+/* pthread is available */
+#cmakedefine HAVE_PTHREAD
+
 /**
  * Values needed by our sha1.h
  */

configure.ac

@@ -87,6 +87,14 @@ AC_CHECK_SIZEOF(long)
 
 AM_GNU_GETTEXT([external])
 AM_GNU_GETTEXT_VERSION(0.18.1)
+if test "$USE_NLS" = "yes"; then
+	AX_PTHREAD([
+		AC_DEFINE(HAVE_PTHREAD, [], "pthread is available")
+		LIBS="$PTHREAD_LIBS $LIBS"
+		CFLAGS="$CLAFGS $PTHREAD_CFLAGS"
+		CC="$PTHREAD_CC"
+	])
+fi
 
 if test "$cross_compiling" = "yes" && test "$os" = "win32"; then
   AC_MSG_WARN([detected cross compilation: disabling tests!!!])

src/disc.c

@@ -43,6 +43,17 @@
 #define TRACK_NUM_IS_VALID(disc, i) \
 	( i >= disc->first_track_num && i <= disc->last_track_num )
 
+#ifdef ENABLE_NLS
+static void init_nls(void) {
+	bindtextdomain(PACKAGE, LOCALEDIR);
+}
+
+#ifdef HAVE_PTHREAD
+#include <pthread.h>
+static pthread_once_t nls_initialized = PTHREAD_ONCE_INIT;
+#endif
+#endif
+
 
 static void create_disc_id(mb_disc_private *d, char buf[]);
 static void create_freedb_disc_id(mb_disc_private *d, char buf[]);
@@ -61,7 +72,11 @@ DiscId *discid_new() {
 	 * just in case the prefix was set weird
 	 */
 #ifdef ENABLE_NLS
-	bindtextdomain(PACKAGE, LOCALEDIR);
+#ifdef HAVE_PTHREAD
+	pthread_once(&nls_initialized, init_nls);
+#else
+	init_nls();
+#endif
 #endif
 	printf(_("one two three\n"));
 	/* initializes everything to zero */