Mercurial > hg > octave-lojdl > gnulib-hg
changeset 5445:fe44cfd53b9d
Include "__fpending.h" once again.
Include <stdbool.h>.
(close_stdout): Don't fail just because stdout was closed initially,
since some programs don't write to stdout in the normal course of
operation (other than --version and --help), and we don't want this
function to make e.g. `touch file >&-' fail.
But do fail if it was closed and someone has tried to write to it.
E.g., `printf foo >&-' must fail.
author | Jim Meyering <jim@meyering.net> |
---|---|
date | Mon, 15 Nov 2004 17:45:32 +0000 |
parents | 68f8bfa26b00 |
children | a44ff3864d31 |
files | lib/closeout.c |
diffstat | 1 files changed, 19 insertions(+), 6 deletions(-) [+] |
line wrap: on
line diff
--- a/lib/closeout.c +++ b/lib/closeout.c @@ -24,6 +24,7 @@ #include "closeout.h" #include <stdio.h> +#include <stdbool.h> #include <errno.h> #include "gettext.h" @@ -32,6 +33,7 @@ #include "error.h" #include "exitfail.h" #include "quotearg.h" +#include "__fpending.h" #if USE_UNLOCKED_IO # include "unlocked-io.h" @@ -71,14 +73,25 @@ void close_stdout (void) { - int e = ferror (stdout) ? 0 : -1; + bool prev_fail = ferror (stdout); + bool none_pending = (0 == __fpending (stdout)); + bool fclose_fail = fclose (stdout); + + if (prev_fail || fclose_fail) + { + int e = fclose_fail ? errno : 0; + char const *write_error; - if (fclose (stdout) != 0) - e = errno; + /* If ferror returned zero, no data remains to be flushed, and we'd + otherwise fail with EBADF due to a failed fclose, then assume that + it's ok to ignore the fclose failure. That can happen when a + program like cp is invoked like this `cp a b >&-' (i.e., with + stdout closed) and doesn't generate any output (hence no previous + error and nothing to be flushed). */ + if (e == EBADF && !prev_fail && none_pending) + return; - if (0 <= e) - { - char const *write_error = _("write error"); + write_error = _("write error"); if (file_name) error (exit_failure, e, "%s: %s", quotearg_colon (file_name), write_error);