simplify atexit and fflush-on-exit handling
authorRich Felker <dalias@aerifal.cx>
Sat, 15 Oct 2011 03:00:24 +0000 (23:00 -0400)
committerRich Felker <dalias@aerifal.cx>
Sat, 15 Oct 2011 03:00:24 +0000 (23:00 -0400)
src/exit/atexit.c
src/exit/exit.c
src/stdio/__towrite.c

index 49c060e..6f00e37 100644 (file)
@@ -12,18 +12,15 @@ static struct fl
        void (*f[COUNT])(void);
 } builtin, *head;
 
        void (*f[COUNT])(void);
 } builtin, *head;
 
-static int run_atexit_functions(void)
+void __funcs_on_exit()
 {
        int i;
        for (; head; head=head->next) {
                for (i=COUNT-1; i>=0 && !head->f[i]; i--);
                for (; i>=0; i--) head->f[i]();
        }
 {
        int i;
        for (; head; head=head->next) {
                for (i=COUNT-1; i>=0 && !head->f[i]; i--);
                for (; i>=0; i--) head->f[i]();
        }
-       return 0;
 }
 
 }
 
-int (*const __funcs_on_exit)(void) = run_atexit_functions;
-
 int atexit(void (*func)(void))
 {
        static int lock;
 int atexit(void (*func)(void))
 {
        static int lock;
index bfdb392..1ff19db 100644 (file)
@@ -3,8 +3,11 @@
 #include <stdio.h>
 #include "libc.h"
 
 #include <stdio.h>
 #include "libc.h"
 
-/* __overflow.c and atexit.c override these */
-static int (*const dummy)() = 0;
+static void dummy()
+{
+}
+
+/* __towrite.c and atexit.c override these */
 weak_alias(dummy, __funcs_on_exit);
 weak_alias(dummy, __fflush_on_exit);
 
 weak_alias(dummy, __funcs_on_exit);
 weak_alias(dummy, __fflush_on_exit);
 
@@ -16,8 +19,8 @@ void exit(int code)
        LOCK(&lock);
 
        /* Only do atexit & stdio flush if they were actually used */
        LOCK(&lock);
 
        /* Only do atexit & stdio flush if they were actually used */
-       if (__funcs_on_exit) __funcs_on_exit();
-       if (__fflush_on_exit) __fflush_on_exit((void *)0);
+       __funcs_on_exit();
+       __fflush_on_exit();
 
        /* Destructor s**t is kept separate from atexit to avoid bloat */
        if (libc.fini) libc.fini();
 
        /* Destructor s**t is kept separate from atexit to avoid bloat */
        if (libc.fini) libc.fini();
index b458741..4bf96f4 100644 (file)
@@ -18,4 +18,7 @@ int __towrite(FILE *f)
 }
 
 /* Link flush-on-exit code iff any stdio write functions are linked. */
 }
 
 /* Link flush-on-exit code iff any stdio write functions are linked. */
-int (*const __fflush_on_exit)(FILE *) = fflush;
+void __fflush_on_exit()
+{
+       fflush(0);
+}