Joakim Tjernlund writes:

Hi Erik

It seems to me that __pthread_once and __pthread_initialize_minimal could be made
WEAKs with no stub. The code in rpc_thread.c and __uClibc_main.c appears to expect this.
Also, __pthread_return_0 __pthread_return_1 and __pthread_return_void can be static, not to
pollute the name space.

  Jocke

1 files changed, 6 insertions, 10 deletions

diff --git a/libc/misc/pthread/weaks.c b/libc/misc/pthread/weaks.c
index 1320433f3..90b4f5494 100644
--- a/libc/misc/pthread/weaks.c
+++ b/libc/misc/pthread/weaks.c
@@ -21,9 +21,9 @@
 #include <limits.h>
 #include <stdlib.h>
 
-extern int __pthread_return_0 __P ((void));
-extern int __pthread_return_1 __P ((void));
-extern void __pthread_return_void __P ((void));
+static int __pthread_return_0 __P ((void));
+static int __pthread_return_1 __P ((void));
+static void __pthread_return_void __P ((void));
 
 /**********************************************************************/
 /* Weaks for application/library use.
@@ -105,25 +105,21 @@ weak_alias (__pthread_return_0, __pthread_mutex_lock)
 weak_alias (__pthread_return_0, __pthread_mutex_trylock)
 weak_alias (__pthread_return_0, __pthread_mutex_unlock)
 
-/* Weaks used internally by the C library rpc code only. */
-weak_alias (__pthread_return_0, __pthread_once)
-weak_alias (__pthread_return_void, __pthread_initialize_minimal)
-
 /**********************************************************************/
 
-int
+static int
 __pthread_return_0 (void)
 {
   return 0;
 }
 
-int
+static int
 __pthread_return_1 (void)
 {
   return 1;
 }
 
-void
+static void
 __pthread_return_void (void)
 {
 }