simplify cancellation push/pop slightly
authorRich Felker <dalias@aerifal.cx>
Wed, 23 May 2012 18:13:54 +0000 (14:13 -0400)
committerRich Felker <dalias@aerifal.cx>
Wed, 23 May 2012 18:13:54 +0000 (14:13 -0400)
no need to pass unnecessary extra arguments on to the core code in
pthread_create.c. this just wastes cycles and code bloat.

src/thread/cancellation.c
src/thread/pthread_create.c

index ab7eebcc02a43e5fc75b315acb98ac7f5b137f8f..9b21764b56b0bea71300b30dedaf950248135112 100644 (file)
@@ -1,23 +1,20 @@
 #include "pthread_impl.h"
 
-void __do_cleanup_push();
-void __do_cleanup_pop();
+static void dummy(struct __ptcb *cb)
+{
+}
+weak_alias(dummy, __do_cleanup_push);
+weak_alias(dummy, __do_cleanup_pop);
 
 void _pthread_cleanup_push(struct __ptcb *cb, void (*f)(void *), void *x)
 {
        cb->__f = f;
        cb->__x = x;
-       __do_cleanup_push(cb, f, x);
+       __do_cleanup_push(cb);
 }
 
 void _pthread_cleanup_pop(struct __ptcb *cb, int run)
 {
-       __do_cleanup_pop(cb, run);
+       __do_cleanup_pop(cb);
        if (run) cb->__f(cb->__x);
 }
-
-static void dummy()
-{
-}
-weak_alias(dummy, __do_cleanup_push);
-weak_alias(dummy, __do_cleanup_pop);
index 917be54f2fb6d4c06bf3ebf9fffcd877af511cdd..ecf93a46617116fb6f7313e888256f16c1b1fde3 100644 (file)
@@ -43,14 +43,14 @@ void pthread_exit(void *result)
        __syscall(SYS_exit, 0);
 }
 
-void __do_cleanup_push(struct __ptcb *cb, void (*f)(void *), void *x)
+void __do_cleanup_push(struct __ptcb *cb)
 {
        struct pthread *self = pthread_self();
        cb->__next = self->cancelbuf;
        self->cancelbuf = cb;
 }
 
-void __do_cleanup_pop(struct __ptcb *cb, int run)
+void __do_cleanup_pop(struct __ptcb *cb)
 {
        __pthread_self()->cancelbuf = cb->__next;
 }