From: Jim Meyering <jim@meyering.net>
Date: Thu, 11 May 2000 07:24:05 +0000 (+0000)
Subject: (usage): Don't call close_stdout_status directly,
X-Git-Tag: FILEUTILS-4_0s~72
X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=9590fbf43cc6f254c305e25792f92d5f69bfbdba;p=thirdparty%2Fcoreutils.git

(usage): Don't call close_stdout_status directly,
since that didn't cover --version output.
(main): Instead, call close_stdout_set_status and arrange to
call close_stdout via atexit.
---

diff --git a/src/printenv.c b/src/printenv.c
index 6ad9692d36..1c6c028f06 100644
--- a/src/printenv.c
+++ b/src/printenv.c
@@ -70,7 +70,6 @@ If no environment VARIABLE specified, print them all.\n\
 "),
 	      program_name, program_name);
       puts (_("\nReport bugs to <bug-sh-utils@gnu.org>."));
-      close_stdout_status (2);
     }
   exit (status);
 }
@@ -90,6 +89,9 @@ main (int argc, char **argv)
   bindtextdomain (PACKAGE, LOCALEDIR);
   textdomain (PACKAGE);
 
+  close_stdout_set_status (2);
+  atexit (close_stdout);
+
   parse_long_options (argc, argv, PROGRAM_NAME, GNU_PACKAGE, VERSION,
 		      AUTHORS, usage);
 
@@ -133,7 +135,5 @@ main (int argc, char **argv)
       exit_status = (matches != argc - optind);
     }
 
-  close_stdout_status (2);
-
   exit (exit_status);
 }
diff --git a/src/tty.c b/src/tty.c
index 04fce48e8b..145f46dd07 100644
--- a/src/tty.c
+++ b/src/tty.c
@@ -69,7 +69,6 @@ Print the file name of the terminal connected to standard input.\n\
 "));
       puts (_("\nReport bugs to <bug-sh-utils@gnu.org>."));
     }
-  close_stdout_status (3);
   exit (status);
 }
 
@@ -84,6 +83,9 @@ main (int argc, char **argv)
   bindtextdomain (PACKAGE, LOCALEDIR);
   textdomain (PACKAGE);
 
+  close_stdout_set_status (3);
+  atexit (close_stdout);
+
   silent = 0;
 
   while ((optc = getopt_long (argc, argv, "s", longopts, NULL)) != -1)
@@ -116,8 +118,6 @@ main (int argc, char **argv)
 	puts (tty);
       else
 	puts (_("not a tty"));
-
-      close_stdout_status (3);
     }
 
   exit (isatty (0) ? 0 : 1);