X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/044363451d877b3c9a278db74763c6c9991e49f0..62bfda03da9ba46273bbbffe6be1df498b21d991:/sbr/error.c?ds=sidebyside diff --git a/sbr/error.c b/sbr/error.c index e4bc9e3f..b33eae54 100644 --- a/sbr/error.c +++ b/sbr/error.c @@ -29,9 +29,9 @@ advise (const char *what, const char *fmt, ...) /* adios calls advertise() with no tail to print fmt, and perhaps what, * to stderr, and "ends" the program with an error exit status. The * route to exit is via the done function pointer and may not be - * straightforward. - * FIXME: Document if this function can ever return. If not, perhaps an - * abort(3) at the end of the routine would make that more clear. */ + * straightforward, e.g. longjmp(3), but it must not return to adios(). + * If it does then it's a bug and adios() will abort(3) as callers do + * not expect execution to continue. */ void adios (const char *what, const char *fmt, ...) { @@ -41,6 +41,7 @@ adios (const char *what, const char *fmt, ...) advertise (what, NULL, fmt, ap); va_end(ap); done (1); + abort(); }