reorder exit code to defer stdio flush until after dtors

this is required in case dtors use stdio.

also remove the old comments; one was cruft from when the code used to
be using function pointers and conditional calls, and has little
motivation now that we're using weak symbols. the other was just
complaining about having to support dtors even though the cost was
made essentially zero in the non-use case by the way it's done here.

diff --git a/src/exit/exit.c b/src/exit/exit.c
index fc29148..03c46ca 100644 (file)
--- a/src/exit/exit.c
+++ b/src/exit/exit.c
@@ -20,13 +20,10 @@ void exit(int code)
        /* If more than one thread calls exit, hang until _Exit ends it all */
        while (a_swap(&lock, 1)) __syscall(SYS_pause);
 
-       /* Only do atexit & stdio flush if they were actually used */
        __funcs_on_exit();
-       __fflush_on_exit();
-
-       /* Destructor s**t is kept separate from atexit to avoid bloat */
        if (libc.fini) libc.fini();
        if (libc.ldso_fini) libc.ldso_fini();
+       __fflush_on_exit();
 
        _Exit(code);
        for(;;);