tap: WANT_PTHREAD not HAVE_PTHREAD
authorRusty Russell <rusty@rustcorp.com.au>
Mon, 4 Jul 2011 07:27:03 +0000 (16:57 +0930)
committerRusty Russell <rusty@rustcorp.com.au>
Mon, 4 Jul 2011 07:27:03 +0000 (16:57 +0930)
I'm not sure that a "pthread-safe" tap library is very useful; how many
people have multiple threads calling ok()?

Kirill Shutemov noted that it gives a warning with -Wundef; indeed, we
should ask in this case whether they want pthread support, not whether the
system has pthread support to offer.

ccan/tap/tap.c

index b7a84ac9900b398a40687e4e2fedd2bef3049518..7c01e140d5c206c475957f439b47eb3462fff994 100644 (file)
@@ -45,8 +45,13 @@ static int test_died = 0;
 static int test_pid;
 
 /* Encapsulate the pthread code in a conditional.  In the absence of
-   libpthread the code does nothing */
-#if HAVE_LIBPTHREAD
+   libpthread the code does nothing.
+
+   If you have multiple threads calling ok() etc. at the same time you would
+   need this, but in that case your test numbers will be random and I'm not
+   sure it makes sense. --RR
+*/
+#ifdef WANT_PTHREAD
 #include <pthread.h>
 static pthread_mutex_t M = PTHREAD_MUTEX_INITIALIZER;
 # define LOCK pthread_mutex_lock(&M)